Code review comment for lp:~maddevelopers/mg5amcnlo/v3_syntax_handler

Revision history for this message
Stefano Frixione (stefano-frixione) wrote :

Hi Rikkert,
thanks a lot for this; please find a number of minor comments below.

- General: do I understand correctly, that = and <= are identical?
   If so, I'd remove one of the two (preferably =).
- Second bullet: " strong), at the diagram level, respectively" -->
  "strong), respectively, at the diagram level".
- Fourth bullet, "requests" -> "imposes".
- Seventh bullet, "greatly adviced" -> "strongly advised"

For what concerns 1804.10017:
- First bullet, "this paper" -> "that paper"; "coupling orders need to
  be specified" -> "coupling orders had been understood"; "while, if
  interpreted...level" -> "while, if the same syntax used in 1804.10017
  were employed literally with the current version of the code, they
  would been applied at the amplitude level".
- Second bullet, "all the apperarance of" -> "all the instances of".

Cheers, Stefano.

On Sun, 18 Apr 2021, Rikkert Frederix wrote:

> I've created a small webpage, http://amcatnlo.cern.ch/co.htm , that describes the coupling order syntax and also "fixes" the automation of EW corrections paper. Please have a look.
>
> Best,
> Rikkert
>
> --
> https://code.launchpad.net/~maddevelopers/mg5amcnlo/v3_syntax_handler/+merge/400921
> Your team MadDevelopers is subscribed to branch lp:~maddevelopers/mg5amcnlo/v3_syntax_handler.
>

« Back to merge proposal