-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Desired features (30.08.2024) #5
Comments
Hey Tara, can you elaborate on this a bit? The rest are very clearly GUI issues to me, but it sounds like this might possibly require changes of the back-end |
Hey Liam, as it stands, the GUI has two buttons on each node: a button to view the source code and another to run the node (which also displays the output). If we have a node with a pyiron job (e.g. LAMMPS), then there's not much of an issue since the node would just reload the existing job unless specified otherwise by the user. But if we have a node that has a large-ish analysis that is not currently included in pyiron, for e.g. some scipy opitmization, then it would be ideal if pressing "run" just displayed the already performed analysis instead of re-running it (unless specified otherwise by the user). |
Ah. I think I'd rather call this "caching" than "hashing". Good news is that the underlying `pyiron_workflow infrastructure already does this by default -- as long as the inputs are the same as the last time the node was run. If the inputs aren't the same I guess you do want to actually rerun it anyhow. |
We still need to sort out the versions of the pyiron_wokflow module, pyiron_base, pyiron_atomistic etc. I think we are still using a slightly older version of workflow which does not have caching/hashing due to compatibility issues. |
|
Currently desired features (30.08.2024)
The text was updated successfully, but these errors were encountered: