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

Update dependencies #2141

Merged
merged 3 commits into from
Sep 22, 2023
Merged

Update dependencies #2141

merged 3 commits into from
Sep 22, 2023

Conversation

charleskawczynski
Copy link
Member

This PR upgrades the dependencies, including ClimaTimeSteppers. The new version fuses some operations, resulting in precision error differences.

@szy21
Copy link
Member

szy21 commented Sep 22, 2023

Ah, I just did #2140, but I didn't update ClimaTimesteppers I think.

@szy21 szy21 mentioned this pull request Sep 22, 2023
1 task
@charleskawczynski
Copy link
Member Author

Ah, I just did #2140, but I didn't update ClimaTimesteppers I think.

No problem, we need to bump the ref counter for this one anyway

@charleskawczynski
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Sep 22, 2023
2137: Fix closures resulting in boxed variables r=charleskawczynski a=charleskawczynski

This PR fixes a closure in the diagnostics, which results in boxing variables. Issue found in https://buildkite.com/clima/climaatmos-ci/builds/13072#018ab62d-8432-4918-8532-8c2bed825ff0

2141: Update dependencies r=charleskawczynski a=charleskawczynski

This PR upgrades the dependencies, including ClimaTimeSteppers. The new version fuses some operations, resulting in precision error differences.

Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 22, 2023

Build failed (retrying...):

bors bot added a commit that referenced this pull request Sep 22, 2023
2121: change edmfx advection r=trontrytel a=trontrytel

Closes #2106 

I wasn't sure if we also want to change the tke advection?

2141: Update dependencies r=charleskawczynski a=charleskawczynski

This PR upgrades the dependencies, including ClimaTimeSteppers. The new version fuses some operations, resulting in precision error differences.

Co-authored-by: Anna Jaruga <ajaruga@caltech.edu>
Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 22, 2023

Build failed (retrying...):

bors bot added a commit that referenced this pull request Sep 22, 2023
2141: Update dependencies r=charleskawczynski a=charleskawczynski

This PR upgrades the dependencies, including ClimaTimeSteppers. The new version fuses some operations, resulting in precision error differences.

Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 22, 2023

Build failed:

@charleskawczynski
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 22, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit a2f8dc1 into main Sep 22, 2023
8 checks passed
@bors
Copy link
Contributor

bors bot commented Sep 22, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot deleted the ck/update_deps branch September 22, 2023 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants