Skip to content

Reproduction setup: SLURM scripts, configs#8

Open
dimanaStoyanova7 wants to merge 12 commits into
jhbastek:mainfrom
squarerootminusone:dimana/reproduction
Open

Reproduction setup: SLURM scripts, configs#8
dimanaStoyanova7 wants to merge 12 commits into
jhbastek:mainfrom
squarerootminusone:dimana/reproduction

Conversation

@dimanaStoyanova7

Copy link
Copy Markdown
  • configs/ - 10 yaml config files for all model variants (Diffusion, PG-Diffusion, CoCoGen, PIDM-ME, PIDM-SE) for both Darcy flow and topology optimization
  • slurm/ - SLURM job scripts for all training and eval jobs, including:
    • Fixed GPU detection (replaced exec/compile with sed + direct python call)
    • Added PYTHONPATH to resolve src/ module imports from temp scripts
    • Added CUDA availability check at job start
    • Configured for gpu-a100 and test-gpu partitions on DelftBlue

@jhbastek

Copy link
Copy Markdown
Owner

Happy to bring this in if these are addressed:

  • Please exclude the SLURM scripts from the PR, these are too user-specific IMO
  • Can we move the plot scripts into the images/ folder?
  • Remove .DS_Store

Thanks a lot for contributing!

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