Check ws allocation
Furthermore, I think we should remove the Gaussian part from the example in order to make it a more general one. @jitschin--tu-dresden.de What do you think about this?
Merge request reports
Activity
requested review from @jitschin--tu-dresden.de
assigned to @mflehmig--tu-dresden.de
added 1 commit
- e52373a3 - Put the Data Lifecycle example for Gaussian on Gaussian's page
added 1 commit
- 984d9bb1 - Imported improved sbatch script, Work in Progress
added 1 commit
- ccb7c370 - I generalized Gaussian's WorkSpace usage example
Summary of my Changes
As suggested I generalized the example usage. Since I would very much dislike to be deleting knowledge, I preserved large parts of the original example for Gaussian.
Additionally I opted for using
bzip2instead ofcpto move the files, in a way to respond to a User Request about ensuring the data integrity of computational results.With the just mentioned commit I also introduced the SBatch constraint, as is recommended elsewhere in the documentation, to improve the reliability of the SBatch script.
Furthermore - since now three conditions need to be fulfilled for the actual removal step to take place (
ws_allocategoing through,bzip2compression going through and WORKSPACE_DIR being actually there) - I removed the admonition, pertaining to a now increasingly unlikely occurrence.Review
Baring some likely inevitable typographical errors, I believe the changes introduced with this Merge Request are ready to be merged.
I would like to thank @mflehmig--tu-dresden.de for his consideration of his matter and his suggested improvements, which I believe fulfill the criteria for 'mergeability'.
Due to my presently limited working hours (refer to
AbwesenheitHPCfor your kind consideration), I suggest to proceed with this Merge Request thusly:Allowing time for further comments and objections I would like to plan to merge this Issue on Friday the 2nd of September.
added 2 commits
added 2 commits
@jitschin--tu-dresden.de I request a final review from your side.