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

Use SurfaceFluxes.jl v0.8.0 #2261

Merged
merged 1 commit into from
Oct 20, 2023
Merged

Use SurfaceFluxes.jl v0.8.0 #2261

merged 1 commit into from
Oct 20, 2023

Conversation

akshaysridhar
Copy link
Member

@akshaysridhar akshaysridhar commented Oct 19, 2023

Update sf functions to use v0.8.0 interface
modified: Project.toml
modified: src/surface_conditions/surface_conditions.jl

@akshaysridhar akshaysridhar self-assigned this Oct 19, 2023
@akshaysridhar akshaysridhar force-pushed the as/up-sf-version branch 2 times, most recently from 286f18c to 8e73d9a Compare October 19, 2023 21:10
@akshaysridhar akshaysridhar marked this pull request as ready for review October 19, 2023 22:12
Copy link
Member

@szy21 szy21 left a comment

Choose a reason for hiding this comment

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

You can also remove all the FLOAT_TYPE: "Float32", as that's the default.

Copy link
Member

@trontrytel trontrytel left a comment

Choose a reason for hiding this comment

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

Is that a good moment to bump the patch version in ClimaAtmos?

@akshaysridhar
Copy link
Member Author

Is that a good moment to bump the patch version in ClimaAtmos?

Seems so - this is done in the latest commit.

@akshaysridhar
Copy link
Member Author

akshaysridhar commented Oct 20, 2023

@sriharshakandala if you could verify the Project / Manifest toml updates w.r.t to the new SurfaceFluxes usage I can update the MSE tables via ref counter and merge this PR.

@szy21
Copy link
Member

szy21 commented Oct 20, 2023

The AMIP simulation in the longrun seems to break immediately. Could you reduce dt by a factor of 2? No need to wait to see whether it works, we can just wait for today's longrun.

@akshaysridhar akshaysridhar force-pushed the as/up-sf-version branch 2 times, most recently from ac98ab2 to 9b3427e Compare October 20, 2023 20:15
Update sf functions to use v0.8.0 interface
	modified:   Project.toml
	modified:   docs/Manifest.toml
	modified:   examples/Manifest.toml
	modified:   perf/Manifest.toml
	modified:   src/surface_conditions/surface_conditions.jl
Update longrun configs to use Float32
Update patch version
	modified:   Project.toml
Bump ref counter 134 > 135
	modified:   regression_tests/ref_counter.jl
Reduce longrun aquaplanet amip dt
	modified:   config/longrun_configs/longrun_aquaplanet_amip.yml
Update coupler compatibility test
	modified:   test/coupler_compatibility.jl
@akshaysridhar
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Oct 20, 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 6994c4b into main Oct 20, 2023
8 checks passed
@bors bors bot deleted the as/up-sf-version branch October 20, 2023 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants