Skip to content
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

AWG Update runtest script for Amber #354

Merged
merged 2 commits into from
Mar 23, 2024
Merged

Conversation

agoetz
Copy link
Collaborator

@agoetz agoetz commented Mar 23, 2024

In Amber

  • Assume this runtest script gets installed into $AMBERHOME/AmberTools/src/quick/
  • While binaries are in $AMBERHOME/bin.

This update will make sure we find the binary folder. It also removes the Run.tests.amber script which is redundant and does not work as intended.

Andreas Goetz added 2 commits March 22, 2024 19:00
In Amber
- Assume this runtest script gets installed into $AMBERHOME/AmberTools/src/quick/
- While binaries are in $AMBERHOME/bin.

This update will make sure we find the binary folder.
This script is not required. We should use runtest instead.
@agoetz agoetz requested a review from Madu86 March 23, 2024 02:16
@agoetz agoetz added the Code cleanup Code cleanup or refactoring label Mar 23, 2024
@Madu86
Copy link
Collaborator

Madu86 commented Mar 23, 2024

Thanks. This resolves #353.

@Madu86 Madu86 merged commit af68ab7 into merzlab:master Mar 23, 2024
4 checks passed
@Madu86 Madu86 deleted the runtest-amber branch March 23, 2024 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code cleanup Code cleanup or refactoring
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants