Skip to content

Plotting Refactor - #242

Open
robelgeda wants to merge 8 commits into
PetroFit:mainfrom
robelgeda:plotting_refactor
Open

Plotting Refactor#242
robelgeda wants to merge 8 commits into
PetroFit:mainfrom
robelgeda:plotting_refactor

Conversation

@robelgeda

Copy link
Copy Markdown
Contributor

This PR refactors plotting functions into a single sub-module plotting. This PR also updates the plotting style.

@codecov

codecov Bot commented Sep 5, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.37838% with 154 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
petrofit/plotting.py 56.11% 85 Missing and 19 partials ⚠️
petrofit/segmentation.py 17.07% 31 Missing and 3 partials ⚠️
petrofit/modeling/fitting.py 84.21% 2 Missing and 4 partials ⚠️
petrofit/photometry.py 70.00% 1 Missing and 5 partials ⚠️
petrofit/petrosian/correction.py 20.00% 4 Missing ⚠️
Files with missing lines Coverage Δ
petrofit/__init__.py 83.33% <100.00%> (+3.33%) ⬆️
petrofit/config/__init__.py 100.00% <100.00%> (ø)
petrofit/config/mpl.py 100.00% <100.00%> (ø)
petrofit/petrosian/core.py 82.81% <100.00%> (+0.24%) ⬆️
petrofit/utils.py 79.26% <ø> (+5.53%) ⬆️
petrofit/petrosian/correction.py 70.40% <20.00%> (+9.97%) ⬆️
petrofit/modeling/fitting.py 82.65% <84.21%> (+21.91%) ⬆️
petrofit/photometry.py 84.00% <70.00%> (-9.19%) ⬇️
petrofit/segmentation.py 55.75% <17.07%> (-1.15%) ⬇️
petrofit/plotting.py 56.11% <56.11%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant