-
Notifications
You must be signed in to change notification settings - Fork 219
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
Upcoming changes on stashing #6764
Labels
type/accepted feature
approved feature request
Comments
What are your goals regarding this issue and release v2.7.0? |
sorry @agoscinski, somehow I overlooked this issue, didn't see your question. So, yeah.. let's see how fast it goes. I'd say whetever we merge from top to bottom we include, the rest can wait. 🤷♂ |
This was referenced Feb 28, 2025
khsrali
added a commit
to khsrali/aiida-core
that referenced
this issue
Mar 7, 2025
…xecmanager.py` to support compressed file formats while stashing. It's implemetation followed the instruction given in class:`RemoteStashData`. It's deployment in `execmanager` follows the same logic as of `RemoteStashFolderData`, meaning it directly calls on Transport methods. This commit is a part of enhancements for stashing feature: aiidateam#6764
khsrali
added a commit
that referenced
this issue
Mar 7, 2025
…xecmanager.py` to support compressed file formats while stashing. It's implemetation followed the instruction given in class:`RemoteStashData`. It's deployment in `execmanager` follows the same logic as of `RemoteStashFolderData`, meaning it directly calls on Transport methods. This commit is a part of enhancements for stashing feature: #6764
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I plan a series of development that I'm bringing to stashing.
To avoid 3-4 K line of PR, which would be hard to review I choose doing it in several PRs, that each individual brings a piece. The stashing so far, is only supported as an attached
step
to a generic calcjob.The upcoming changes are as following:
TARGZ
,etc.Transport
: featcompress
&extract
methods #6743RemoteStashCompressedData
: new data class providing compression modes when stashing #6746StashCalculation
: a newCalcJob
plugin #6772step
within a generic calcjob.RemoteStashCustomData
: a class for stashed data via custom script, and the implementation of stashing via custom script #6777A supporting verdi cli, perhapsverdi stash
that run those calcjobs.@agoscinski for your info --since you showed interest to review these changes 😉
The text was updated successfully, but these errors were encountered: