Skip to content

cosmic analysis script rewritten in numpy#34

Open
Jennefer Maldonado (jennmald) wants to merge 5 commits into
NSLS2:mainfrom
jennmald:analysis-tools
Open

cosmic analysis script rewritten in numpy#34
Jennefer Maldonado (jennmald) wants to merge 5 commits into
NSLS2:mainfrom
jennmald:analysis-tools

Conversation

@jennmald
Copy link
Copy Markdown
Contributor

based on Daniel Henriksen (@dihenriksen)'s script

  • still need to test fully

@jennmald
Copy link
Copy Markdown
Contributor Author

oh and finish documentation!

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new “cosmics” analysis helper script intended to visualize and quantify cosmic-trigger-like pixels in counting detector data stored in HDF5.

Changes:

  • Introduces cosmics.py with helpers to load HDF5 data, visualize frames, label connected components, and plot per-image count distributions.
  • Adds functions to detect candidate frames (find_cosmics) and inspect data quality (check_empty, show_image, movie).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
Comment thread src/cditools/analysis-scripts/cosmics.py Outdated
@jennmald Jennefer Maldonado (jennmald) marked this pull request as ready for review May 1, 2026 16:46
Copy link
Copy Markdown
Contributor

@dihenriksen Daniel Henriksen (dihenriksen) left a comment

Choose a reason for hiding this comment

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

LGTM

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants