Skip to content

create dir before writing to netcdf#51

Closed
aurelienfalco wants to merge 6 commits intodevfrom
af/dev
Closed

create dir before writing to netcdf#51
aurelienfalco wants to merge 6 commits intodevfrom
af/dev

Conversation

@aurelienfalco
Copy link
Copy Markdown
Collaborator

Minor fix to avoid having to create the directory by hand when creating a netcdf

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 13, 2026

Codecov Report

❌ Patch coverage is 34.48276% with 19 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.90%. Comparing base (9a572a9) to head (661521a).

Files with missing lines Patch % Lines
xdas/core/routines.py 29.16% 17 Missing ⚠️
xdas/core/dataarray.py 66.66% 1 Missing ⚠️
xdas/core/datacollection.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev      #51      +/-   ##
==========================================
- Coverage   83.21%   82.90%   -0.31%     
==========================================
  Files          43       43              
  Lines        4569     4598      +29     
==========================================
+ Hits         3802     3812      +10     
- Misses        767      786      +19     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@atrabattoni atrabattoni changed the base branch from main to dev March 5, 2026 14:29
@atrabattoni
Copy link
Copy Markdown
Contributor

this pull request is linked to the wrong branch. I close it to reopen it.

@atrabattoni
Copy link
Copy Markdown
Contributor

Re-open as #56.

@atrabattoni
Copy link
Copy Markdown
Contributor

Re-open as #56.

@atrabattoni atrabattoni closed this Mar 5, 2026
@atrabattoni atrabattoni deleted the af/dev branch March 5, 2026 15:26
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.

2 participants