fix: deprecation of xarray Dataset.dims

[nannau]

GitHub Issue: #246

Description

Update core.py to reflect xarray Dataset.dims deprecation in favour of Dataset.sizes.

Local test steps

Ran without any arguments comparing two identical file copies (taken from https://www.unidata.ucar.edu/software/netcdf/examples/files.html). The proposed changes resolves the FutureWarning.

ncompare file1.nc file2.nc

Overview of integration done

I couldn't find docs for this in contributing so I'm assuming this is referencing an internal process.

PR Acceptance Checklist

• Unit tests added/updated and passing.
• Integration testing
• CHANGELOG.md updated
• Documentation updated (if needed).

---

📚 Documentation preview 📚: https://ncompare--247.org.readthedocs.build/en/247/

[danielfromearth]

Thanks for contributing this fix, @nannau! I'm changing the base branch so we can run the integration tests, and then get it merged!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.92%. Comparing base (21f43db) to head (a1aab98).

Additional details and impacted files

@@                      Coverage Diff                      @@
##           feature/xarray_dims_to_sizes     #247   +/-   ##
=============================================================
  Coverage                         90.92%   90.92%           
=============================================================
  Files                                 5        5           
  Lines                               463      463           
=============================================================
  Hits                                421      421           
  Misses                               42       42           

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

[nannau]

Thanks for contributing this fix, @nannau! I'm changing the base branch so we can run the integration tests, and then get it merged!

Cheers, @danielfromearth!