Skip to content

Dynamically calculate calibration_resuts path#2362

Open
0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q wants to merge 1 commit into
remindmodel:developfrom
0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q:dev/fix_calibration_results_remind_repos
Open

Dynamically calculate calibration_resuts path#2362
0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q wants to merge 1 commit into
remindmodel:developfrom
0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q:dev/fix_calibration_results_remind_repos

Conversation

@0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q
Copy link
Copy Markdown
Member

Change calibration_results/.Rprofile_calibration_results to use the current directory the file is located in for the remind_repos option to allow for moving project directories after the calibration_results directory has been set up.
See https://mattermost.pik-potsdam.de/rd3/pl/iiionod59bdquxpj6io7z87bur

@robertpietzcker
Copy link
Copy Markdown
Contributor

Thanks! sounds great, I just can't comment on the code - I don't know/understand these scripts. so maybe add Lavinia or David as reviewer?

this would lead to an automatic change, or one would need to specify something in the call?

@0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q 0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q force-pushed the dev/fix_calibration_results_remind_repos branch 2 times, most recently from 9af9b88 to fb5d30e Compare June 5, 2026 11:46
Change calibration_results/Rprofile_calibration_results to use the
current directory the file is located in for the remind_repos option to
allow for moving project directories after the calibration_results
directory has been set up.
See https://mattermost.pik-potsdam.de/rd3/pl/iiionod59bdquxpj6io7z87bur
@0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q 0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q force-pushed the dev/fix_calibration_results_remind_repos branch from fb5d30e to eb90cc3 Compare June 5, 2026 11:49
@0UmfHxcvx5J7JoaOhFSs5mncnisTJJ6q
Copy link
Copy Markdown
Member Author

this would lead to an automatic change, or one would need to specify something in the call?

With this change, when make set-local-calibration is run, REMIND is set up in such a way that the directory ./calibration_results/.Rprofile_calibration_results is in will be added to the remind_repos option at runtime, so it can be copied around without impact.

No change to existing ./calibration_results/ directories, but one can copy

cp ./scripts/utils/set-local-calibration/.Rprofile_calibration_results ./calibration_results/

and then it will work.

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