Amedeo Chiefa

Results 27 comments of Amedeo Chiefa

Hi @scarlehoff, done. Now the branch history should be clean. I apologise for that - yesterday night I didn't pay much attention on the rebase procedure and I messed it...

My understanding from the last pheno meeting was to rebase on top of `master`, but I may be wrong. Should I rebase on top of `new_commondata_utils` ?

No, there isn't. How can I produce such a report?

I'm quite confused by `process_type.py` in the **`master`** branch. It seems there are two instantiations of the class `_Process` that look the same (`DY_NC_PT` in particular), at least to me....

Hi @scarlehoff, I've resolved the conflict and now all checks have passed. I just want to mention that I removed the jet data from `dataset_names.yaml` since we're not going to...

Ok, I've just removed it.

I think the PR is now complete. A few comments before merging: - I resolved the conflict with the master branch in `process_options`. Specifically: - There is now a single...

Hi @Roy, I've updated the code so that now the covmat for power corrections can be constructed as in the case of scale variations. Please, look and let me know...

> Don't forget to also delete the markdown file you're replacing. Yes, I was't sure whether I should've deleted them or not. I was waiting for your feedback, many thanks!

I'm currently working on this PR. I just want to mention that nested inline markups are currently not possible in reStructuredText, as explained [here](https://docutils.sourceforge.io/FAQ.html#is-nested-inline-markup-possible). There are some workarounds, but they...