Skip to content

Conversation

@theengineear
Copy link
Contributor

Note that we don’t necessarily require numpy for in all the FF
methods. We’re just looking to make the compatibility-breaking
change so that we can depend on it in 2.0.0.

Also note that I’d forgotten to include figure_factory in setup.py
before ;__;.

@theengineear
Copy link
Contributor Author

@chriddyp for review. Seem ok?

//cc @choldgraf 😸 just some setup towards a faster figure factory methods.

@theengineear theengineear mentioned this pull request Jan 18, 2017
@theengineear
Copy link
Contributor Author

^^ huh, rebuilt, seems like an intermittent npm fail?

@choldgraf
Copy link
Contributor

sweet! thx for keeping me in the loop

@theengineear theengineear mentioned this pull request Jan 18, 2017
10 tasks
Copy link
Member

@chriddyp chriddyp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💃

Note that we don’t necessarily *require* `numpy` for in all the FF
methods. We’re just looking to make the compatibility-breaking
change so that we can depend on it in `2.0.0`.

Also note that I’d forgotten to include `figure_factory` in setup.py
before ;__;.
@theengineear theengineear merged commit b81d9f1 into 2.0.0 Jan 18, 2017
@theengineear theengineear deleted the ff-requires-numpy branch January 18, 2017 21:08
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.

4 participants