JuliaHEP / JuliaHEP-2023

Materials for the JuliaHEP 2023 Workshop
https://juliahep.github.io/JuliaHEP-2023/
Creative Commons Attribution 4.0 International
4 stars 4 forks source link

Get Binder to build for this repo #5

Closed Moelf closed 10 months ago

Moelf commented 11 months ago
Moelf commented 11 months ago

image

https://github.com/Moelf/BinderTest

here's a POC. Binder

Moelf commented 11 months ago

@graeme-a-stewart @aoanla what you guys think?

I got permission from ssl-hep people we can use their resource for our workshop. This way people can skip installing Julia if they want to.

ssl-hep link: https://binderhub.ssl-hep.org/v2/gh/Moelf/BinderTest/HEAD

image

and you can use CERN or University log in

graeme-a-stewart commented 11 months ago

Hi @Moelf - it's a great idea (not that Julia is hard to install), but anything that lowers the barrier to entry I like a lot.

Practically, will this work with the way we have the repo setup now, or do we need to move the training material down to the root of the repo?

Moelf commented 11 months ago

the only thing we need is Project.toml and Manifest.toml at the repo's root directory. Ideally, these should include everything any notebook in the repo that users will need.

But if you have some specific Project.toml, and if you include some using Pkg; magic as part of the notebook, that should also just work. They just have to install more stuff on the fly

graeme-a-stewart commented 11 months ago

I think we just move the Project.toml from julia-intro to the root. I just put it there in case we added other material - but if we do that we can just have the superset.

Adding Pkg stuff to the top of every notebook is a bit tedious and easy to forget (we have rather a lot of notebooks and I like that style, to keep each notebook focused on a specific topic).

Moelf commented 11 months ago

yeah, pin me once the repo is ready and I can test it and also walk you through so you can try it as well.

graeme-a-stewart commented 11 months ago

https://github.com/JuliaHEP/JuliaHEP-2023/pull/6/commits/0b24a7dbed5c5b1c6b400618095ae3eeae878e99

Just waiting for @aoanla to merge the additional content

tamasgal commented 11 months ago

Yeah, this is a really good idea.

I think we can live with a single Project.toml and Manifast.toml pair...

graeme-a-stewart commented 10 months ago

Hi - having merged #8 this is now good to test

Moelf commented 10 months ago

It looks like you can't access private github repo unless you control the Binderhub instance and supply it with an access token. So we need to become public in order to test that

graeme-a-stewart commented 10 months ago

@tamasgal - do you have an objection to making this repo public? I can't really see any reason to have it private and it's causing issues with binder (and also setting branch protections).