Skip to content

Update gallery docs for data processing and instrument response examples#396

Merged
joyvelasquez merged 7 commits into
HinodeXRT:mainfrom
joyvelasquez:gallery-docs-updates
Jul 2, 2026
Merged

Update gallery docs for data processing and instrument response examples#396
joyvelasquez merged 7 commits into
HinodeXRT:mainfrom
joyvelasquez:gallery-docs-updates

Conversation

@joyvelasquez

@joyvelasquez joyvelasquez commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Extracts and updates four gallery example scripts that were bundled in PR #367 (XRT DEM Iterative Solver), per review feedback from @namurphy.

Files updated:

  • docs/gallery/data_processing/deconvolving.py
  • docs/gallery/data_processing/remove_lightleak.py
  • docs/gallery/data_processing/temperature_from_filter_ratios.py
  • docs/gallery/instrument_response/effective_area.py

No changes to source code or other documentation.

@joyvelasquez joyvelasquez self-assigned this Jun 26, 2026
@joyvelasquez

Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@joyvelasquez

Copy link
Copy Markdown
Contributor Author

Hi @namurphy - pre-commit is now passing. The two failing checks are CI timeouts unrelated to these changes:

  • Windows Python 3.11 — the full test suite (1815 tests) hits the 30-minute CI limit before finishing
  • macOS Python 3.12 — gets cancelled when Windows times out; also noticed the coverage command runs --cov=plasmapy which may be a config issue

Can you help and take a look?

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.34%. Comparing base (440f3e9) to head (59b3b8f).

Additional details and impacted files
@@            Coverage Diff            @@
##           main     #396       +/-   ##
=========================================
+ Coverage      0   87.34%   +87.34%     
=========================================
  Files         0       26       +26     
  Lines         0     1818     +1818     
  Branches      0      106      +106     
=========================================
+ Hits          0     1588     +1588     
- Misses        0      184      +184     
- Partials      0       46       +46     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

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

@joyvelasquez

Copy link
Copy Markdown
Contributor Author

pre-commit.ci autofix

@joyvelasquez joyvelasquez merged commit 315139e into HinodeXRT:main Jul 2, 2026
9 checks passed
@joyvelasquez joyvelasquez deleted the gallery-docs-updates branch July 2, 2026 15:52
joyvelasquez added a commit to joyvelasquez/xrtpy that referenced this pull request Jul 2, 2026
…les (HinodeXRT#396)

* Update gallery docs for data processing and instrument response examples

* Add missing __init__.py files for gallery subpackages

* Force the Agg backend and fix the coverage target

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Check the Fido.search calls in temperature_from_filter_ratios.py

* Check the Fido.search calls in deconvolving.py

---------

Co-authored-by: joyvelasquez <joy.velasquez@cfa.harvard>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

1 participant