simplify screenshot testing by kushalkolar · Pull Request #447 · fastplotlib/fastplotlib · GitHub
Skip to content

simplify screenshot testing#447

Merged
kushalkolar merged 15 commits intomainfrom
simplify-screenshot-testing
Mar 31, 2024
Merged

simplify screenshot testing#447
kushalkolar merged 15 commits intomainfrom
simplify-screenshot-testing

Conversation

@kushalkolar
Copy link
Copy Markdown
Member

@kushalkolar kushalkolar commented Mar 23, 2024

use ideas from pygfx/pygfx#684

closes #446
A threshold of 0.2 seemed like it was too lenient, I was able to change the color of one of the circles in the line collection examples and the RMSE was 0.09

@kushalkolar kushalkolar marked this pull request as ready for review March 23, 2024 04:24
@kushalkolar kushalkolar requested a review from clewis7 as a code owner March 23, 2024 04:24
@kushalkolar
Copy link
Copy Markdown
Member Author

kushalkolar commented Mar 23, 2024

Comment thread examples/tests/test_examples.py Outdated
@kushalkolar
Copy link
Copy Markdown
Member Author

So on Mac there seems to be large diffs with the heatmaps, no sure why, will figure out later.

diff-heatmap-rgb
diff-heatmap_cmap-rgb
diff-heatmap_data-rgb
diff-heatmap_vmin_vmax-rgb

@kushalkolar
Copy link
Copy Markdown
Member Author

kushalkolar commented Mar 23, 2024

@kushalkolar kushalkolar requested a review from almarklein March 23, 2024 05:17
@kushalkolar kushalkolar merged commit 9a1bfbe into main Mar 31, 2024
@kushalkolar kushalkolar deleted the simplify-screenshot-testing branch March 31, 2024 09:35
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.

better image comparison for tests

2 participants