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

Update binder #162

Merged
merged 8 commits into from
Nov 12, 2020
Merged

Update binder #162

merged 8 commits into from
Nov 12, 2020

Conversation

malmans2
Copy link
Contributor

No description provided.

@malmans2
Copy link
Contributor Author

I've updated the binder repo. You can see a preview here: https://mybinder.org/v2/gh/hainegroup/oceanspy/binder?filepath=binder

Copy link
Collaborator

@ThomasHaine ThomasHaine left a comment

Choose a reason for hiding this comment

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

Looks great! I ran the demo presentation and tutorial notebook with no issues (apart from a few warnings). I couldn't find the demo presentation in readthedocs after a brief exploration. It's worth putting it in (e.g.) Quick Start.

@malmans2
Copy link
Contributor Author

Good idea! I've added the link in the home page and at the bottom of quick start. What do you think?

@ThomasHaine
Copy link
Collaborator

I don't see the presentation links.

@malmans2
Copy link
Contributor Author

Are you looking at the readthedocs website?
If yes, it won't show the additions until we merge (you could compile locally following this).
If you look at the readme of this PR (https://github.com/hainegroup/oceanspy/tree/bind), there is this link: www.bndr.it/nakt7

The binder link it's not showing the Live_Demo.ipynb yet, it will once we merge. (Once we merge www.bndr.it/nakt7 will display this https://mybinder.org/v2/gh/hainegroup/oceanspy/binder?filepath=binder)

@malmans2
Copy link
Contributor Author

Also, I'm fixing the tests in a separate PR!

@malmans2
Copy link
Contributor Author

I pasted the wrong link before. Here is the correct link of this branch: https://github.com/hainegroup/oceanspy/tree/binder

@ThomasHaine
Copy link
Collaborator

OK, understood. I was looking at the readthedocs website. I'm happy to merge now and see the updated docs when they're live.

@ThomasHaine
Copy link
Collaborator

I pasted the wrong link before. Here is the correct link of this branch: https://github.com/hainegroup/oceanspy/tree/binder

Looks good!

@malmans2
Copy link
Contributor Author

OK! If #163 passes all tests, I'll merge that one first, then this.

@malmans2 malmans2 merged commit 9c1283f into master Nov 12, 2020
@malmans2
Copy link
Contributor Author

All set!

@asiddi24
Copy link
Collaborator

asiddi24 commented Nov 12, 2020

Is it strange that when I run the binder it always runs on a jupyter notebook but not on jupyter lab ? Or is that just default in my system ?

@malmans2
Copy link
Contributor Author

I don't remember if that was intentional, but I'm pretty sure there is a way to set it. (Maybe here https://github.com/hainegroup/oceanspy/blob/master/binder/postBuild or something similar?).

I'm thinking that classic jupyter might be easier for people that are not familiar with it, but feel free to explore the lab option if you think it's better!

@malmans2 malmans2 deleted the binder branch March 2, 2021 15:58
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.

3 participants