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

Starting work on plotly 3.0.0 migration #122

Merged
merged 27 commits into from Sep 9, 2018
Merged

Starting work on plotly 3.0.0 migration #122

merged 27 commits into from Sep 9, 2018

Conversation

timkpaine
Copy link
Contributor

@timkpaine timkpaine commented Jul 11, 2018

this is just a first pass, theres some major work to do:

  • 2 "FIXME TKP" sections were just commented out in plotlytools
  • Docstrings and defaults are wrong in many cases (e.g. "horizontal"/"vertical" not valid, "dot" not a valid marker type)
  • 3 "FIXME TKP" sections commented out in quant_figure
  • Fix JupyterLab support

@timkpaine
Copy link
Contributor Author

should probably follow #119 and switch everything to use raw dictionaries until the last step

@timkpaine
Copy link
Contributor Author

all the examples from the notebooks are working (albeit with warnings, also for latest pandas you need to call .mean() after resample). Need to check quantfigs (I commented out the tests for now)

@timkpaine
Copy link
Contributor Author

shouldn't need bullet 3 plotly/plotly.py#1071

@timkpaine
Copy link
Contributor Author

somebody get me a beer
screen shot 2018-08-21 at 12 22 49 pm

@timkpaine
Copy link
Contributor Author

timkpaine commented Aug 21, 2018

this is done, just waiting on the plotly bullet but it should still work

@dhirschfeld
Copy link

🍻!

@timkpaine
Copy link
Contributor Author

plotly bullet is resolve, make sure to install @jupyterlab/plotly-extension in addition to plotlywidget

@dhirschfeld
Copy link

Confirmed that this PR works great with nteract:
image

(previously threw an exception)

@dhirschfeld
Copy link

Would love to get this functionality merged!

@dnanhkhoa
Copy link

Great, it works 👍 can't wait for merging
image

@davhin
Copy link

davhin commented Aug 26, 2018

The cufflinks version in the pull request from timkpaine works well for me with:

plotly 3.1.0,
jupyterlab 0.34.2

and
jupyter-widgets/jupyterlab-manager v0.37.2
jupyterlab/plotly-extension v0.16.0
plotlywidget v0.2.1

Just trying to add some confidence, not meaning to confirm that it works at all ;)

@santosjorge
Copy link
Owner

Going through this PR this week. Thank you for the work so far @timkpaine !

@lilcatria
Copy link

secondary axis example seem to be breaking ,

plotly 3.1.1

fig=cf.datagen.lines(3,columns=['a','b','c']).figure()
fig=fig.set_axis('b',side='right')
cf.iplot(fig)

AttributeError Traceback (most recent call last)
in ()
1 fig=cf.datagen.lines(3,columns=['a','b','c']).figure()
----> 2 fig=fig.set_axis('b',side='right')
3 cf.iplot(fig)

AttributeError: 'dict' object has no attribute 'set_axis'

@timkpaine
Copy link
Contributor Author

I'll take a look, there's one other issue I know about that I need to fix

@santosjorge
Copy link
Owner

alright. here we go.

@santosjorge santosjorge merged commit c7bb91f into santosjorge:master Sep 9, 2018
@timkpaine
Copy link
Contributor Author

timkpaine commented Sep 9, 2018

@jonmmease FYI

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

Successfully merging this pull request may close these issues.

None yet

6 participants