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

add/update tutorials, advancedPython, inference notebook #23

Merged
merged 40 commits into from
Jun 3, 2021
Merged

add/update tutorials, advancedPython, inference notebook #23

merged 40 commits into from
Jun 3, 2021

Conversation

jonas-eschle
Copy link
Collaborator

@jonas-eschle jonas-eschle commented Nov 2, 2020

Switch to uproot4
add advanced Python tutorial with context manager, decorator and (un)packing
add inference (sceleton) to talk about model fitting and inference

@chrisburr
Copy link
Member

@jonas-eschle jonas-eschle changed the title add/update tutorials, advancedPython, inference notebook WIP: add/update tutorials, advancedPython, inference notebook Nov 3, 2020
@jonas-eschle
Copy link
Collaborator Author

Hm, OSError: remote server does not support HTTP range requests seems to happen with uproot4 , opened an issue

@jonas-eschle
Copy link
Collaborator Author

This is finished except of the mentioned bug in uproot4.

Added also a small Scikit-HEP tutorial with particles, hepunits, formulate
Added the sPlot calculation with hepstats
Added the advanced Python with exceptions, context manager, etc
Updated README with an overview
and extended some of the tutorials slightly

@jonas-eschle jonas-eschle changed the title WIP: add/update tutorials, advancedPython, inference notebook add/update tutorials, advancedPython, inference notebook Nov 25, 2020
@jonas-eschle
Copy link
Collaborator Author

This fails currently due to uproot4 new version not yet released. As soon as >0.1.2 is out, this should work.

@chrisburr chrisburr closed this Jan 8, 2021
@chrisburr chrisburr reopened this Jan 8, 2021
@jonas-eschle
Copy link
Collaborator Author

jonas-eschle commented Mar 19, 2021

Needs hsf-training/python-lesson#1 to get merged

@jonas-eschle
Copy link
Collaborator Author

Fixed all typos and everything, ready to be merged

@jonas-eschle
Copy link
Collaborator Author

@chrisburr can we merge this maybe?

@jonas-eschle jonas-eschle mentioned this pull request Jun 1, 2021
@jonas-eschle
Copy link
Collaborator Author

@eduardo-rodrigues can this maybe be merged? And if there is a lack of maintainers to keep it up to date, I would volunteer to become one

@chrisburr
Copy link
Member

Sorry for not getting to this. You should have an invite with merge rights for this repository.

One comment: Can you remove the output from the notebooks to avoid inflating the repository? They should be added by the CI for the actual webpage.

@eduardo-rodrigues
Copy link
Member

(OK, @chrisburr already replied / did it all :-).)

@jonas-eschle
Copy link
Collaborator Author

Many thanks, sorry for double pinging, and yes, I'll remove the output

@jonas-eschle
Copy link
Collaborator Author

Removed the outputs, all working now, merging

@jonas-eschle jonas-eschle merged commit 400b0f7 into hsf-training:master Jun 3, 2021
@jonas-eschle jonas-eschle deleted the je_advanced_python branch June 3, 2021 13:02
@chrisburr
Copy link
Member

Thanks @mayou36! 🍰

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