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

Better plots tikz 2 #412

Merged
merged 12 commits into from
Jan 2, 2025
Merged

Better plots tikz 2 #412

merged 12 commits into from
Jan 2, 2025

Conversation

cedounet
Copy link
Collaborator

@cedounet cedounet commented Jan 1, 2025

Still missing one plot but for early preview

@cedounet cedounet requested a review from hendricius January 1, 2025 23:06
@hendricius
Copy link
Owner

Wow. Incredible work. I just compiled the book locally and this looks lovely.

@cedounet
Copy link
Collaborator Author

cedounet commented Jan 2, 2025

Not sure if you are sarcastic or not... but yeah there is value in having same looking plots and text in a document... I now need to reproduce the other plot as a CSV file and we can have it.

I find the 1:1 ratio (i.e. square plots) not to be super adequate for the temperature plots, and was thinking of shrinking height to maybe half the width?

@hendricius
Copy link
Owner

@cedounet not sarcastic - I mean it ❤️.

Yes - that would be a great improvement.

Just looks nicer... maybe a littue too squashed?
@cedounet cedounet marked this pull request as ready for review January 2, 2025 18:12
@cedounet
Copy link
Collaborator Author

cedounet commented Jan 2, 2025

I did rescale them... it is better. if you find them too squashed then go ahead and change the height to 0.6 or something :)

I am sure somewhere there is a piece of software that can extract curves from a pic to csv... but didn't find anything useful from a quick google. So propose we merge this now if you are ok with it?

@cedounet cedounet merged commit 48916a5 into main Jan 2, 2025
2 checks passed
@cedounet cedounet deleted the better_plots_tikz_2 branch January 3, 2025 16:33
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.

2 participants