Fixed the lack of mask for LEL images

CARTAvis / carta-backend

Source code repository for the backend component of CARTA, a new visualization tool designed for the ALMA, the VLA and the SKA pathfinders.

GNU General Public License v3.0

22 stars 11 forks source link

Description

What is implemented or fixed? Mention the linked issue(s), if any. Fixed #1291.
How does this PR solve the issue? Give a brief summary. Get mask data from the LEL image, and then reset the pixel value as NaN if its mask is false.
Are there any companion PRs (frontend, protobuf)? No.
Is there anything else that testers should know (e.g. exactly how to reproduce the issue)? The rendered image will show a mask if we apply a boolean check in LEL syntax, like "dice_one.fits"["dice_one.fits" >2].

Checklist

[x] changelog updated ~/ no changelog update needed~
[x] e2e test passing / ~added corresponding fix~
[x] ~protobuf updated to the latest dev commit /~ no protobuf update needed
[ ] added reviewers and assignee
[ ] added ZenHub estimate, milestone, and release

Package	Line Rate	Health
src.Cache	66%	➖
src.DataStream	52%	➖
src.FileList	67%	➖
src.Frame	50%	➖
src.HttpServer	43%	➖
src.ImageData	28%	❌
src.ImageFitter	89%	✔
src.ImageGenerators	44%	➖
src.ImageStats	76%	✔
src.Logger	44%	➖
src.Main	54%	➖
src.Region	18%	❌
src.Session	29%	❌
src.Table	52%	➖
src.ThreadingManager	87%	✔
src.Timer	85%	✔
src.Util	48%	➖
Summary	38% (7002 / 18533)	❌

Package

Line Rate

Health

src.Cache

66%

➖

src.DataStream

52%

➖

src.FileList

67%

➖

src.Frame

50%

➖

src.HttpServer

43%

➖

src.ImageData

28%

❌

src.ImageFitter

89%

✔

src.ImageGenerators

44%

➖

src.ImageStats

76%

✔

src.Logger

44%

➖

src.Main

54%

➖

src.Region

18%

❌

src.Session

29%

❌

src.Table

52%

➖

src.ThreadingManager

87%

✔

src.Timer

85%

✔

src.Util

48%

➖

Summary

38% (7002 / 18533)

❌

CARTAvis / carta-backend

Fixed the lack of mask for LEL images #1292