Skip to content

Remove option to run as threaded execution#23

Merged
jomey merged 5 commits into
mainfrom
remove_threads
Oct 2, 2025
Merged

Remove option to run as threaded execution#23
jomey merged 5 commits into
mainfrom
remove_threads

Conversation

@jomey

@jomey jomey commented Oct 1, 2025

Copy link
Copy Markdown
Member

Closes #13

@jomey
jomey requested a review from a team October 1, 2025 19:13
@jomey jomey added the maintenance Technical updates to code base, dependent libraries, package install, or software architecture label Oct 1, 2025
@jomey

jomey commented Oct 1, 2025

Copy link
Copy Markdown
Member Author

Note that this should be reviewed after #22

I did not test how the new HRRR Thermal would work with this option. Since no current installation uses this (to the best of my knowledge), I opted to remove this all together.

jomey added a commit to iSnobal/model_setup that referenced this pull request Oct 1, 2025
jomey added 3 commits October 1, 2025 19:59
This was an unused and recently untested configuration. Going forward, all variables are
distributed in serial manner to ensure dependent values are available when calculating
individual forcings.
Also update the description and example in the header of the file.
More to be added.
jomey added a commit to iSnobal/awsm that referenced this pull request Oct 2, 2025
The thread option was removed from SMRF in (iSnobal/smrf#23
This is a follow up clean up to remove this option when running through AWSM. This also
removes the unused `smrf_ipysnobal` option, which partly used threaded too making for a
streamlined execution logic.
@jomey
jomey merged commit baea8a5 into main Oct 2, 2025
1 check passed
@jomey
jomey deleted the remove_threads branch October 2, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Technical updates to code base, dependent libraries, package install, or software architecture

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove "threaded" execution option

2 participants