Skip to content
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

parcoords documentation mentions non-existent property #1298

Closed
skylogic004 opened this issue Dec 7, 2018 · 5 comments
Closed

parcoords documentation mentions non-existent property #1298

skylogic004 opened this issue Dec 7, 2018 · 5 comments

Comments

@skylogic004
Copy link

In the documentation for the parcoords chart, under the dimensions section, in tickvals and ticktext it says:

Only has an effect if tickmode is set to "array".

However, I've read the entire documentation for parcoords and there is no tickmode property (except in the colorbar section which is unrelated).

So, if I'm understanding correctly, then the documentation is wrong. Probably a copy+paste bug where someone forgot to remove the reference to tickmode?

p.s. is this the right place to report documentation issues? or only code issues?
p.p.s. it's also possible that this is a code issue, meaning that the tickmode was supposed to exist in the code but doesn't.

@jonmmease
Copy link
Contributor

Thanks for the report @skylogic004! Reporting docs issues here is fine, and then we can sort out the source of the error. In this case, it will need to be addressed in plotly.js, because that's where the master schema for all of the figure properties is maintained.

I've opened an issue over there at plotly/plotly.js#3313. I'm going to close this issue since plotly.py will automatically get the documentation updates once they are implemented and released in plotly.js. Thanks again for taking the time to bring this up!

@JamesPHoughton
Copy link

There still seems to be some difference between what is listed in the docs and what is available in the API. For example, the docs suggest a way to change properties of the coordinate labels:

However, using these results in an error:

ValueError: Invalid property specified for object of type plotly.graph_objs.Parcoords: 'labelangle'

    Valid properties:
        customdata
            ...
        customdatasrc
            ...
        dimensions
            ...
        dimensiondefaults
            ...
        domain
            ...
        ids
            ...
        idssrc
            ...
        labelfont
            ...
        line
            ...
        meta
            ...
        metasrc
            ...
        name
            ...
        rangefont
            ...
        stream
            ...
        tickfont
            ...
        uid
            ...
        uirevision
            ...
        visible
            ...

It would be quite nice to actually have these features, so this may not be a docstring error, but a problem with exposing the method.

@jonmmease
Copy link
Contributor

Hi @JamesPHoughton,

These properties are coming in plotly.py 4.1.0. They were just released in Plotly.js 1.49.0 a couple weeks ago. I guess the docs got ahead of us!

You can install version 4.1.0rc1 (released yesterday) if you'd like to try them out now.

@JamesPHoughton
Copy link

You just made my day. =)

@nicolaskruchten
Copy link
Contributor

We're working on a way of locking the Py docs and Py versions more tightly together so this kind of thing doesn't happen any more... sorry for the confusion!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants