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 PR test routine/ChangeLog #294

Merged
merged 6 commits into from
Nov 5, 2024
Merged

Conversation

ahagen-pik
Copy link
Contributor

Adds a routine to check and archive the changes that will be introduced with each bigger PR.

Please use this after incrementing the version with buildlibrary and before opening the PR

Usage

It can be called via
./test-standard-runs <folderName> <refFolder> <path>
in /p/projects/edget/PRchangeLog/.

[folderName] name of the new folder to be generated, relating to the PR

[refFolder ] optional, any unique substring of the folder name as identifier, runs from this folder are chosen for compScens, shares path with folderName

[path ] optional, path to folderName and refFolder, should end with "/"

Functionality

  • creates a folder YYYYMMDD_folderName in path
  • interactively initializes renv in YYYYMMDD_folderName , individual packages can be installed from source via:
    - /local/path/to/repo on the cluster
    - username/repo@brach from github
  • calls the Rscript PRtestEDGET.R using slurm
    - runs edgeT SA for four default scenarios on the cluster
    - generates extended compScens to compare each scenario run to a previous version

The script submit_compScenPR.sh calls compScenPR.R and can be used to generate additional compScens with another reference folder. Its usage is:
sbatch submit_compScenPR.sh <reference>
where reference can be any unique substring of the folder name in path that should be used for reference.

@ahagen-pik ahagen-pik merged commit bda9e78 into pik-piam:master Nov 5, 2024
1 of 2 checks passed
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.

None yet

2 participants