Skip to content

Conversation

@andthum
Copy link
Owner

@andthum andthum commented Nov 3, 2022

[submit_mdt_analyses_lintf2_ether.py]: Additional --scripts Options

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

Add the additional --script options

  • 11.3: All "normal" bulk renewal event lifetimes.
  • 11.4: All spatially discretized renewal event lifetimes.

This allows users to submit "normal" bulk renewal event lifetimes and spatially discretized renewal event lifetimes separately which was not possible before.

Additionally, remove the analysis scripts
renewal_events_Li-ether_state_lifetime.sh and
renewal_events_Li-NTf2_state_lifetime.sh from the list of scripts that require the spatially discretized
${settings}_${system}_discrete-z_Li_dtrj.npy trajectory. The scripts were listed wrongly there.

PR Checklist

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

Add the additional `--script` options

* 11.3: All "normal" bulk renewal event lifetimes.
* 11.4: All spatially discretized renewal event lifetimes.

This allows users to submit "normal" bulk renewal event lifetimes and
spatially discretized renewal event lifetimes separately which was not
possible before.

Additionally, remove the analysis scripts
`renewal_events_Li-ether_state_lifetime.sh` and
`renewal_events_Li-NTf2_state_lifetime.sh` from the list of scripts that
require the spatially discretized
`${settings}_${system}_discrete-z_Li_dtrj.npy` trajectory.  The scripts
were listed wrongly there.
@github-actions github-actions bot added enhancement New feature or request minor Change that results in a new minor version labels Nov 3, 2022
@andthum andthum merged commit 63bc9db into main Nov 3, 2022
@andthum andthum deleted the feat/submit-mdt-analyses/new-scripts-options branch November 3, 2022 10:25
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.

2 participants