Remove masked columns from pointing table - [merged]

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 07:15

Merges handle-masked-columns -> master

Fixes #124

This ensures that the columns that are used in the pointing update calculation do not have a mask associated with them.

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 07:19

added 1 commit

• 9afa69e7 - add changelog

Compare with previous version

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 07:22

marked this merge request as draft

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 07:36

added 1 commit

• cb0b958e - fix mask checking

Compare with previous version

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 07:51

marked this merge request as ready

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 08:16

added 5 commits

• cb0b958e...079b71fe - 2 commits from branch LMSAL_HUB/aia_hub:master
• 03455a65 - dont use masked cols in pointing update
• 3aa6f26c - add changelog
• 15ff4cd6 - fix mask checking

Compare with previous version

[nabobalis]

In GitLab by @wtbarnes on Jan 14, 2022, 08:52

added 1 commit

• 10abe315 - fix test

Compare with previous version

[nabobalis]

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

Codecov Report

Merging #151 (10abe31) into master (079b71f) will increase coverage by 0.02%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
+ 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 079b71f...10abe31. Read the comment docs.

[nabobalis]

approved this merge request

[nabobalis]

enabled an automatic merge when the pipeline for 10abe315133a20e67de2527aeab2d97ccdee7ce4 succeeds

[nabobalis]

mentioned in commit cbd768861938639d7d7a1fe17551c56049c7779e

[nabobalis]

picked the changes into the branch cherry-pick-cbd76886 with commit 27d9f910ca1cb2fad4976035b0b405c24ed7a3d9

[nabobalis]

mentioned in commit 27d9f910ca1cb2fad4976035b0b405c24ed7a3d9