Skip to content

psimage: fix -B+t>title> issue#9014

Open
joa-quim wants to merge 2 commits into
masterfrom
fix-5635
Open

psimage: fix -B+t>title> issue#9014
joa-quim wants to merge 2 commits into
masterfrom
fix-5635

Conversation

@joa-quim
Copy link
Copy Markdown
Member

Fix #5635

Assites-by: Claude Opus 4.7

Fix #5635

Assites-by: Claude Opus 4.7
@joa-quim joa-quim requested a review from a team May 14, 2026 10:47
@Esteban82 Esteban82 added the add-changelog Add PR to the changelog label May 14, 2026
Copy link
Copy Markdown
Member

@Esteban82 Esteban82 left a comment

Choose a reason for hiding this comment

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

It works. I got this image with this command:

gmt image @needle.jpg -Dx0/0+w7c -JM12c -JZ14c -p200/30 -B0 -B+w1p+gred3+t"title" -png plot

Image

@joa-quim
Copy link
Copy Markdown
Member Author

And if you do not use -B0 you should see no frame.

@joa-quim
Copy link
Copy Markdown
Member Author

But things are not completely right (and Claude warned me about it). The title is not rotated.

@Esteban82
Copy link
Copy Markdown
Member

But things are not completely right (and Claude warned me about it). The title is not rotated.

Right. I didn't pay attention to that.

@joa-quim
Copy link
Copy Markdown
Member Author

The title angle is now fixed.

This PR now causes test/grdmix/grdmix.sh test to fail. But I think the grdmix.ps baseline is wrong because it misses the frame that is implied by that script using -B0. So, it turns out that apparently we killed two bugs with one stone.

@Esteban82
Copy link
Copy Markdown
Member

The title angle is now fixed.

I'm affraid that I try it and I am getting the same figure.

@joa-quim
Copy link
Copy Markdown
Member Author

Maybe you didn't get the updated branch? Can't retest right now as I'm debugging something else in another branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

-B option in image module

2 participants