The GridFile class gave too much flexibility that led to confusion in certain scripts based on the mode that was used when making a file. This PR simplifies things:
When a GridFile is instantiated a new file is made (in w mode).
Any subsequent interaction with the same GridFile will be done in append mode (a/r+).
Additionally, there's no need to pass a units string explicitly when making a dataset since the units can be extracted from the incoming data.
Issue Type
Enhancement
Checklist
[x] I have read the [CONTRIBUTING.md]()
[x] I have added docstrings to all methods
[x] I have added sufficient comments to all lines
[x] I have made corresponding changes to the documentation
[x] My changes generate no pep8 errors
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
WillJRoper
commented
1 week ago
The
GridFileclass gave too much flexibility that led to confusion in certain scripts based on the mode that was used when making a file. This PR simplifies things:GridFileis instantiated a new file is made (inwmode).GridFilewill be done in append mode (a/r+).Additionally, there's no need to pass a units string explicitly when making a dataset since the units can be extracted from the incoming data.
Issue Type
Checklist