Skip to content

NTasks & Timestep Changes suggested by Mike#18

Merged
manishvenu merged 7 commits into
mainfrom
ntasks_timestep_calc
Oct 10, 2025
Merged

NTasks & Timestep Changes suggested by Mike#18
manishvenu merged 7 commits into
mainfrom
ntasks_timestep_calc

Conversation

@manishvenu

Copy link
Copy Markdown
Collaborator

Slightly adjusted the timestep default and ntasks max cores amt.

@alperaltuntas

Copy link
Copy Markdown
Member

Looks like test_cores_case_creator.py needs to be updated to assert new numbers.

@manishvenu

Copy link
Copy Markdown
Collaborator Author

Shoot yeah, just updated it

@mnlevy1981 mnlevy1981 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@manishvenu manishvenu merged commit 6ae49ac into main Oct 10, 2025
1 of 2 checks passed
@alperaltuntas alperaltuntas deleted the ntasks_timestep_calc branch December 1, 2025 03:03
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.

3 participants