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.

Sign up for GitHub

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

Jump to bottom

Remove UM timeseries outputs from standard STASHC #21

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Remove UM timeseries outputs from standard STASHC #21

merged 1 commit into from
Dec 20, 2024

Conversation

blimlim
Copy link

@blimlim blimlim commented Dec 20, 2024

Closes #20. This pull request removes the timeseries output from the "standard" STASHC preset. I've left them in place in the "detailed" preset.

@blimlim blimlim requested a review from rml599gh December 20, 2024 05:25
rml599gh
rml599gh approved these changes Dec 20, 2024
View reviewed changes
Copy link

@rml599gh rml599gh left a comment

Choose a reason for hiding this comment

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

I'd forgotten about reducing the NUM_REQ. Thanks for remembering that.

@blimlim
Copy link
Author

blimlim commented Dec 20, 2024 •
edited
Loading

I'd forgotten about reducing the NUM_REQ. Thanks for remembering that.

I've been caught out too many times in the past!

@blimlim blimlim merged commit 7257fda into dev-preindustrial+concentrations Dec 20, 2024
10 checks passed
@blimlim blimlim deleted the 20-remove-UM-timeseries branch January 28, 2025 22:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rml599gh rml599gh rml599gh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@blimlim @rml599gh