Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 02:00
    QuLogic labeled #22308
  • 02:00
    QuLogic commented #22308
  • 01:58
    QuLogic labeled #22308
  • 00:40
    QuLogic milestoned #22294
  • 00:31
    QuLogic labeled #22294
  • Jan 24 23:54

    QuLogic on v3.5.x

    Backport PR #22306: FIX: ensure… Merge pull request #22307 from … (compare)

  • Jan 24 23:54
    QuLogic closed #22307
  • Jan 24 23:53
    QuLogic synchronize #22279
  • Jan 24 23:50
    jiashuhan edited #22308
  • Jan 24 23:30
    jiashuhan opened #22308
  • Jan 24 23:11

    QuLogic on main

    In mpl_toolkits, use the same f… Merge pull request #22097 from … (compare)

  • Jan 24 23:11
    QuLogic closed #22097
  • Jan 24 22:47
    meeseeksdev[bot] milestoned #22307
  • Jan 24 22:47
    meeseeksmachine opened #22307
  • Jan 24 22:47

    timhoffm on main

    FIX: bury import of matplotlib.… Merge pull request #22306 from … (compare)

  • Jan 24 22:47
    timhoffm closed #22306
  • Jan 24 22:47
    timhoffm closed #22305
  • Jan 24 22:32
    tacaswell commented #22283
  • Jan 24 22:20
    oscargus commented #22283
  • Jan 24 22:18
    tacaswell milestoned #22298
Thomas A Caswell
@tacaswell
it looks like something changed under us in the OSX on azure envirmont and imagemagik is fighting back...
Rohit Rawat
@xordux
Previously (few weeks ago) every check passed smoothly, but today I change assert statement to py.warn and this started failing(I also synced my repo today).
Thomas A Caswell
@tacaswell
between the last CI that passed on azure OSX and the first that failed imagemagick changed from 7.0.10-31 to 7.0.10-34
Rohit Rawat
@xordux
I can't find any release notes of imagemagick in web. Should comparing two releases' code help ?
Thomas A Caswell
@tacaswell
https://github.com/ImageMagick/ImageMagick/releases <- looks like they have a very rapid super-micro release cadence
Rohit Rawat
@xordux
And comparing releases got me more confused :D .. still checking if I get any hint
Jody Klymak
@jklymak
The tk error is more problematic.
Rohit Rawat
@xordux
@jklymak strange thing is that it is passing in Windows_Py38 but not in Windows_Py37

Regarding tk error: Will increasing _test_timeout (which in 10 right now) help ?

Edit: 7th line in test_backend_tk.py
_test_timeout = 10 # Empirically, 1s is not enough on Travis.

