Skip to content
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

Additional REGION_HISTOGRAM_DATA ICD message return when I request SetHistogramRequirements ICD message #1266

Closed
acdo2002 opened this issue May 16, 2023 · 0 comments · Fixed by #1268
Assignees
Labels
bug Something isn't working
Milestone

Comments

@acdo2002
Copy link
Contributor

Describe the bug
After adding HistogramConfig in the SetHistogramRequirements and REGION_HISTOGRAM_DATA.
When open an image and request SetHistogramRequirements, there are TWO REGION_HISTOGRAM_DATAs returned. The first REGION_HISTOGRAM_DATA is identical to the REGION_HISTOGRAM_DATA that returned with OpenFileAck.
(If set a region and then request SetHistogramRequirements, there is only ONE REGION_HISTOGRAM_DATA returned, it is what I expected)

To Reproduce
Steps to reproduce the behavior:
0. Open console.log, set "Preference - Log Events" to select All

  1. Open "HH211_IQU.fits" (backend <== OPEN_FILE, ==> OPEN_FILE_ACK, ==> REGION_HISTOGRAM_DATA)
  2. Click on "Histogram widget"
  3. Set Polarization to "Stokes Q"
  4. Click top right "setting", and adjust X min to"0"
  5. You can see there is ONE request "SET_HISTOGRAM_REQUIREMENTS" but the backend return TWO "REGION_HISTOGRAM_DATA". The first one is identical to Step 1 "REGION_HISTOGRAM_DAT"

Expected behavior
Based on v3, if change the setting in the Histogram widget, there is only ONE "REGION_HISTOGRAM_DAT" returned from the backend

Screenshots or videos
Screen Shot 2023-05-16 at 11 58 54 AM
Screen Shot 2023-05-16 at 11 59 09 AM

Platform info (please complete the following information):

  • OS [e.g. macOS Monterey]: macOS Big Sur 11.6
  • Browser [e.g. chrome, safari, electron app]: chrome
  • Browser version [e.g. 22]: Version 111.0.5563.110 (Official Build) (arm64)
  • Backend branch [e.g. dev, v3b2 release]: dev
  • Frontend branch [e.g. dev, v3b2 release]: dev

Additional context
Add any other context about the problem here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants