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

Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies; add FAQ #398

Conversation

willfurnass
Copy link
Collaborator

@willfurnass willfurnass commented Mar 27, 2020

Need to do so without resurrecting #158 (duplicate Figures created upon ipywidget state changes).

The trick here is to set a minimum version of ipython (7.10.0).

Also added a FAQ that recommends users update their dependencies if they
encounter the duplicate Figure issue.

@willfurnass willfurnass self-assigned this Mar 27, 2020
@willfurnass willfurnass changed the title Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies [WIP] Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies Mar 27, 2020
@willfurnass willfurnass changed the title [WIP] Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies (WIP) Mar 27, 2020
This unpinning requires that we also set a minimum version of the
ipython dependency to avoid the recurrence of an issue where
interactively changing the state of ipywidgets causes duplicate Figures
to be created.

Also added a FAQ that recommends users update their dependencies if they
encounter this issue.
@willfurnass willfurnass force-pushed the unpin_ipykernel_notebook_pyzmq_tornado branch from 7ddbdef to 560adbe Compare March 27, 2020 13:03
@willfurnass willfurnass changed the title Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies (WIP) Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies Mar 27, 2020
@willfurnass willfurnass changed the title Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies Unpin versions of ipykernel, notebook, pyzmq and tornado dependencies; add FAQ Mar 27, 2020
@willfurnass
Copy link
Collaborator Author

Huh, CI job status info isn't being shown for this PR. Jobs are running though: https://travis-ci.org/github/DiODeProject/MuMoT/builds/667690510

@willfurnass willfurnass added this to the Second release milestone Mar 27, 2020
@willfurnass
Copy link
Collaborator Author

CI jobs completed sucessfully; awaiting review.

Copy link
Contributor

@jarmarshall jarmarshall left a comment

Choose a reason for hiding this comment

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

Having checked on MyBinder via Safari, this seems not to reopen #158 , and close #397 - suggest @joefresna takes a look in other browsers, but I'm happy

Copy link
Contributor

@joefresna joefresna left a comment

Choose a reason for hiding this comment

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

I tested it on Chrome and Safari and in both cases it works fine for me.
Latex symbols do not break the line and I don't have duplicated figures.

To be sure, I executed the following command to test the system. Let me know if I've done something wrong...

git fetch --all
git checkout unpin_ipykernel_notebook_pyzmq_tornado
conda update conda
conda create -n mumot-env-pr python=3.7
conda activate mumot-env-pr
conda install graphviz
python -m pip install /Users/joefresna/MuMoT/
jupyter notebook

@jarmarshall
Copy link
Contributor

@willfurnass unless you have a reason not to I think this is now good to merge...

@willfurnass willfurnass merged commit c34a247 into DiODeProject:master Apr 2, 2020
@willfurnass willfurnass deleted the unpin_ipykernel_notebook_pyzmq_tornado branch April 2, 2020 08:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants