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

IO: multiindeces and general fixes #202

Merged
merged 5 commits into from
Nov 10, 2023
Merged

IO: multiindeces and general fixes #202

merged 5 commits into from
Nov 10, 2023

Conversation

FabianHofmann
Copy link
Collaborator

  • The IO with NetCDF files was made more secure and fixed for some cases. In particular, variables and constraints with a dash in the name are now supported (as used by PyPSA). The object sense and value are now properly stored and retrieved from the netcdf file.
  • The IO with NetCDF file now supports multiindexed coordinates.
  • The representation of single indexed expressions and constraints with non-empty dimensions/coordinates was fixed, e.g. x.loc[["a"]] > 0 where x has only one dimension. Therefore the representation now shows the coordinates.

Copy link

codecov bot commented Nov 10, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (b437f93) 89.17% compared to head (2ac42c5) 89.23%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #202      +/-   ##
==========================================
+ Coverage   89.17%   89.23%   +0.05%     
==========================================
  Files          15       15              
  Lines        3298     3343      +45     
  Branches      757      768      +11     
==========================================
+ Hits         2941     2983      +42     
  Misses        248      248              
- Partials      109      112       +3     
Files Coverage Δ
linopy/common.py 92.50% <ø> (+0.83%) ⬆️
linopy/constraints.py 90.66% <100.00%> (-0.51%) ⬇️
linopy/expressions.py 89.44% <100.00%> (-0.31%) ⬇️
linopy/testing.py 100.00% <100.00%> (ø)
linopy/io.py 88.32% <95.12%> (+0.41%) ⬆️

... and 2 files with indirect coverage changes

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

@FabianHofmann FabianHofmann merged commit e642a1f into master Nov 10, 2023
12 checks passed
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