Closed erogluorhan closed 6 months ago
Some of the changes here overlap with what I started looking at in #708. However, feel free to close that one since this one seems more comprehensive than what I started. I'll give this a look tommorow!
@philipc2 thanks for letting me know, and sorry I missed that one, but you are right the changes here are comprehensively addressing multiple warnings as described in the first cell. Yeah, please give it a review when you have time.
Closes #710
Overview
Addresses:
Dataset.dims
will be changed to return a set of dimension names in future, in order to be more consistent withDataArray.dims
. To access a mapping from dimension names to lengths, please useDataset.sizes
"pyarrow
to fix "FutureWarning: Minimal version of pyarrow will soon be increased to 14.0.1. You are using 12.0.1. Please consider upgrading."assertEquals()
calls in the test files by converting them toassertEqual()
assertAlmostEquals()
calls in the test files by converting them toassertAlmostEqual()
PR Checklist
General