Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Require matplotlib < 3.8 for pgf.common_texification #597

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

JasonGross
Copy link

As per #559

@FriedrichFroebel
Copy link

Wouldn'it it be nicer to have an actual fix instead?

@JasonGross
Copy link
Author

It would, but I don't have time to prepare an actual fix right now, and this is an easy patch to revert when a fix lands, and I don't think breaking pip's solver on this package while we wait for a fix(/wait to find time to fix it) is a good idea. Note that I didn't mark this PR as fixing the corresponding issue, and I don't think it makes sense to close that issue until a proper fix lands.

Copy link

@XxSonicGhostxX XxSonicGhostxX left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just want everything in my env to be normal not sure what would implement that but that's my requested change

@JasonGross
Copy link
Author

Wouldn'it it be nicer to have an actual fix instead?

It seems there are a bunch of fixes for this in the wild that have not been turned into PRs, e.g., RobertDeibel#2, lukashof@9eddaba, lucaslrodri@860e4a5

@lukashof
Copy link

lukashof commented Apr 3, 2024

I created a PR #602. But I think it's been a while since something got merged, unfortunately.

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.

4 participants