Skip to content

Adjust recalculated figure size based on ratio instead of difference#6

Open
Naikless wants to merge 2 commits into
MagicMuscleMan:masterfrom
Naikless:master
Open

Adjust recalculated figure size based on ratio instead of difference#6
Naikless wants to merge 2 commits into
MagicMuscleMan:masterfrom
Naikless:master

Conversation

@Naikless

Copy link
Copy Markdown

Naikless and others added 2 commits December 26, 2022 00:59
This solves issues with pgfplots' groupplot environments that lead to negative widths/heights while iteratively searching for the correct scaling factor.
@fwdekker

fwdekker commented Jan 17, 2025

Copy link
Copy Markdown

Thank you for this, I was getting the same bug. I compiled your code*, put the resulting tikzscale.sty in my project, and the bug was solved :-)

* I couldn't get compilation working so I actually just took the .sty file bundled with my LaTeX installation and applied the same changes you made in this PR.

@Naikless

Copy link
Copy Markdown
Author

Glad that the effort helped someone else, too. 😄

Bit of a bummer that this project has been abandoned quite a while ago, same as tikzplotlib.

@MagicMuscleMan

Copy link
Copy Markdown
Owner

Yes, sorry, for that, but I haven't found the motivation to work on this for years (not because this is boring, but because other projects interest me more) and it is unlikely to change soon.
But if anyone wants to take over I am open to that. That's why it's open source after all.

@Naikless

Copy link
Copy Markdown
Author

No need to be sorry, that's just what happens with these kind of open source projects, I didn't mean it as a complaint.

I mean, my fork already exists, in the end it's just the question of which version is referenced in CTAN. Or it just needs someone here to merge potential PRs.

@fwdekker

Copy link
Copy Markdown

@MagicMuscleMan Don't worry, I know the feeling :-)

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.

3 participants