Skip to content

Fix/chelpg tutorial default#88

Open
kayleigh222 wants to merge 2 commits intomainfrom
fix/chelpg-tutorial-default
Open

Fix/chelpg tutorial default#88
kayleigh222 wants to merge 2 commits intomainfrom
fix/chelpg-tutorial-default

Conversation

@kayleigh222
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Collaborator

@seankhl seankhl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Make sure not to commit output dirs, workspace dirs, or history.json files for local example runs
  • I think we wanna be a bit clearer in general here about some of these things. EXESS does have the walltime reporting, and the docs should talk about fetch_run_info and how it reports walltime and SUs for supercomputers only. If we talk about estimating runtime from the Rush run statuses, we should explain what those statuses are and that the actual program's runtime might be shorter, since the Rush and tengu infrastructure does take some additional time on top of that.

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