(Fix) Burst Average code failing with ALL bad qc flag variable - fix https://github.com/aodn/content/issues/453

[lbesnard]

No description provided.

[codecov]

Codecov Report

Merging #279 (45521ae) into master (d6fe7f1) will increase coverage by 0.15%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #279      +/-   ##
==========================================
+ Coverage   88.28%   88.44%   +0.15%     
==========================================
  Files          50       50              
  Lines        3295     3296       +1     
  Branches      542      542              
==========================================
+ Hits         2909     2915       +6     
+ Misses        243      239       -4     
+ Partials      143      142       -1     

Impacted Files	Coverage Δ
aodndata/moorings/burst_average.py	89.95% <100.00%> (+2.05%)	⬆️

---

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 d6fe7f1...45521ae. Read the comment docs.

[mhidas]

Thanks @lbesnard . I haven't had a chance to test this. However, I'm not sure you're actually addressing the core problem here. The original issue said

The code ... crashes when it encounters a variable that has all its values (within a burst) flagged bad. It should just output a missing value for that burst.

From what I can see here, your change is to look for variables that have missing values (as opposed to flagged "Bad") for the entire dataset and removing them from entire variable from the output file... I don't think that's what we want.