Skip to content

Title: docs: add physics callout box for Higgs search data distribution #127#167

Open
Riya-code2003 wants to merge 7 commits intohsf-training:gh-pagesfrom
Riya-code2003:gh-pages
Open

Title: docs: add physics callout box for Higgs search data distribution #127#167
Riya-code2003 wants to merge 7 commits intohsf-training:gh-pagesfrom
Riya-code2003:gh-pages

Conversation

@Riya-code2003
Copy link
Copy Markdown

This PR addresses issue #127 by adding a challenge/callout box to the Higgs search module (Episode 4).
​As an MSc Physics student, I have added a "Physics Tip" to clarify the distinction between experimental Data (ATLAS points) and Monte Carlo (MC) simulations in the invariant mass plot. This helps students understand the significance of the 125 \text{ GeV} Higgs signal peak.
​Changes
​Inserted {: .challenge} block before the Data vs. MC exercise.
​Used LaTeX formatting for units and particle decay notation.
1000162834

…istributioney6

Added a physics tip about the Higgs signal and clarified data vs. MC comparison.
@michmx
Copy link
Copy Markdown
Member

michmx commented Jan 26, 2026

Thanks @Riya-code2003 ! To clarify, this is not addressing #127 , as the issue is intended to explain the Python unpacking

fig, (ax_1, ax_2) = plt.subplots(1, 2)

However, it is a welcoming contrubition. It is true that for newcomes the logic of data/MC plots may not be evident.

So my suggestion is to address in the same PR the python unpacking fig, (ax_1, ax_2) = plt.subplots(1, 2). In other words, explain what that line of code is doing.

I am also adding @amorenobr and @ariostas in case they have other comments

… search callout

Added a new challenge section on Python unpacking and physics logic, clarifying the distinction between data and Monte Carlo simulations.
@Riya-code2003
Copy link
Copy Markdown
Author

Thanks @Riya-code2003 ! To clarify, this is not addressing #127 , as the issue is intended to explain the Python unpacking

fig, (ax_1, ax_2) = plt.subplots(1, 2)

However, it is a welcoming contrubition. It is true that for newcomes the logic of data/MC plots may not be evident.

So my suggestion is to address in the same PR the python unpacking fig, (ax_1, ax_2) = plt.subplots(1, 2). In other words, explain what that line of code is doing.

I am also adding @amorenobr and @ariostas in case they have other comments

Thanks @Riya-code2003 ! To clarify, this is not addressing #127 , as the issue is intended to explain the Python unpacking

fig, (ax_1, ax_2) = plt.subplots(1, 2)

However, it is a welcoming contrubition. It is true that for newcomes the logic of data/MC plots may not be evident.

So my suggestion is to address in the same PR the python unpacking fig, (ax_1, ax_2) = plt.subplots(1, 2). In other words, explain what that line of code is doing.

I am also adding @amorenobr and @ariostas in case they have other comments

Hi @michmx, thank you for the warm welcome and for clarifying the original intent of #127! I have updated the PR to include a section on Python tuple unpacking to explain the fig (ax_1, ax_2) syntax.

I also maintained the Physics Tip about the Higgs signal at 125Gev and Data vs. MC logic, as we agreed it's helpful for newcomers. I'm excited to contribute more to HSF training materials. Ready for another review whenever you have time!

Comment thread _episodes/04-higgs-search.md Outdated
Riya-code2003 and others added 2 commits January 29, 2026 05:41
Clarified explanations on Python unpacking and physics logic related to Higgs signal verification.
Copy link
Copy Markdown
Author

@Riya-code2003 Riya-code2003 left a comment

Choose a reason for hiding this comment

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

Thanks for the review, @ariostas! I have addressed your comments by:

  • Moving the challenge block line to 412 (closer to the relevant code)
  • Updating the Markdown formatting to use backticks for code snippets.
  • Adding the requested example for nested tuple unpacking.
    Let me know if there's anything else

{: .output}

![m4lep_histogram_1]({{ page.root }}/fig/m4lep_histogram_1.png)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's keep the empty line here to separate things.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done, I have restored the empty line after the image link to maintain proper spacing.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's still not empty. There's a >

Comment thread _episodes/04-higgs-search.md Outdated
Comment thread _episodes/04-higgs-search.md Outdated
Riya-code2003 and others added 3 commits February 5, 2026 05:46
Updated section titles and improved clarity in explanations regarding Python unpacking and physics logic.
Correct formatting and spacing in the Python unpacking section.
{: .output}

![m4lep_histogram_1]({{ page.root }}/fig/m4lep_histogram_1.png)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's still not empty. There's a >


![m4lep_histogram_5]({{ page.root }}/fig/m4lep_histogram_5.png)

>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's also remove the > here. And let's just leave one empty line.

~~~
{: .output}

And then make a plot, actually, let's make 2 plots, with matplotlib we can add sub-plots to the figure, then, we will be able to compare the MC distribution without and with weights.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think the comma should stay

@stale
Copy link
Copy Markdown

stale Bot commented Apr 7, 2026

This issue or pull request has been automatically marked as stale because it has not had recent activity. Please manually close it, if it is no longer relevant, or ask for help or support to help getting it unstuck. Let me bring this to the attention of @amorenobr @vlukashenko @richa2710 @michmx for now.

@stale stale Bot added the stale label Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants