Skip to content

2025 update#42

Merged
AntonReinhard merged 3 commits into
mainfrom
2025-update
Jul 26, 2025
Merged

2025 update#42
AntonReinhard merged 3 commits into
mainfrom
2025-update

Conversation

@AntonReinhard
Copy link
Copy Markdown
Collaborator

  • Rename the repository so it's not referencing the year anymore (see Rename JuliaHEP-2023 repo to Julia-intro #37)
  • Rename all the links too
  • Remove Manifest.toml
  • Some further minor changes, rephrasings and additions
  • Rewrite install instructions so juliaup is the main and recommended way

Some further minor changes, rephrasings and additions
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 24, 2025

PR Preview Action v1.6.2

🚀 View preview at
https://JuliaHEP.github.io/Hands-on-Julia-for-particle-physicists/pr-preview/pr-42/

Built to branch gh-pages at 2025-07-25 14:22 UTC.
Preview will be ready when the GitHub Pages deployment is complete.

@AntonReinhard AntonReinhard marked this pull request as ready for review July 24, 2025 13:11
@AntonReinhard AntonReinhard requested a review from szabo137 July 24, 2025 13:11
@AntonReinhard
Copy link
Copy Markdown
Collaborator Author

It looks like the Pkg using statements get an Interrupt for some reason and print lots of stack traces, but then the rest works anyway?
For example here: https://juliahep.github.io/Hands-on-Julia-for-particle-physicists/pr-preview/pr-42/10-julia-intro-plots.html
Does anyone have an idea why?

@Moelf
Copy link
Copy Markdown
Member

Moelf commented Jul 24, 2025

that's weird.... at some point maybe we should write a new plotting section with Makie

Copy link
Copy Markdown
Member

@graeme-a-stewart graeme-a-stewart left a comment

Choose a reason for hiding this comment

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

Hi Anton - without being able to preview it (my local micromamba environment seems to have completely died and will take a bit of effort to fix), your changes look good to me. Small point about the copyright years. Thanks!

Comment thread README.md Outdated
@AntonReinhard
Copy link
Copy Markdown
Collaborator Author

AntonReinhard commented Jul 25, 2025

without being able to preview it

Does the online preview not work for you?

@graeme-a-stewart
Copy link
Copy Markdown
Member

that's weird.... at some point maybe we should write a new plotting section with Makie

There is a bit of Makie now and I would be in favour of promoting the to the primary plotting solution we tell people about (I'm a convert!)

I have no idea about the jupyterbook failure in CI and my local install is broken ATM 😞

@graeme-a-stewart
Copy link
Copy Markdown
Member

without being able to preview it

Does the online preview not work for you?

It does - with the caveat about the broken plots, which I don't understand...

@AntonReinhard
Copy link
Copy Markdown
Collaborator Author

that's weird.... at some point maybe we should write a new plotting section with Makie

There is a bit of Makie now and I would be in favour of promoting the to the primary plotting solution we tell people about (I'm a convert!)

I have no idea about the jupyterbook failure in CI and my local install is broken ATM 😞

I also use Makie mostly, but I will likely not get to plotting in the live tutorial anyway, so I didn't look at it too closely here

@AntonReinhard AntonReinhard merged commit c6bd54a into main Jul 26, 2025
2 checks passed
@AntonReinhard AntonReinhard deleted the 2025-update branch July 26, 2025 18:11
@Moelf
Copy link
Copy Markdown
Member

Moelf commented Jul 26, 2025

image

I guess it's real

@AntonReinhard
Copy link
Copy Markdown
Collaborator Author

Hmm, the deploy-book logs say some stuff like

KERNEL EXCEPTION/home/runner/work/Hands-on-Julia-for-particle-physicists/Hands-on-Julia-for-particle-physicists/julia-intro/docs/08-julia-intro-broadcasting.ipynb: WARNING: Executing notebook failed: CellTimeoutError [mystnb.exec]
/home/runner/work/Hands-on-Julia-for-particle-physicists/Hands-on-Julia-for-particle-physicists/julia-intro/docs/08-julia-intro-broadcasting.ipynb: WARNING: Notebook exception traceback saved in: /home/runner/work/Hands-on-Julia-for-particle-physicists/Hands-on-Julia-for-particle-physicists/julia-intro/docs/_build/html/reports/08-julia-intro-broadcasting.err.log [mystnb.exec]
/home/runner/work/Hands-on-Julia-for-particle-physicists/Hands-on-Julia-for-particle-physicists/julia-intro/docs/10-julia-intro-plots.ipynb: WARNING: Executing notebook failed: CellTimeoutError [mystnb.exec]

Maybe some timeout is set too aggressively?

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