Skip to content

Pass igraph (per-event MLM graph) through Fortran bridge to SMATRIX_MULTI#1086

Open
Copilot wants to merge 9 commits intomasterfrom
copilot/pass-igraph1-to-fortran-bridge
Open

Pass igraph (per-event MLM graph) through Fortran bridge to SMATRIX_MULTI#1086
Copilot wants to merge 9 commits intomasterfrom
copilot/pass-igraph1-to-fortran-bridge

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 12, 2026

  • Revert all non-CODEGEN files (ee_mumu, gg_tt, gg_ttg, etc. directories) to base commit state, keeping only CODEGEN template changes

Copy link
Copy Markdown
Collaborator

@Qubitol Qubitol left a comment

Choose a reason for hiding this comment

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

So, I have 2 things which I'd change:

  • I would squash/rebase the commits, so that we have only one single commit making the changes inside CODEGEN (also, it seems it didn't remove all the changes done elsewhere, so I'd remove them). I'm saying that because for MadGraph7 I'd like to cherry-pick the commit, and having unrelated things makes it more annoying.
  • we can do a second commit that updates the processes (but in that case we use the script to properly regenerate all of them)
  • I would upgrade igraph to a class member (to be consistent with the rest), and not just a variable passed around

@oliviermattelaer
Copy link
Copy Markdown
Member

No problem for squashing.

I would upgrade igraph to a class member (to be consistent with the rest), and not just a variable passed around

I have asked the agent to do that (he will upgrade the PR)

we can do a second commit that updates the processes (but in that case we use the script to properly regenerate all of them)

Let's wait for the final form and then tackle that

@Qubitol
Copy link
Copy Markdown
Collaborator

Qubitol commented Apr 13, 2026

I think the core is done, but still there are changes in the various processes. I guess we can solve this by hand via interactive rebase.

@oliviermattelaer
Copy link
Copy Markdown
Member

Ok I have asked the agent to remove the spurious change. Do we squash commit now?
or run the update script first? (or just squash now and then run the update script?)

@Qubitol Qubitol force-pushed the copilot/pass-igraph1-to-fortran-bridge branch from 6dce342 to 5dbd7bf Compare April 14, 2026 10:12
@Qubitol Qubitol self-requested a review April 14, 2026 16:55
@Qubitol Qubitol marked this pull request as ready for review April 14, 2026 16:55
@Qubitol Qubitol changed the title Pass igraph1 (per-event MLM graph) through Fortran bridge to SMATRIX_MULTI Pass igraph (per-event MLM graph) through Fortran bridge to SMATRIX_MULTI Apr 14, 2026
…uble sourcing

This fix requires also a fix upstream to remove the previous vector.inc
sourcing.
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