Script that modifies MATRIX y_ttbar grids#98
Conversation
|
The pre-commit complains? |
| pass | ||
|
|
||
|
|
||
| class TestReverseBins: |
There was a problem hiding this comment.
Don't add it here - regardless if it will (already) become a proper pinefarm module or no. This lives just as well in the place where it belongs: tests/
There was a problem hiding this comment.
Moving the tests part into tests/ is indeed a good idea regardless. Right now I don't have any time at all but will do so as soon as I have some.
| required=True, | ||
| help="Name of the output Grid (w/o the extension `pineappl.lz4`).", | ||
| ) | ||
| def main(grid_path: str, output_name: str) -> None: |
There was a problem hiding this comment.
Do you want to test the CLI as well? since there are a series of commands
There was a problem hiding this comment.
Yes, that'd be added along with an additional test for modify_grids.
|
Hi @Radonirinaunimi, this works great! One comment, while using this script I noticed that the normalisation given by
gives and as you can see |
In order to perform fewer runs with MATRIX, predictions for$\pm y_{t \bar{t}}$ can be extracted from $|y_{t \bar{t}}|$ . The following script does so by performing the following operations:
TODO:
pinefarmmodule (?) - deprecates 3rd point