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

[WIP] branch with discrete crystal env experiments #224

Closed
wants to merge 36 commits into from

Conversation

vict0rsch
Copy link
Collaborator

@vict0rsch vict0rsch commented Sep 25, 2023

This is for running discrete env experiments.

This branch may not ever be merged, TBD.

Changes of interest:

  • (See [👶 PR] Improve Launcher to handle SLURM_TMPDIR #228) launch.py now handles code_dir: $SLURM_TMPDIR or --code_dir='$SLURM_TMPDIR'
    • If this is used, the sbatch script will:
      1. git clone the repo
      2. git checkout whatever value is in git_checkout: or --git_checkout=. If git_checkout is empty, the current branch name is used
      3. 💥 From the command-line, single quotes are mandatory around '$SLURM_TMPDIR' otherwise bash will try to evaluate it instead of giving the raw string to Python
  • (See [👶 PR] Handle logger notes in wandb #229) There is a new logger.notes key that can be used to describe a wandb experiment

Note: You can use environment variables in hydra (via OmegaConf), in particular if you want to see the $SLURM_JOB_ID somewhere you can do this:

${user.logdir.root}/workshop23/discrete-matbench/${oc.env:SLURM_JOB_ID,local}/${now:%Y-%m-%d_%H-%M-%S}

^ Here ${oc.env:SLURM_JOB_ID,local} will resolve to local if $SLURM_JOB_ID does not exist

@alexhernandezgarcia
Copy link
Owner

@vict0rsch is there anything here that is not in more recent branches and is worth keeping?

@vict0rsch
Copy link
Collaborator Author

I would keep the commits related to launch.py but I can cherry pick and make that in a new PR

@vict0rsch
Copy link
Collaborator Author

Moved to #292

@vict0rsch vict0rsch closed this Feb 22, 2024
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.

4 participants