Pass igraph (per-event MLM graph) through Fortran bridge to SMATRIX_MULTI#1086
Pass igraph (per-event MLM graph) through Fortran bridge to SMATRIX_MULTI#1086
Conversation
Qubitol
left a comment
There was a problem hiding this comment.
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
igraphto a class member (to be consistent with the rest), and not just a variable passed around
|
No problem for squashing.
I have asked the agent to do that (he will upgrade the PR)
Let's wait for the final form and then tackle that |
|
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. |
|
Ok I have asked the agent to remove the spurious change. Do we squash commit now? |
6dce342 to
5dbd7bf
Compare
…uble sourcing This fix requires also a fix upstream to remove the previous vector.inc sourcing.
Uh oh!
There was an error while loading. Please reload this page.