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

Write helper to convert repo and folder paths to an interact link #26

Closed
SamLau95 opened this issue Feb 11, 2016 · 8 comments
Closed

Write helper to convert repo and folder paths to an interact link #26

SamLau95 opened this issue Feb 11, 2016 · 8 comments
Assignees

Comments

@SamLau95
Copy link
Member

Basically, we'd like to put a little helper on http://data8.org/connector-instructors so that instructors can input their repo + path(s) and get back a link.

This will help avoid spelling errors and other common mistakes.

@SamLau95 SamLau95 self-assigned this Feb 11, 2016
@choldgraf
Copy link
Collaborator

Hey - I recently heard about Binder, and wanted to give it a shot. It's pretty cool! As a start, I wrote a short notebook that makes url conversion a bit easier (still clunky since you have to open a notebook, but that's not so bad since they'd need to click a URL anyway). All you should need to do is go here:

http://mybinder.org/repo/choldgraf/connector-instructors/ntbk/url_to_interact.ipynb

And follow the instructions. What do you think?

maybe @elaine84 would be interested in this as well, as on-the-fly notebooks are pretty nifty :)

@elaine84
Copy link
Collaborator

@choldgraf this is cool! I think it's great as an immediate thing that would be useful to instructors. You might want to try out some more examples, I think there's a bug that doesn't properly handle deeper file paths (e.g., a notebook in a folder in a repo such as https://github.com/data-8/smart-cities-connector/blob/gh-pages/labs_in_progress/Minilab3%20-%20Mapping%20travel%20times.ipynb).

Chris et al, are you also interested in looking into Binder some more? We might be able to learn something from its implementation. (Presently it looks like they can support 700 containers at a time; when I looked at their status they had 628/700 available.) We might want to open another issue about this.

@SamLau95 what do you think? For some added context, it's not obvious to me that we want to continue using these interact links as part of the long-term workflow we envision (or to make a less strong statement, I could imagine depending less on the interact links for content distribution, while also supporting them since they seem useful).

@SamLau95
Copy link
Member Author

Binder does look cool!

I agree that we probably want to have a different default file distribution
system.

This notebook looks useful for the time being.

On Thu, Feb 18, 2016 at 11:32 AM, elaine84 notifications@github.com wrote:

@choldgraf https://github.com/choldgraf this is cool! I think it's
great as an immediate thing that would be useful to instructors. You might
want to try out some more examples, I think there's a bug that doesn't
properly handle deeper file paths (e.g., a notebook in a folder in a repo
such as
https://github.com/data-8/smart-cities-connector/blob/gh-pages/labs_in_progress/Minilab3%20-%20Mapping%20travel%20times.ipynb).

Chris et al, are you also interested in looking into Binder some more? We
might be able to learn something from its implementation. (Presently it
looks like they can support 700 containers at a time; when I looked at
their status they had 628/700 available.) We might want to open another
issue about this.

@SamLau95 https://github.com/SamLau95 what do you think? For some added
context, it's not obvious to me that we want to continue using these
interact links as part of the long-term workflow we envision (or to make a
less strong statement, I could imagine depending less on the interact links
for content distribution, while also supporting them since they seem
useful).


Reply to this email directly or view it on GitHub
#26 (comment)
.

Sam Lau
EECS | UC Berkeley
Project Mentor | Blueprint, Technology for Nonprofits
samlau95@gmail.com samlau95@gmail.com

@ryanlovett
Copy link
Contributor

It'd be good to get Yuvi's feedback since binder uses kubernetes to spawn containers.

The 700 container limit is on mybinder.org's infrastructure. One can deploy the same setup to another set of nodes.

@elaine84
Copy link
Collaborator

Totally agreed, I sent him a link

@choldgraf
Copy link
Collaborator

thanks for the thoughts everybody - I agree this is a short-term solution, just something to help people right now and buy time until something more well-thought-out is made.

Regarding Binder, we could also reach out to Jeremy Freeman, as I believe that they've done a lot of the core work on Binder. Though maybe Yuvi is also doing the same?

@elaine84 thanks for finding that bug. I've updated the notebook to be more robust to deeper file structures etc, and I also added some "example" URLs at the end that can also serve as tests of a kind. :)

Binder should just take a minute or two to update, then check the updated repo:
http://mybinder.org/repo/choldgraf/connector-instructors/ntbk/url_to_interact.ipynb

@SamLau95, maybe we should just put a link to my Binder instance in the Readme somewhere and send it out as an email? I don't see a real reason to add this notebook to the official connector-instructors repo...

@SamLau95
Copy link
Member Author

Sending it via email sounds good to me.

On Thu, Feb 18, 2016 at 1:00 PM, Chris Holdgraf notifications@github.com
wrote:

thanks for the thoughts everybody - I agree this is a short-term solution,
just something to help people right now and buy time until something more
well-thought-out is made.

Regarding Binder, we could also reach out to Jeremy Freeman, as I believe
that they've done a lot of the core work on Binder. Though maybe Yuvi is
also doing the same?

@elaine84 https://github.com/elaine84 thanks for finding that bug. I've
updated the notebook to be more robust to deeper file structures etc, and I
also added some "example" URLs at the end that can also serve as tests of a
kind. :)

Binder should just take a minute or two to update, then check the updated
repo:

http://mybinder.org/repo/choldgraf/connector-instructors/ntbk/url_to_interact.ipynb

@SamLau95 https://github.com/SamLau95, maybe we should just put a link
to my Binder instance in the Readme somewhere and send it out as an email?
I don't see a real reason to add this notebook to the official
connector-instructors repo...


Reply to this email directly or view it on GitHub
#26 (comment)
.

Sam Lau
EECS | UC Berkeley
Project Mentor | Blueprint, Technology for Nonprofits
samlau95@gmail.com samlau95@gmail.com

@choldgraf
Copy link
Collaborator

there's a function to do this quickly in the connectortools module (though it may need to be updated for new course infrastructure) as well as a binder notebook to demo it, so I'm closing this. If we'd like a more interactive solution (e.g., a tiny javascript app) let's open the issue again if the need arises.

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

No branches or pull requests

4 participants