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

Binder now supports Dataverse DOIs #4714 #6453

Merged
merged 6 commits into from
Dec 16, 2019
Merged

Binder now supports Dataverse DOIs #4714 #6453

merged 6 commits into from
Dec 16, 2019

Conversation

pdurbin
Copy link
Member

@pdurbin pdurbin commented Dec 11, 2019

What this PR does / why we need it:

This pull requests helps ensure that installations of Dataverse are aware that they can now use https://mybinder.org to spin up Jupyter Notebooks based on DOIs of datasets in Dataverse as described in my write up linked from https://twitter.com/philipdurbin/status/1204472858413371392

Which issue(s) this PR relates to:

Relates to #4714

Special notes for your reviewer:

At this time we cannot create an external tool for Binder in Dataverse because we can only pass DOIs as query parameters and Binder requires DOIs to be in paths.

Suggestions on how to test this:

From https://groups.google.com/d/msg/dataverse-community/K7vKWSPLaQY/8go3fvV3AwAJ

To try it out, go to https://mybinder.org and click the dropdown to select "Dataverse DOI". Then type in the DOI of a dataset hosted in an installation of Dataverse and click "Launch" (screenshot 1, attached).

step1-enter-doi

After a while, an instance of Jupyter Lab will appear with the files from the Dataverse dataset (screenshot 2).

step2-jupyter-lab

From Jupyter Lab you can either open an existing Jupyter Notebook that is one of the files in the Dataverse dataset or you can create a new Jupyter Notebook. (No screenshot. Hmm! Time to upload a Jupyter notebook to the dataset!)

To return to the dataset landing page in Dataverse from Jupyter Lab, click "visit repo" (screenshot 3).

step3-return-to-dataverse

Does this PR introduce a user interface change?:

No.

Is there a release notes update needed for this change?:

Yes, suggestion added.

Additional documentation:

Dataverse is now listed at https://binderhub.readthedocs.io/en/latest/developer/repoproviders.html#supported-repoproviders

@coveralls
Copy link

coveralls commented Dec 11, 2019

Coverage Status

Coverage remained the same at 19.514% when pulling 5ece2a2 on 4714-binder into 709162d on develop.

Copy link
Contributor

@djbrooke djbrooke left a comment

Choose a reason for hiding this comment

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

Discussed with @pdurbin and made some edits. On to QA!

@djbrooke djbrooke removed their assignment Dec 13, 2019
@kcondon kcondon self-assigned this Dec 16, 2019
@kcondon kcondon merged commit 373d98f into develop Dec 16, 2019
@kcondon kcondon deleted the 4714-binder branch December 16, 2019 16:09
@djbrooke djbrooke added this to the 4.19 milestone Dec 16, 2019
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.

5 participants