Skip to content

Pangeo Binder for Parcels#634

Merged
delandmeterp merged 14 commits intomasterfrom
binder_branch
Sep 6, 2019
Merged

Pangeo Binder for Parcels#634
delandmeterp merged 14 commits intomasterfrom
binder_branch

Conversation

@delandmeterp
Copy link
Copy Markdown
Contributor

@delandmeterp delandmeterp commented Sep 4, 2019

This PR provides the yml file that Binder needs to build Parcels,
and a direct link to go the Pangeo Binder

@delandmeterp
Copy link
Copy Markdown
Contributor Author

@willirath, I'm not sure to have fully understood the Binder idea. Since it goes with all our source codes, we should provide the yml for Parcels dependencies or for full Parcels?

In xgcm, they provide a yml for the full model, but then source files are not useful...

If we only yml the dependencies, then user needs to run python setup.py install.
Is there some "better" practice?

And of course the example data still need to be downloaded. I haven't found I can write a small setup script in the binder directory (for installing and downloading)

@willirath
Copy link
Copy Markdown
Contributor

I think postBuild is what you want. At least, this is what I've used in my test repo for providing a parcels-examples binder: https://github.com/willirath/parcels_examples_binder

Instead of integrating the binder functionality here, it might be advisable to have a separate repo that only has what is necessary and uses the conda-forge package. As Binder rebuilds the Docker image that the users are working in every time there's a new commit in master, the separate repo pointing to a stable and released version conda package should result in a shorter startup time and less traffic for the binder infrastructure.

@willirath
Copy link
Copy Markdown
Contributor

@delandmeterp @erikvansebille I gave you write permissions in the binder test-repo I created a while ago.

@delandmeterp
Copy link
Copy Markdown
Contributor Author

Thanks! Yes Indeed, so we should have on parcels only the link to the binder with the examples.

@willirath
Copy link
Copy Markdown
Contributor

In willirath/parcels_examples_binder#2 I've bumped to the latest parcels release and updated the way Binder launches (uses Juptyerlab and starts at the parcels_tutorial.ipynb). If this is working, I'd suggest

@willirath
Copy link
Copy Markdown
Contributor

willirath commented Sep 4, 2019

Release procedure would also need to be updated to contain

  • bump the parcels version in the environment file of the binder repo to the latest release on conda-forge.

(Note that this step is necessary because Binder only can be forced to rebuild after the binder-repo itself was changed. Changed dependencies (i.e. parcels on conda-forge being updated) do not trigger updates of what the user sees on binder.)

Copy link
Copy Markdown
Contributor

@willirath willirath left a comment

Choose a reason for hiding this comment

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

Ah. I think it's not necessary to use pangeo binder here. Just go for the link in the parcels_examples_binder repo that uses mybinder.org.

@erikvansebille
Copy link
Copy Markdown
Member

This binder is really awesome! Should we also add binder links on all the tutorials at http://oceanparcels.org/#tutorials?

@willirath
Copy link
Copy Markdown
Contributor

This binder is really awesome! Should we also add binder links on all the tutorials at http://oceanparcels.org/#tutorials?

Yes, I'd go for this (have a launch all on binder button at the top and deep links on all tutorials):
tutorial_binder_deeplinks

@willirath
Copy link
Copy Markdown
Contributor

Yes, I'd go for this (have a launch all on binder button at the top and deep links on all tutorials):

Wait, I thought deep links into the same repo would end up on the same instance. This does not seem to be the case. So I'd recommend just adding the launch button / link at the top for now.

@willirath
Copy link
Copy Markdown
Contributor

@delandmeterp delandmeterp merged commit 479508b into master Sep 6, 2019
@delandmeterp delandmeterp deleted the binder_branch branch September 6, 2019 13:28
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