Skip to content

Conversation

@robyww
Copy link
Contributor

@robyww robyww commented Jul 1, 2021

Firefly-810: Fixed mask not loading correctly when zooming

  • Change mask to use local data
  • Mask should be faster with smaller memory load, mask data compress to bits
  • Added transparency to the color palette of the color change dialog
  • Fixed: Firefly-819 (simple one line fix)
  • Fixed: Bias sliders not working in 3-color mode

Testing URL

https://fireflydev.ipac.caltech.edu/firefly-810-mask/firefly/

Testing Steps

  1. Load a fits file with mask
  2. Enable the mask and zoom up and down

Ticket(s)

https://jira.ipac.caltech.edu/browse/FIREFLY-810
https://jira.ipac.caltech.edu/browse/FIREFLY-819

@robyww robyww added the bug label Jul 1, 2021
@robyww robyww added this to the 2021.2 milestone Jul 1, 2021
@robyww robyww requested a review from gpdf July 1, 2021 18:04
@robyww robyww self-assigned this Jul 1, 2021
 - Change mask to use local data
 - Mask should be faster with smaller memory load, mask data compress to bits
 - Added transparency to the color palette of the color change dialog
 - Fixed: Firefly-819 (simple one line fix)
 - Fixed: Bias sliders not working in 3-color mode
@robyww robyww force-pushed the firefly-810-mask branch from 2118e9a to cab8def Compare July 1, 2021 18:26
@robyww robyww marked this pull request as ready for review July 1, 2021 18:35
@robyww robyww added the multi-ticket This PR implements multiple Jira tickets label Jul 1, 2021
Copy link
Contributor

@gpdf gpdf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mask zooming fix works and performance is now really impressive! Thank you.

FIREFLY-819 HiPS-properties links are now valid, accessible URLs, but they appear to be the wrong ones for the corresponding rows in the HiPS table.

@robyww
Copy link
Contributor Author

robyww commented Jul 1, 2021

Mask zooming fix works and performance is now really impressive! Thank you.

FIREFLY-819 HiPS-properties links are now valid, accessible URLs, but they appear to be the wrong ones for the corresponding rows in the HiPS table.

This is a second problem with the HiPS table. We can't fix it in this release. I made a ticket: https://jira.ipac.caltech.edu/browse/FIREFLY-822

@robyww robyww merged commit d85482c into rc-2021.2 Jul 2, 2021
@robyww robyww deleted the firefly-810-mask branch July 2, 2021 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug multi-ticket This PR implements multiple Jira tickets

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants