Skip to content

Add Phase Resolved Analysis tools (#395)#423

Closed
KoothodilAbhijithAugustine wants to merge 6 commits intodevelopfrom
Abhi-phase-resolved-spectroscopy
Closed

Add Phase Resolved Analysis tools (#395)#423
KoothodilAbhijithAugustine wants to merge 6 commits intodevelopfrom
Abhi-phase-resolved-spectroscopy

Conversation

@KoothodilAbhijithAugustine
Copy link
Copy Markdown
Member

@KoothodilAbhijithAugustine KoothodilAbhijithAugustine commented Oct 22, 2025

Fixes
Implements the tool to obtain a phasogram as described in issue
Adds PulsarAnalyzer class, phase_resolved_analysis module, and tutorial notebook.

…nown

Implement a PulsarAnalyzer class for analyzing pulsar data using Z²₂ test and visualization utilities. It will plot a three panel figure with a pulse profile, Z²₂ figure and phaseogram.
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 22, 2025

Codecov Report

❌ Patch coverage is 0% with 147 lines in your changes missing coverage. Please review.
✅ Project coverage is 77.85%. Comparing base (dc46b9b) to head (e9f313b).
⚠️ Report is 91 commits behind head on develop.

Files with missing lines Patch % Lines
cosipy/phase_resolved_analysis/pulse_profile.py 0.00% 145 Missing ⚠️
cosipy/phase_resolved_analysis/__init__.py 0.00% 2 Missing ⚠️
Files with missing lines Coverage Δ
cosipy/phase_resolved_analysis/__init__.py 0.00% <0.00%> (ø)
cosipy/phase_resolved_analysis/pulse_profile.py 0.00% <0.00%> (ø)

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@israelmcmc
Copy link
Copy Markdown
Collaborator

Thanks @KoothodilAbhijithAugustine. While this is reviewed, can you please add unit tests that cover the new code?

There's an explanation here: https://cositools.github.io/cosipy/dev/unit_tests.html
And multiple example here: https://github.com/cositools/cosipy/tree/develop/tests

@israelmcmc israelmcmc added phase_analysis Modulations, pulses and phase resolved analyses pull-request-waiting-for-reviewer The ball is on the reviewer side. labels Oct 27, 2025
@KoothodilAbhijithAugustine KoothodilAbhijithAugustine removed phase_analysis Modulations, pulses and phase resolved analyses pull-request-waiting-for-reviewer The ball is on the reviewer side. labels Feb 27, 2026
@israelmcmc israelmcmc deleted the Abhi-phase-resolved-spectroscopy branch April 1, 2026 20:51
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