LM-SAL / aiapy

Python library for AIA data analysis
https://aiapy.readthedocs.io/en/stable/
BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

Merge branch 'handle-masked-columns' into 'master' - [merged] #277

Closed nabobalis closed 8 months ago

nabobalis commented 2 years ago

Merges cherry-pick-cbd76886 -> 0.6

Remove masked columns from pointing table

Closes #124

See merge request LMSAL_HUB/aia_hub/aiapy!151

(cherry picked from commit cbd768861938639d7d7a1fe17551c56049c7779e)

03455a65 dont use masked cols in pointing update 3aa6f26c add changelog 15ff4cd6 fix mask checking 10abe315 fix test

nabobalis commented 2 years ago

approved this merge request

nabobalis commented 2 years ago

enabled an automatic merge when the pipeline for 27d9f910ca1cb2fad4976035b0b405c24ed7a3d9 succeeds

nabobalis commented 2 years ago

In GitLab by @codecov on Jan 14, 2022, 09:53

Codecov Report

Merging #153 (27d9f91) into 0.6 (8c2613a) will increase coverage by 0.02%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              0.6     #153      +/-   ##
==========================================
+ Coverage   95.87%   95.90%   +0.02%     
==========================================
  Files          19       19              
  Lines         606      610       +4     
==========================================
+ Hits          581      585       +4     
  Misses         25       25              
Impacted Files Coverage Δ
aiapy/calibrate/util.py 100.00% <100.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 8c2613a...27d9f91. Read the comment docs.

nabobalis commented 2 years ago

mentioned in commit 027dd7abfa1a1569b92364613bb9768c02801b88