Closed mhidas closed 5 years ago
Merging #38 into master will increase coverage by
3.8%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #38 +/- ##
=======================================
+ Coverage 96.19% 100% +3.8%
=======================================
Files 2 2
Lines 184 169 -15
Branches 38 29 -9
=======================================
- Hits 177 169 -8
+ Misses 5 0 -5
+ Partials 2 0 -2
Impacted Files | Coverage Δ | |
---|---|---|
ncwriter/template.py | 100% <100%> (+4.51%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 1e3f55c...737ed67. Read the comment docs.
The code was already handling these special "structural" attributes, which get passed on to the
netCDF4.Dataset.createVariable
method, but they were not allowed in the template schema. This PR adds those attributes in the schema, and also simplifies the code a bit.