[FIX] Tick-formatting for spharpy.plot.scatter - #319
Conversation
7a8cd21 to
b0d4fcc
Compare
ahms5
left a comment
There was a problem hiding this comment.
Thank you for taking care. Have you recreate the baseline plots? I bet, sth would change there. I would also recomment to add a test, so that this woulnt happen again.
b0d4fcc to
aefa613
Compare
spharpy.plot.scatterspharpy.plot.scatter
spharpy.plot.scatterspharpy.plot.scatter
|
Hey @hoyer-a, I was wondering if there's any progress on this? If no I think I'd prepare the next bug release without this PR included. I feel that the bugfixes currently waiting to be released are more urgent than this one, so don't worry if you don't have the bandwidth to finalize this one at the moment. |
|
Not really, i made some progress with the tick locators and formatting but it's still no good solution. |
spharpy.plot.scatterspharpy.plot.scatter
spharpy.plot.scatterspharpy.plot.scatter
828e9c9 to
12b6207
Compare
fa49f0a to
e1a6068
Compare
mberz
left a comment
There was a problem hiding this comment.
Looks good! Only have a tiny nitpick on the code to make it more readable.
Can you please add a history entry?
Co-authored-by: Marco Berzborn <mberz@users.noreply.github.com>
e409683 to
c3df7ef
Compare
Which issue(s) are closed by this pull request?
Closes #313