Comment 1 for bug 1642338

Dear Philipp,

I’m actually not able to reproduce your bug.
This is actually deeper than that, since I do not have any event with that information written.

> <rwgt>
> <wgt id='bias'> 0.1000000E+01</wgt>
> </rwgt>

Can you give me more details on how you are performing your event generation, such that I can try to reproduce your problem.
(I did not check if the above happens for gridpack for example)

> The XML parser in SysCalc 1.1.6 cannot handle this new layout when the
> rwgt block comes before the mgrwt one. The FirstChildElement function of
> the parser should be able to find the <mgrwt> block when its after the
> <rwgt> block, but fails in practice.

I will assign this to the main SysCalc author. I have actually no idea if that code is still maintained or not.

> Aside from this issue, could you comment on whether SysCalc is actually
> needed to do the PDF reweighing, or can this be done in the main code
> now? If yes, is there some documentation to port the syscalc card to the
> new format?

So yes, we have a python method which can do the same.
It requires that lhapdf6 is installed and that it can be linked to python (so it requires some python library)
Here is the documentation: https://cp3.irmp.ucl.ac.be/projects/madgraph/wiki/Systematics
The only parameter of SysCalc which is not supported in this new module is the scalecorrelation option.
(you can only do the full 9 scale variation or vary them one at the time.)

Cheers,

Olivier

> On Nov 16, 2016, at 18:19, Philipp Pigard <email address hidden> wrote:
>
> Public bug reported:
>
> Dear Authors,
>
> The 2.5.1 update included the feature to bias the event generation. Even when deactivated, this causes a new block to be added to the LHE:
> <rwgt>
> <wgt id='bias'> 0.1000000E+01</wgt>
> </rwgt>
>
> The XML parser in SysCalc 1.1.6 cannot handle this new layout when the
> rwgt block comes before the mgrwt one. The FirstChildElement function of
> the parser should be able to find the <mgrwt> block when its after the
> <rwgt> block, but fails in practice.
>
> Aside from this issue, could you comment on whether SysCalc is actually
> needed to do the PDF reweighing, or can this be done in the main code
> now? If yes, is there some documentation to port the syscalc card to the
> new format?
>
> Cheers
> Philipp
>
> ** Affects: mg5amcnlo
> Importance: Undecided
> Status: New
>
> --
> You received this bug notification because you are subscribed to
> MadGraph5_aMC@NLO.
> https://bugs.launchpad.net/bugs/1642338
>
> Title:
> SysCalc fails to parse XML in 2.5.1
>
> Status in MadGraph5_aMC@NLO:
> New
>
> Bug description:
> Dear Authors,
>
> The 2.5.1 update included the feature to bias the event generation. Even when deactivated, this causes a new block to be added to the LHE:
> <rwgt>
> <wgt id='bias'> 0.1000000E+01</wgt>
> </rwgt>
>
> The XML parser in SysCalc 1.1.6 cannot handle this new layout when the
> rwgt block comes before the mgrwt one. The FirstChildElement function
> of the parser should be able to find the <mgrwt> block when its after
> the <rwgt> block, but fails in practice.
>
> Aside from this issue, could you comment on whether SysCalc is
> actually needed to do the PDF reweighing, or can this be done in the
> main code now? If yes, is there some documentation to port the syscalc
> card to the new format?
>
> Cheers
> Philipp
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/mg5amcnlo/+bug/1642338/+subscriptions