Jody Klymak
@jklymak
Yeah, it could just be a transitory failure.
Rohit Rawat
@xordux
So should I test it again or increase timeout to some value?
Jody Klymak
@jklymak
I'd just try running again. I don't understand the test well enough to say. @dopplershift is the resident animation guru
Thomas A Caswell
@tacaswell
Do you use brew locally?
convert -size 800x600 -depth 8 -delay 20.0 -loop 0 rgba:- /tmp/temp.m4v
Rohit Rawat
@xordux
me? I work on Windows 10.
Thomas A Caswell
@tacaswell
I know @jklymak is a mac user ;) . If you update the imagemagick does that command work?
Rohit Rawat
@xordux
I am not sure if it is failing in my system. I am checking it
BTW is there some magic phrase to re-test a PR? Like mentioned here
Rohit Rawat
@xordux
or I need to do something like mentioned in this answer ? To retest PR.
and about imagemagick: I tried running this test(in my branch and current synced master branch both) and getting this error:
AttributeError: module 'matplotlib._c_internal_utils' has no attribute 'display_is_valid'
Thomas A Caswell
@tacaswell
did you re-install from source?
also try a git clean -xfd to clear the build products and force the c-extensions to be re-complied
Rohit Rawat
@xordux
No, I thought it's not required. I will try it now.
Thomas A Caswell
@tacaswell
also make sure you imagemagick is all the way updated, I can produce this on linux as well
Rohit Rawat
@xordux
sure thing 👍
Thomas A Caswell
@tacaswell
✔ 13:02:38 $ convert -size 800x600 -depth 8 -delay 20.0 -loop 0 /tmp/test.png /tmp/temp.m4v
convert: delegate failed `'avconv' -nostdin -v -1 -i '%M%%d.pam' -plays %I '%u.%m' 2> '%u'' @ error/delegate.c/InvokeDelegate/1898.
Rohit Rawat
@xordux
@tacaswell just to mention, imagemagick is not already installed in my system. I should install it and check then, right?
Thomas A Caswell
@tacaswell
definitly an imagemagick bug :-p
given that I think this is a regression in convert that shows up on both mac and linux, unless you want to help out imagmagick and confirm the bug is also present in 7.0.10-34 in windows I would not worry about it
Rohit Rawat
@xordux
is that test.png present somewhere or you created a blank image of size 800x600 to test it?
Thomas A Caswell
@tacaswell
I saved a mostly empty 800x600 image
In [1]: import matplotlib.pyplot as plt                                                                                                                                                                                                                                                                

In [2]: fig, ax = plt.subplots()                                                                                                                                                                                                                                                                       

In [3]: fig.set_size_inches(8, 6)                                                                                                                                                                                                                                                                      

In [4]: fig.savefig('/tmp/test.png')
Rohit Rawat
@xordux
ok, thank you.
@tacaswell getting same in windows as well:
(test_env) C:\PersonalProjects\MachineLearning\Github\18151>"C:\Program Files\ImageMagick-7.0.10-Q16-HDRI\convert" -size 800x600 -depth 8 -delay 20.0 -loop 0 tmp.png temp.m4v
convert: delegate failed `"ffmpeg.exe" -nostdin -v -1 -i "%M%%d.pam" -plays %I "%u.%m" 2> "%u"' @ error/delegate.c/InvokeDelegate/1898.
Thomas A Caswell
@tacaswell
can you please leave a comment on the imagemagick issue ?
Rohit Rawat
@xordux
yes, right away.
Thomas A Caswell
@tacaswell
thanks!
Rohit Rawat
@xordux
If I want to retest a PR without new commit, then should I do it like this ? Or is there some other way you do it?
Thomas A Caswell
@tacaswell
if the UI does not have a "re run the test" button in the check section (it is not always clear to me when things show up for me because I am an admin and when things are there for everyone), then close and re-open the PR
on the "checks" tab I have a "re-run failed" link next to the failed tests
Rohit Rawat
@xordux
There is no such button that I can find. I will close and re-open then.
Grigory Starkov
@Gregstrq
When a 3d surface is plotted, each of the surface patches has a constant color. Is it possible to have the colors change continously? (I think one can use linear interpolation inside the patch)
Bruno Beltran
@brunobeltran

looks like font_manager._rebuild is gone on master. What's the go-to advice for users that are installing many new fonts after installing matplotlib? Should I tell them to clear mpl.get_cachedir() directly each time?

The only way to do it from inside mpl itself now seems to be something like

font_manager.fontManager = font_manager._load_fontmanager(try_read_cache=False)?

(not saying that my go-to advice for people was to call a private method, but if someone who isn't me needed to update their recommendations 🐑)
Antony Lee
@anntzer
I think there should just be a public API for that (I believe this may be mentioned somewhere in one of the long font_manager issues...)
Benjamin Root
@WeatherGod
#18738 is some sort of spam. Have we decided on how to handle these, or just close without comment?
Dieter Werthmüller
@prisae

I have an issue with matplotlib v3.3 and interactive plots. Everything works fine for matplotlib<3.3, when updating there is no interactivity. The ipywidget buttons work, but nothing happens:

  • In a Jupyter Notebook
  • using %matplotlib notebook
  • using ipywidgets

I read through the v3.3 release notes, but I couldn't find anything in particular ...

5 replies
Jody Klymak
@jklymak
Best open an issue.
7 replies