Excellent! Would be great to get these fixes in - let me know once you need a review.

@TomNicholas Apologies that it took me too long to come back to this, but things have been extremely busy...

I added a minor test for the minor improvements I am adding. These are:

• Creation of a test file with an empty scalar variable, with no chunks, no dimensions, but a fill_value. This is the specific scenario of dmrpp parsing fails for variables without a dim #666.
• Added the dmrpp for this file, and assert that virtualizarr can create the virtual dataset of this.
• In the dmrpp for this file, I added a fake container attribute. Now, the parser warns about this container attribute and drops it, and allow the process of virtualization to continue. This is exactly one of the issues of Support flattened dmrpp files. #581.

@betolink
Regarding icesat-2 data: The container attributes are a consequence of flattening the access to these hierarchical datasets, and parsing these container attributes correctly will take some time. I think droping these attributes is the easiest thing to do for now, and allow for data exploration while the decision to produce unflatten the dmpps is explored by nsidc.

Lastly, there are some other issues with some variables, mostly metadata variables, and these can be dropped during the parsing. So for now, I think this may at least enable data exploration of IceSat2 data and I can look what is going on with those metadata variables on a separate issue

Please add a note to the release notes!

Codecov Report

❌ Patch coverage is 75.00000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.74%. Comparing base (7638893) to head (d3beede).

@@            Coverage Diff             @@
##             main     #757      +/-   ##
==========================================
- Coverage   87.75%   87.74%   -0.02%     
==========================================
  Files          35       35              
  Lines        1879     1885       +6     
==========================================
+ Hits         1649     1654       +5     
- Misses        230      231       +1     

Ready to merge?

Yes, ready to merge :). Thanks @TomNicholas !