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

Add an option to set a destination directory for the simulation output #33

Closed
wants to merge 2 commits into from

Conversation

anabramo
Copy link

@anabramo anabramo commented Nov 1, 2023

I've added an option to specify a destination directory for the simulation data. This is to support submission from AFS, but file storage on EOS. When an output directory is specified, the job directory structure is replicated there and the output from the jobs is routed there via job script amendment. The default for this new option is None and the submission behaves as it does now.

Copy link

codeclimate bot commented Nov 2, 2023

Code Climate has analyzed commit e0a99ab and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 47.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 65.0% (-0.7% change).

View more on Code Climate.

@JoschD
Copy link
Member

JoschD commented Nov 2, 2023

Moved to #34 . The problem with the failing test was, that a PR from a fork does not have access to the repo-secrets.

@JoschD JoschD closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants