GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||
GeezerVison is showing github.com — Mnt/multi imageset by tacaswell · Pull Request #25734 · matplotlib/matplotlib Conversation
Member
PR SummaryThis is not ready for review at all yet, but opening so that I can refer to it in the readme of https://github.com/tacaswell/mpl-imageset-demo. This code is .... sub optimal and needs to be refactored (the The rough scheme is as such:
An example of the image list file looks like The logic on this format is :
The json dumped in the generated baseline directory looks like Details{
"test_basic/image.png": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
},
"test_basic/line.pdf": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
},
"test_basic/line.png": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
},
"test_basic/line.svg": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
},
"test_other/hist.png": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 1,
"sha": "0000000000000000000000000000000000000000"
},
"test_other/scatter.pdf": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
},
"test_other/scatter.png": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
},
"test_other/scatter.svg": {
"mpl_version": "3.8.0.dev913+g8ce98ade16.d20230420",
"rev": 0,
"sha": "8691c2d2bf868a23c80f6ac85ba184a917e49f03"
}
}
The sha is for the computer, the rev is for the human, and the mpl version is for debugging. open questions
PR ChecklistDocumentation and Tests
Release Notes
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactions
Member
Author
All reactions
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page.
tacaswell
force-pushed
the
mnt/multi_imageset
branch
from
c047f0b to
8dab81f
Compare
tacaswell
force-pushed
the
mnt/multi_imageset
branch
2 times, most recently
from
26911a3 to
4a0d7ad
Compare
|
There was an error while loading. Please reload this page.
|
Of course there are complications 🤣
Tasks:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
The image_lists.txt for Matplotlib are now checked in so people can take a look at what those look like at full scale. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Should the metadata.json files be included in the repo? or are those intended to be local only? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
They are intended to go with the generated sets of images, however given that we are not at the point where we can actually pull the images out of the repo, but I still want the tests to run they need to be checked in for now. The final version of this PR will squash them out of existence. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I think this is at a point where (modulo the metadata.json files) where it is starting to be reviewable. The internal names are...not great, but I think I have pulled it apart enough that it is not complete spaghetti code. MPLTESTIMAGEPATH='/tmp/test_images2' MPLGENERATEBASELINE=1 pytestshould work and generate you a full tree of test images on this branch! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
8da8195 to
5afeca3
Compare
5afeca3 to
6125208
Compare
…_neg_coords"" This partially reverts commit 7b71257. Too many tests were removed, restore the extra tests.
Flagging that we should generate images via the plugin means we do not have access at import time to if we intend to generate the images so remove this check.
The factor of 100 reduces the window of collisions to 10ms which is an acceptable risk.
Force this out of existence eventually
6125208 to
cab2a30
Compare
|
This now includes a pytest plugin (😱) so that works. The All of these names need some feedback. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
This seems to be working (successfully generated test images with a new freetype and it passed against them)! I'm now sure that this is going to work, but lots of details left. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
not going to get this done in the next few weeks. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.