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.

Sign up for GitHub

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

Jump to bottom

COO-24 Add ability to view histograms #5

Merged
merged 2 commits into from
Sep 3, 2024
Merged

COO-24 Add ability to view histograms #5

merged 2 commits into from
Sep 3, 2024

Conversation

prkrtg
Copy link
Contributor

@prkrtg prkrtg commented Aug 30, 2024

  • New class for histograms which adds capability to view histograms for given images
  • Histogram class is integrated to FITSViewer where the viewer passes in images to be displayed in the histogram
  • Clean up

Example below of the UI:
Screenshot from 2024-08-29 15-22-55
Screenshot from 2024-08-30 14-47-52

mikelangmayr
mikelangmayr approved these changes Aug 30, 2024
View reviewed changes
Copy link

@mikelangmayr mikelangmayr left a comment

Choose a reason for hiding this comment

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

Looks good!

@prkrtg prkrtg merged commit 31f84c1 into main Sep 3, 2024
0 of 3 checks passed
@prkrtg prkrtg deleted the pgupta/histogram branch September 18, 2024 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mikelangmayr mikelangmayr mikelangmayr approved these changes

@scizen9 scizen9 Awaiting requested review from scizen9

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@prkrtg @mikelangmayr