Skip to content

feat(gsoc): Negative Weights #1820

Merged
maszyman merged 12 commits intoHSF:mainfrom
Saptaparna:main
Feb 10, 2026
Merged

feat(gsoc): Negative Weights #1820
maszyman merged 12 commits intoHSF:mainfrom
Saptaparna:main

Conversation

@Saptaparna
Copy link
Contributor

PR for two projects on negative weight mitigation.

@netlify
Copy link

netlify bot commented Jan 30, 2026

Deploy Preview for earnest-hotteok-b1e1bf ready!

Name Link
🔨 Latest commit 92e0a77
🔍 Latest deploy log https://app.netlify.com/projects/earnest-hotteok-b1e1bf/deploys/698b367db26ce400082dbcfb
😎 Deploy Preview https://deploy-preview-1820--earnest-hotteok-b1e1bf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@Saptaparna
Copy link
Contributor Author

@graeme-a-stewart @eduardo-rodrigues Tagging admins

@Saptaparna Saptaparna changed the title Negative Weights feat(gsoc): Negative Weights Jan 30, 2026
@Saptaparna
Copy link
Contributor Author

@simonspa

@Saptaparna Saptaparna marked this pull request as draft January 30, 2026 01:50
@Saptaparna Saptaparna changed the title feat(gsoc): Negative Weights feat(gsoc): Negative Weights label:"GSoC" Jan 30, 2026
@Saptaparna Saptaparna changed the title feat(gsoc): Negative Weights label:"GSoC" feat(gsoc): Negative Weights Jan 30, 2026
@Saptaparna
Copy link
Contributor Author

label:"GSoC"

@Saptaparna Saptaparna marked this pull request as ready for review January 30, 2026 01:54
@maszyman maszyman added the GSoC Related to Google Summer of Code activity label Jan 30, 2026
@maszyman maszyman requested a review from a team January 30, 2026 09:24
Copy link
Contributor

@maszyman maszyman 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 proposals.

Please have a look at my comments inline.

Also, please consider adding AI policy, for example as suggested in #1804 (comment)

Comment on lines 1 to 14
Copy link
Contributor

Choose a reason for hiding this comment

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

This does not seem to be needed. Please have a loot at template in e.g. #1804

Copy link
Contributor

Choose a reason for hiding this comment

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

This should correspond to the project that needs to be defined in _gsocproject/2026/. We already have there ATLAS is that what this proposal targets.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
project: Negative weight, ATLAS event generator workflows
project: ATLAS

Copy link
Contributor

Choose a reason for hiding this comment

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

This name should correspond to the definition in _gsocorgs/2026. Please check the examples, but you may need to create smu.md in there.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
organization: Southern Methodist University
organization: SMU

Copy link
Contributor

Choose a reason for hiding this comment

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

Please make those arXiv references the clickable links.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
* Familiarity with python and C++
* Familiarity with Python and C++

* [Saptaparna Bhattacharya](mailto:saptaparna.bhattacharya@cern.ch)

## Links
* [Cell resampling](https://cres.hepforge.org/)
Copy link
Contributor

Choose a reason for hiding this comment

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

This looks to be a Rust project. Would it make sense to add it in requirements?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

Copy link
Contributor

Choose a reason for hiding this comment

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

Same editorial comments as for the previous proposal.


## Expected results
Release a modified version of MCFM that successfully incorporates the negative weight mitigation strategy, demonstrating reduced statistical dilution.

Copy link
Contributor

@maszyman maszyman Jan 30, 2026

Choose a reason for hiding this comment

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

You are missing Mentors section (should be done via project_mentors in the metadata at the top).

If you are the only mentor also for this project, please consider adding a co-mentor as Google suggests that mentors should mentor 1 contributor project, and there should be preferably 2 mentors per project.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done.

@maszyman
Copy link
Contributor

maszyman commented Feb 2, 2026

@Saptaparna could you please update the PR following the comments?

@maszyman
Copy link
Contributor

maszyman commented Feb 4, 2026

@Saptaparna just a kind reminder to update this PR.

@Saptaparna
Copy link
Contributor Author

@maszyman done!

Copy link
Contributor

@maszyman maszyman 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 updates. There are still a couple changes needed to properly render the project.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
organization: Southern Methodist University
organization: SMU

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
project: Negative weight, ATLAS event generator workflows
project: ATLAS

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
-
---

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
project_mentors: Saptaparna Bhattacharya (saptaparnab@smu.edu) and authors of the "cell-resampling" team (Jeppe Andersen(mailto:eppe.andersen@durham.ac.uk) and Andreas Maier(mailto:andreas.martin.maier@desy.de))
project_mentors:
- email: saptaparnab@smu.edu
first_name: Saptaparna
last_name: Bhattacharya
organization: SMU
is_preferred_contact: yes

and similarly for the co-mentors

Comment on lines 36 to 37
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Mentors
* [Saptaparna Bhattacharya](mailto:saptaparna.bhattacharya@cern.ch) and authors of the "cell-resampling" team (Jeppe Andersen(mailto:eppe.andersen@durham.ac.uk) and Andreas Maier(mailto:andreas.martin.maier@desy.de))

This is handled by project_mentors metadata

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
project: Negative weight, MCFM
project: MCFM

and please add MCFM.md to _gsocprojects/2026/

Copy link
Contributor

Choose a reason for hiding this comment

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

same comment as previous proposal

@Saptaparna
Copy link
Contributor Author

AI policy is now there too.

@maszyman
Copy link
Contributor

Thanks @Saptaparna , I've made few adjustments, but I think now it's ready.

@maszyman maszyman merged commit 807d277 into HSF:main Feb 10, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

GSoC Related to Google Summer of Code activity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments