Skip to content

Implement always compare mode for hash library - #108

Merged
Cadair merged 18 commits into
matplotlib:masterfrom
Cadair:always_compare
Jan 5, 2022
Merged

Implement always compare mode for hash library#108
Cadair merged 18 commits into
matplotlib:masterfrom
Cadair:always_compare

Conversation

@Cadair

Copy link
Copy Markdown
Contributor

Useful for CI so you can see the changes from the reference images even if the hash library has been updated

@Cadair
Cadairforce-pushed the always_compare branch 3 times, most recently from ea3cffc to 80239e0CompareNovember 9, 2020 14:32
@Cadair

Copy link
Copy Markdown
ContributorAuthor

Needs tests.

@ConorMacBrideConorMacBride mentioned this pull request Dec 9, 2021
Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride

Copy link
Copy Markdown
Member

I've rebased locally and this PR conflicts with #121 so need to need to find a way around that.

Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride

Copy link
Copy Markdown
Member

The hash libraries for mpl20, mpl21, mpl22 and mpl30 are using ft2104 even though Matplotlib apparently hasn't changed the Freetype library in 6 years: https://github.com/matplotlib/matplotlib/blame/085fdd0771c01ccf964459033904d31bf79e611a/lib/matplotlib/__init__.py#L1191

I don't understand how this could have worked before but I'll rename the libraries to have ft261 and update the hashes where necessary. Let me know if I have missed something.

Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
Signed-off-by: Conor MacBride <conor@macbride.me>
@ConorMacBride

Copy link
Copy Markdown
Member

@Cadair Can you review please?

@Cadair
Cadair merged commit 94d732c into matplotlib:masterJan 5, 2022
@Cadair
Cadair deleted the always_compare branch January 5, 2022 13:41
Sign up for freeto 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.

3 participants

@Cadair@ConorMacBride@dstansby