Script updates #15
Script updates #15
Conversation
… included torchrun_jsc in slurm
There was a problem hiding this comment.
please do not push your current configs if you just changed vars, otherwise each config has different vals
There was a problem hiding this comment.
you forgit that comment i think
There was a problem hiding this comment.
I thought this was as an advice for the future. What did I do wrong here? I am sorry, I didn't understand.
dynamic_inputs.yaml
Outdated
There was a problem hiding this comment.
remove please, i removed it before in your branch, but somehow it got added again
There was a problem hiding this comment.
please also update vega and lumi runall and slurm in the same PR otherwise there functionality diverges
static_inputs.yaml
Outdated
There was a problem hiding this comment.
same as before I commited on your branch with those removed, now they appeared again apparently
lineick
left a comment
lineick
left a comment
There was a problem hiding this comment.
btw, if you resolved comments, you can close them by clicking on resolve conversation so it does not get crowded
There was a problem hiding this comment.
you forgit that comment i think
There was a problem hiding this comment.
did you test this on lumi? making sure it works with hython
There was a problem hiding this comment.
I don't have access to lumi yet
Summary
Standardized Slurm scripts to have consistent variable names and commands, making them easier to read; and updated juwels config parameters like date window, sequence length, and downsampling fraction