search

aodn / python-aodntools

Repository for templates and code relating to generating standard NetCDF files for the Australia Ocean Data Network
GNU Lesser General Public License v3.0
10 stars 3 forks source link

Fix typo in names of pressure variables in aggregated_timeseries #139

Closed mhidas closed 3 years ago

mhidas commented 3 years ago

Fixes #138

codecov[bot] commented 3 years ago

Codecov Report

Merging #139 (d1126e6) into master (2d7b4aa) will not change coverage. The diff coverage is 80.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #139   +/-   ##
=======================================
  Coverage   70.09%   70.09%           
=======================================
  Files          12       12           
  Lines        1117     1117           
  Branches      146      146           
=======================================
  Hits          783      783           
  Misses        317      317           
  Partials       17       17
Impacted Files Coverage Δ
...tools/timeseries_products/aggregated_timeseries.py 95.29% <80.00%> (ø)

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 2d7b4aa...d1126e6. Read the comment docs.

mhidas commented 3 years ago

@ocehugo This is a quick & easy one. It fixes the issue discussed in https://github.com/aodn/anmn-internal-discussions/issues/72

mhidas commented 3 years ago

Also added a simple change that partially addresses #118 (only for aggregated_timeseries)