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

Setup notebook to run with au_notebook as a library; resolves #50 #52

Merged
merged 3 commits into from Apr 15, 2019

Conversation

Projects
None yet
3 participants
@ruebot
Copy link
Member

commented Apr 11, 2019

  • Update Dockerfile
  • Update README
  • Update notebook to use au_notebook library
  • Remove library

Test this out. If y'all are cool with it, I'll proceed on getting au_notebook setup on PyPi, and push a 0.0.1 release so we can use that.

We'll want to do this before #51 is merged.

Setup notebook to run with au_notebook as a library; resolves #50
- Update Dockerfile
- Update README
- Update notebook to use au_notebook library
- Remove library

@ruebot ruebot requested review from greebie and ianmilligan1 Apr 11, 2019

@ianmilligan1
Copy link
Member

left a comment

Tested both locally + with Docker, all checks out well.

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Apr 11, 2019

Here's the repo on the test PyPi: https://test.pypi.org/project/au-notebook/

@greebie
Copy link
Collaborator

left a comment

Works fine for me!

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Apr 12, 2019

Cool. I'll cut a 0.0.1 release of au_notebook, and update this PR.

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Apr 12, 2019

@greebie

This comment has been minimized.

Copy link
Collaborator

commented Apr 12, 2019

Reminder that you may have to uninstall au_notebook first if you pip installed the test version (I was wondering how I ended up with a 0.0.2 version).

@ruebot ruebot marked this pull request as ready for review Apr 12, 2019

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Apr 12, 2019

Sorry, had to push another release to PyPI. Missed a bad typo that cause the library note to fully load 🤦‍♂️

It all should be good to go now.

@ruebot

This comment has been minimized.

Copy link
Member Author

commented Apr 12, 2019

The original commit message should be good when y'all squash and merge. If you have any questions, let me know.

Screenshot from 2019-04-12 14-29-04

@greebie

This comment has been minimized.

Copy link
Collaborator

commented Apr 12, 2019

Oh thank goodness. I was going crazy trying to figure out why I couldn't get it to import! :) Works great now.

@ianmilligan1 ianmilligan1 merged commit e5f78a4 into master Apr 15, 2019

1 check passed

ci/dockercloud Your tests passed in Docker Cloud
Details

@ianmilligan1 ianmilligan1 deleted the au_notebook-library branch Apr 15, 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.