Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Re-enable read-only flag on output of depstrip #99

Open
OndrejMarsalek opened this issue Nov 8, 2015 · 0 comments
Open

Re-enable read-only flag on output of depstrip #99

OndrejMarsalek opened this issue Nov 8, 2015 · 0 comments

Comments

@OndrejMarsalek
Copy link
Collaborator

The read-only flag here:

https://github.com/ceriottm/i-pi-dev/blob/f5817d977fc4f8ead76b768d90fcb2357ceabe05/ipi/utils/depend.py#L690

is generally a good idea, as the output of depstrip is a bare Numpy array and should not be modified, as that bypasses the depend mechanism. However, as it was not enforced so far, there are places in the code with assignment to the output of depstrip. Thus, some testing and fixing is needed before this goes in master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant