Repository navigation
[Bug]: ax.plot(x,y) disappears after changing y_scale #25124
Description
Bug summary
The output of ax.plot(x,y) disappears while changing y_scale from 'log' (initial scale for the y axis) to 'linear'.
Code for reproduction
import numpy as np
import matplotlib.pyplot as plt
samples = np.random.normal(size = 1000)
x = np.linspace(-5,5,1000)
fig, ax = plt.subplots()
ax.hist(samples, log = True, density = True)
ax.plot(x, np.exp(-x**2/2)/np.sqrt(2*np.pi))
fig.savefig('log.pdf')
ax.set_yscale('linear')
fig.savefig('lin.pdf')Actual outcome
Expected outcome
Additional information
The expected outcome is generated with matplotlib==3.5.3
fill_between and hist are working fine.
Operating system
MacOSX
Matplotlib Version
3.6.3
Matplotlib Backend
MacOSX
Python version
3.9.13
Jupyter version
No response
Installation
pip
Activity
tacaswell commented on Feb 1, 2023
I can confirm that this is a regression between 3.6.2 and 3.6.3, is still present in 3.7.0rc1 and is not a pdf related problem.
Slightly modified reproduction script:
Thank you for the very clear report @sterinaldi !
ksunden commented on Feb 1, 2023
bisects to 8d8d3f5 (#24684)
which is the (manual) backport of #24654
Also interacts with #24634, which was modifying some of the same areas of code
Cache invalidation strikes again
ksunden commented on Feb 1, 2023
Interestingly, the original PR (#24654) explicitly mentions the "fix log/linear toggle", and states that it "clearly works fine even with this removal"
tacaswell commented on Feb 1, 2023
It matters if there is a draw before the switch. Commenting out the save and the pause makes the linear one look correct.
Further linear -> log works, but log -> linear does not...
tacaswell commented on Feb 1, 2023
ok, I have a fix for this and see how to test it, PR coming soon.
FIX: fully invalidate TransformWrapper parents before swapping
FIX: fully invalidate TransformWrapper parents before swapping
FIX: fully invalidate TransformWrapper parents before swapping
FIX: fully invalidate TransformWrapper parents before swapping