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

Test Notebooks in Other OSs #7

Closed
greebie opened this Issue Feb 22, 2019 · 15 comments

Comments

Projects
None yet
3 participants
@greebie
Copy link
Collaborator

greebie commented Feb 22, 2019

We have the project working for OSx, but checking with other dependencies would be helpful before we send it to the cloud.

@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Feb 22, 2019

@ruebot I wonder if you are willing to give this a test run on Ubuntu given the instructions we have in the README? No rush.

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Feb 22, 2019

Sure!

@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Feb 22, 2019

Feel free to try out some different collections as well. It should just be a matter of adding the files to the data folder and changing coll_id to whatever. (When we I create the view in AUK, I'm going to change the default folder to ./)

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Mar 1, 2019

No dice getting past the "Setup" cell. Biggest difference in the versions of the required libraries is matplotlib at 3.0.3. Can you verify what you have on your end with pip show matplotlib?

Local desktop, Ubuntu 18:04

  • Python 3.6.7
  • jupyter 1.0.0
  • ggplot 0.11.5
  • matplotlib 3.0.3
  • numpy 1.16.0
  • pandas 0.24.1
  • networkx 2.2
  • nltk 3.4

screenshot from 2019-03-01 08-11-18

Clean Ubuntu 18.04 VM:

  • Python 3.7.2
  • jupyter 1.0.0
  • ggplot 0.11.5
  • matplotlib 3.0.3
  • numpy 1.16.2
  • pandas 0.24.1
  • networkx 2.2
  • nltk 3.4

screenshot from 2019-03-01 08-46-38

We also might want to consider creating a Dockerflle for this since we're going to have these problems all over the place. I'm happy to do this if y'all want.

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Mar 1, 2019

Forgot to add this:

$ pip install matplotlib==1.15.1
Collecting matplotlib==1.15.1
  Could not find a version that satisfies the requirement matplotlib==1.15.1 (from versions: 0.86, 0.86.1, 0.86.2, 0.91.0, 0.91.1, 1.0.1, 1.1.0, 1.1.1, 1.2.0, 1.2.1, 1.3.0, 1.3.1, 1.4.0, 1.4.1rc1, 1.4.1, 1.4.2, 1.4.3, 1.5.0, 1.5.1, 1.5.2, 1.5.3, 2.0.0b1, 2.0.0b2, 2.0.0b3, 2.0.0b4, 2.0.0rc1, 2.0.0rc2, 2.0.0, 2.0.1, 2.0.2, 2.1.0rc1, 2.1.0, 2.1.1, 2.1.2, 2.2.0rc1, 2.2.0, 2.2.2, 2.2.3, 2.2.4, 3.0.0rc2, 3.0.0, 3.0.1, 3.0.2, 3.0.3)
No matching distribution found for matplotlib==1.15.1
@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Mar 1, 2019

Hi Nick: Here's what I have for "pip show matplotlib"

Name: matplotlib
Version: 3.0.2
Summary: Python plotting package
Home-page: http://matplotlib.org
Author: John D. Hunter, Michael Droettboom
Author-email: matplotlib-users@python.org
License: BSD
Location: /usr/local/lib/python3.7/site-packages
Requires: python-dateutil, cycler, kiwisolver, numpy, pyparsing
Required-by: ggplot
@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Mar 1, 2019

Also - I think Dockerfile is a great idea once we get everything figured out. If you could take that on, it would be great, since I'd have to read a lot of Dockermentation to do same.

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Mar 1, 2019

Ok. So the matplotlib version is off in the README. Can you verify the other version numbers here, and then I'll start on a Docker image.

@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Mar 1, 2019

Will do. Just a note: Ian did the README, so Matplotlib 1.15.1 may have worked for him. Probably worth moving to 3 anyway though.

@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Mar 1, 2019

Only differences I have are:
numpy 1.15.1
pandas 0.23.4

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Mar 1, 2019

Dockerfile and instructions added in docker branch.

We need to sort out some issues still, this will be the best place to do it so it just works... or well, works in a controlled Docker environment for everybody that can use it.

screenshot from 2019-03-01 11-55-29

@greebie

This comment has been minimized.

Copy link
Collaborator Author

greebie commented Mar 2, 2019

Locally, I hacked ggplot to fix the issue. I think we should just drop ggplot, as it will always be behind the scipy libraries. The notebook is not using it now.

@ianmilligan1

This comment has been minimized.

Copy link
Member

ianmilligan1 commented Mar 2, 2019

Let’s drop ggplot then if nothing is using it?

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Mar 2, 2019

ggplot removed in #12

@ruebot

This comment has been minimized.

Copy link
Member

ruebot commented Mar 3, 2019

Resolved with e880b87

@ruebot ruebot closed this Mar 3, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.