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 pitzer exp #46

Merged
merged 4 commits into from
Aug 27, 2020
Merged

Add pitzer exp #46

merged 4 commits into from
Aug 27, 2020

Conversation

johrstrom
Copy link
Contributor

add pitzer expansion

  • change capitalize function to capitalize_words to change 'pitzer-exp'
    to PitzerExp (to access the data attributes)
  • add option-for attributes to the node types because hugemem is not
    available on pitzer-exp and largemem is only available on pitzer-exp
  • lastly obviously, submit.yml.erb has to change to accomidate slurm
    options and torque options.

Again, borrowed heavily work done in jupyter: OSC/bc_osc_jupyter#48

- change capitalize function to capitalize_words to change 'pitzer-exp'
  to PitzerExp (to access the data attributes)
- add option-for attributes to the node types because hugemem is not
  available on pitzer-exp and largemem is _only_ available on pitzer-exp
- lastly obviously, submit.yml.erb has to change to accomidate slurm
  options and torque options.
@treydock
Copy link
Contributor

@johrstrom Pitzer expansion is 48 cores per node. Also at this time largemem nodes which are also 48 core but 768GB of RAM are not available for users to run as we evaluate why their benchmarks like HPL and HPCG are not consistent.

form.yml Outdated Show resolved Hide resolved
form.yml Outdated Show resolved Hide resolved
form.yml Outdated Show resolved Hide resolved
@johrstrom
Copy link
Contributor Author

Thanks! Good catches. I'll remove largemem from jupyter too.

@johrstrom johrstrom merged commit 23f9a48 into master Aug 27, 2020
@johrstrom johrstrom deleted the add-pitzer-exp branch August 27, 2020 14:06
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