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.

Sign up for GitHub

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

Jump to bottom

Avoid the IPython widget deprecation warning if possible. #311

Merged
merged 2 commits into from
Dec 28, 2015
Merged

Avoid the IPython widget deprecation warning if possible. #311

merged 2 commits into from
Dec 28, 2015

Conversation

moorepants
Copy link
Member

Partially addresses #261.

  • There are no merge conflicts.
  • If there is a related issue, a reference to that issue is in the
    commit message.
  • Unit tests have been added for the bug. (Please reference the issue #
    in the unit test.)
  • The tests pass both locally (run nosetests) and on Travis CI.
  • The code follows PEP8 guidelines. (use a linter, e.g.
    pylint, to check your code)
  • The bug fix is documented in the Release
    Notes    .
  • The code is backwards compatible. (All public methods/classes must
    follow deprecation cycles.)
  • All reviewer comments have been addressed.

@moorepants moorepants added this to the 0.3.1 milestone Dec 26, 2015
@moorepants
Copy link
Member Author

@pydy/pydy-developers Needs quick review and merge.

@chrisdembia
Copy link
Contributor

Merge conflict.

@moorepants
Copy link
Member Author

It is updated now.

chrisdembia added a commit that referenced this pull request Dec 28, 2015
@chrisdembia
Merge pull request #311 from moorepants/ipy4
419451e 
Avoid the IPython widget deprecation warning if possible.
@chrisdembia chrisdembia merged commit 419451e into pydy:master Dec 28, 2015
@moorepants moorepants deleted the ipy4 branch January 6, 2016 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.3.1
Development

Successfully merging this pull request may close these issues.
2 participants
@moorepants @chrisdembia