Skip to content

New Slurm Job Scripts densmap-z_*.sh to Run the MDTools Script densmap.py #144

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

Merged
merged 13 commits into from
Dec 3, 2023

Conversation

andthum
Copy link
Owner

@andthum andthum commented Dec 1, 2023

New Slurm Job Scripts densmap-z_*.sh to Run the MDTools Script densmap.py

Type of Change

  • Bug fix.
  • New feature.
  • Code refactoring.
  • Dependency update.
  • Documentation update.
  • Maintenance.
  • Other: Description.
  • Non-breaking (backward-compatible) change.
  • Breaking (non-backward-compatible) change.

Proposed Changes

  • Create new Slurm job scripts densmap-z_Li.sh, densmap-z_NBT.sh, densmap-z_OBT.sh, densmap-z_OE.sh, densmap-z_NTf2.sh, densmap-z_ether.sh that run the MDTools script scripts/structure/densmap.py for different compounds.
  • Bash script .cleanup_gmx_analyses.sh: Create compound sub-directories for densmap-z_*.sh and rdf_slab-z_*.sh analyses.
  • Python submit scripts submit_gmx_analyses_lintf2_ether.py and submit_mdt_analyses_lintf2_ether.py: Use absolute path to Slurm job scripts.
  • Slurm job scripts lig_change_at_pos_change_blocks_*.sh: Increase Memory Limit

PR Checklist

  • I followed the guidelines in the Developer's Guide.
  • New/changed code is properly tested.
  • New/changed code is properly documented.
  • The CI workflow is passing.

Create a Slurm job script that runs the MDTools script
`scripts/structure/densmap.py` for lithium ions.
Create a Slurm job script that runs the MDTools script
`scripts/structure/densmap.py` for nitrogen atoms of TFSI anions.
Create a Slurm job script that runs the MDTools script
`scripts/structure/densmap.py` for oxygen atoms of TFSI anions.
Create a Slurm job script that runs the MDTools script
`scripts/structure/densmap.py` for ether oxygens.
@github-actions github-actions bot added enhancement New feature or request minor Change that results in a new minor version labels Dec 1, 2023
Create a Slurm job script that runs the MDTools script
`scripts/structure/densmap.py` for the center of mass of TFSI anions.
Create a Slurm job script that runs the MDTools script
`scripts/structure/densmap.py` for nitrogen atoms of PEO chains.
@andthum andthum force-pushed the feat/analysis/lintf2-ether/mdt/densmap-z branch from 4c5fc06 to a01f38b Compare December 1, 2023 16:56
So far, the Python submit script did not submit the Slurm job script
from the same directory as the Python submit script but from the
`${PATH}` variable.

Now, absolute paths are used to submit the Slurm job scripts that lie
within the same directory as the Python submit script.
So far, the Python submit script did not submit the Slurm job script
from the same directory as the Python submit script but from the
`${PATH}` variable.

Now, absolute paths are used to submit the Slurm job scripts that lie
within the same directory as the Python submit script.
@andthum andthum force-pushed the feat/analysis/lintf2-ether/mdt/densmap-z branch from a01f38b to c8f4f8a Compare December 3, 2023 10:39
@andthum andthum merged commit 48aec85 into main Dec 3, 2023
@andthum andthum deleted the feat/analysis/lintf2-ether/mdt/densmap-z branch December 3, 2023 10:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor Change that results in a new minor version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant