Skip to content

Firefly-1776: 3 color wavelength support #1799

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

robyww
Copy link
Contributor

@robyww robyww commented Jun 16, 2025

#### Firefly-1776: 3 color wavelength support

  • includes a little improvement popout readout UI

Testing

@robyww robyww added this to the 2025.4 milestone Jun 16, 2025
@robyww robyww force-pushed the FIREFLY-1776-3c-wavelengths branch 2 times, most recently from 5bc7a28 to da6744f Compare June 17, 2025 18:22
@robyww robyww self-assigned this Jun 17, 2025
@lrebull
Copy link
Contributor

lrebull commented Jun 17, 2025

it works for me!
i think it should be "bandwidth" not "Band Width" but i don't know SPHEREx's convention.

@robyww
Copy link
Contributor Author

robyww commented Jun 17, 2025

i think it should be "bandwidth" not "Band Width" but i don't know SPHEREx's convention.

Its general. I will change it.

- a little improvement in popout readout UI and layout
- includes resposne to feedback

more
@robyww robyww force-pushed the FIREFLY-1776-3c-wavelengths branch from da6744f to 9b8aa67 Compare June 17, 2025 19:06
Copy link
Contributor

@kpuriIpac kpuriIpac left a comment

Choose a reason for hiding this comment

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

I skimmed through the code and tested according to the instructions. Looks good to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants