Skip to content

quick_image improvements#83

Open
bhazelton wants to merge 7 commits intomainfrom
quick_image_fix
Open

quick_image improvements#83
bhazelton wants to merge 7 commits intomainfrom
quick_image_fix

Conversation

@bhazelton
Copy link
Copy Markdown
Member

Fixed a couple small bugs in quick_image and added an option to set the colormap or match the colors used in the IDL version of quick_image.

The color matching is especially useful when trying to compare arrays at various points in FHD and pyFHD.

Please also remove any changelog sections that don't match/relate to your pull request to ensure the changelog in the pull request is succinct, except for additional Translation, any additional translation will likely include all changes and checklists.

Types of Changes

  • Bug Fixes
  • New Feature(s)
  • Breaking Changes
  • Documentation
  • Version Change
  • Translation from FHD
  • Build or CI Change

Checklist

General PR Checklist

  • I have the read the contribution guide
  • Add all the above to the Changelog into the unreleased section

New Features Checklist

  • New Features have all new functions documented using the numpydoc docstring format
  • Updated an existing tutorial or created a tutorial to use the new feature in the documentation
  • Have added new tests or changed existing tests to cover the new feature

Bug Fixes Checklist

  • Added all bug fixes with any issues associated with them linked.
  • Added new tests to cover bugs
  • Adjusted existing tests to cover bug, in which case check the Existing Tests Checklist

@bhazelton bhazelton requested a review from nicholebarry April 1, 2026 23:24
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

❌ Patch coverage is 87.50000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 35.30%. Comparing base (4c11ade) to head (d5eac23).

Files with missing lines Patch % Lines
PyFHD/plotting/image.py 87.50% 0 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #83      +/-   ##
==========================================
+ Coverage   32.74%   35.30%   +2.55%     
==========================================
  Files          27       27              
  Lines        5540     5549       +9     
  Branches     1012     1015       +3     
==========================================
+ Hits         1814     1959     +145     
+ Misses       3525     3346     -179     
- Partials      201      244      +43     

☔ View full report in Codecov by Sentry.
📢 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.

@bhazelton bhazelton added bug Something isn't working enhancement New feature or request labels Apr 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant