Skip to content

Conversation

@austinorr
Copy link
Owner

All baseline images that are changed have been verified to be identical to previous output save for freetype issues.

For many tests, freetype issues have been eliminated by using 'remove_text' and by creating a backward compatible kwarg in the helper functions to refrain from labeling the titleblock frames. The frame positions are what we are testing, not the text, so this is an improvement.

image tolerance has been constrained to '2' for all tests that don't contain text, or which test image exif parsing.

Additionally, this PR shrinks the figure size of the test images so that differences in the output are a significantly higher fraction of the overall image.

@codecov
Copy link

codecov bot commented Feb 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.42%. Comparing base (2936cd7) to head (0a5b814).
Report is 6 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
+ Coverage   95.70%   96.42%   +0.72%     
==========================================
  Files           3        3              
  Lines         396      392       -4     
  Branches       43       41       -2     
==========================================
- Hits          379      378       -1     
  Misses          7        7              
+ Partials       10        7       -3     

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

@austinorr austinorr merged commit 288e209 into main Feb 28, 2025
7 checks passed
@austinorr austinorr deleted the dev branch February 28, 2025 18:02
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.

2 participants