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

Add support for purely vertical and point spaces with NetCDF #99

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Sbozzolo
Copy link
Member

@Sbozzolo Sbozzolo commented Dec 10, 2024

Leveraging the new feature introduced in ClimaCore, this commit adds support to interpolating purely vertical and purely point fields and saving them as a NetCDF file.

Leveraging the new feature introduced in ClimaCore, this commit adds
support to interpolating purely vertical fields and saving them as a
NetCDF file.
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

Attention: Patch coverage is 96.25000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 87.81%. Comparing base (070a5b1) to head (45b1580).

Files with missing lines Patch % Lines
src/netcdf_writer.jl 95.94% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #99      +/-   ##
==========================================
+ Coverage   85.42%   87.81%   +2.38%     
==========================================
  Files          13       13              
  Lines         542      599      +57     
==========================================
+ Hits          463      526      +63     
+ Misses         79       73       -6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Sbozzolo Sbozzolo changed the title Add support for purely vertical spaces with NetCDF Add support for purely vertical and point spaces with NetCDF Jan 22, 2025
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.

1 participant