-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Some documentation on main page points to obsolete subdirectories #10016
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
Comments
I can't replicate this on Master, and the doc example built OK, so I'm not clear where your error is coming from. Are you sure you are using 2.1.1? i.e. |
Hmmm, well I can't replicate using 2.1.1 and MacOSX as well, so... My guess would be your install is slightly messed up. Try a fresh install? |
How are you running the example from inside |
I run the example from the terminal, not from |
Not convinced you were actually running w 2.1.1 then. Copy the example locally and run w the print statements. If it was me I’d stick w Anaconda. Make sure you install pip and IPython into your environment. Don’t install matpltolib w pip. Conda works really well but can still lead to conflicting versions of things lying around. |
You're right, it's the 1.3.1 version with the python 2.7.10. So, the only way to run properly the example is with another version than the 2.1.1. Sorry, I'm very slow because I'm a little lost with the different version of python and the conflicts between the versions. So you think it's better to work with Conda than the version with Homebrew ? |
No I’m not convinced you were using 2.1.1 when you got the error. Can you please reproduce the error with the version number printed from the script. I can not reproduce the error on a Mac using 2.1.1 so the most likely problem is your setup. Again strongly suggest anaconda and using conda to install matplotlib not pip. |
I'll close. Feel free to reopen if you really think there is a matplotlib bug and you have solid debug info. If you continue to have install problems please try the mailing list or stack overflow. |
Please put the print statements in the example. Otherwise you can easily trick yourself as to what version of python you are running. |
Copy the script you are running to your local directory, and add the print statements after you import matplotlib:
|
Sorry, typo. Don't put the version in quotes.... |
Yes, I just modified the image, I saw after posting sorry |
And that gives you the distorted radar chart? I'll re-open, but someone else will have to reproduce... |
Are you sure you didn't modify |
I modified the legend only just to test. But to be definitely sure I will test to download the exact file but there is no chance to be that because I have already redowloaded it and tested it. I have already completely uninstall conda and try again too. |
It seems to be the center of the axis, doesn't it ? |
Have you ever modified your matplotlibrc file? Maybe there is a modified
rcparam that is messing with autoscaling?
…On Mon, Dec 18, 2017 at 12:36 PM, Quentin François ***@***.*** > wrote:
It seems to be the center of the axis, doesn't it ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#10016 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AARy-LeXDKRM5HkyZM7G922sgAlE6ivDks5tBqKhgaJpZM4RDj65>
.
|
I've never modified the matplotlibrc file. Just downloaded conda and updated matplotlib to 2.1.1. |
Just to be sure, can you remove |
I come back with another test : with anaconda2 (python version 2.7) I have the same problem while with python2.7 coming with MacOS I haven't the bug. Very weird .. |
I just tried by installing python 3.6.4., create a virtualenv and try to run and the same bug ! The only way is to use the python2.7 coming with MacOS |
I tried on W10 with python 3.6 and I have the bug too. I think there is something I'm making wrong.. This isn't the code because we juste copy-paste the code from internet. I'm really lost .. |
What exact code are you using? Please re-write it here. |
I copy paste the code on this page : https://matplotlib.org/examples/api/radar_chart.html |
That example changed in 2.1.1. Whats more, the docs shouldn't have those links any more, and indeed you are linking to 2.0.2 there. The new link is https://matplotlib.org/gallery/api/radar_chart.html This is definitely a documentation bug though. The canonical docs should not still have links to https://matplotlib.org/examples/ |
ping @tacaswell @QuLogic @anntzer or anyone else who understands how the docs are all linked up. This was pretty confusing: |
Yes it is working !! Finally !! Thank's a lot for your help @jklymak and sorry I really didn't see that was a old version .. |
Not at all your fault. Sure google sent you there and there is no indication it’s old doc except up in the tab title. The canonical docs really should not point to old pages! |
See #9556. |
What is different between the two examples? The only change I can see is 202f40f and I do not understand why that would cause this... And yes, we definitely need to clean up the 'canonical' docs... @quentinfrancois0 Sorry for the frustration and thanks for sticking with us! |
No |
Theta changed meaning in one of the calls. |
I'm going to close this, as the example in the current version of the docs works fine. |
I am having the same problem using W10, python 3.6 and matplotlib v2.2 |
Could you upgrade to Matplotlib 2.2.2 (the latest version) and try again? |
They should make the v2.2.2 bigger in the websites from where we are copying tho code and have a specific warning for that . It took me 2 hours to figure the version problem. |
I'll re-open. Someone needs to sort out how to get the old pages to at least have a big blaring banner across them that says they are obsolete. |
@JLegs and I are working on documentation duplication at the sprint. The static site has a ton* of old-version docs, that are manually propagated to the top-level on release. Our plan is to inject old-version docs with approximate canonical links (for bots) and banners (for humans) via script, one folder at a time. The first-approximation canonical link (#9556) is to delete the version string from the url. More sophisticated approaches (#10854) to identifying similar-content/different-named descendants are punted. In the future, such would improve cleanup on docs release, relevant to OP's issue. * All versions of matplotlib will remain documented in the (huge) static site because scientific users need reproducibility but can't maintain local packages. (Notably, 2.2.2 is long-term support officially; 1.5.3 is not.) Edit: PR submitted. But - nothing apparent forwards to a 404 because nothing, not even 'examples' transitioned to 'gallery', has been cleanup'd. We could begin cautiously: instead of deleting, add warnings to 'current' (unversioned) docs which aren't in latest version? |
Canonical links and redirects were added in matplotlib/matplotlib.github.com#49. There is also a banner noting that the page is outdated at the top of the page. |
EDIT (jklymak) Old title is below:
Fundamental issue is that If you do a search on "radar plot matplotlib" you get sent here:
https://matplotlib.org/examples/api/radar_chart.html
This of course refers to 2.0.2 and
/examples/
isn't a directory any more in our tree.Example of radar chart (matplotlib v.2.1.1) doesn't work
I just tried to run the radar chart example on a Mac (MacOS 10.13.1) with python 3.5.4 and matplotlib 2.1.1. and the radars aren't displayed correctly.
I found that this is the
set_varlabels
function which shifts the radars. Without the line 189 :ax.set_varlabels(spoke_labels)
the radars look like that (but without the labels).As I'm new on matplotlib and python I didn't succeed in finding a solution.
Thank's for your help
The text was updated successfully, but these errors were encountered: