Conversation
|
Given that we need to download also the grids for single-inclusive jets, I can take advantage of this PR and add the relative scripts for the single-jet cases. I'll start with NNPDF/nnpdf#2407. |
|
Thank you very much for this. I have just a two (global) comments: RE the download itself. Would it be possible to separate it from the per-grid script? In particular I am thinking you could add the list of links in something like a in every script. RE the conversion: I'm ok with this part. However, please remove the Btw, in particular I'm very much against semi-inclusive checks like this: It is practically impossible to be fully inclusive in possible conda installations so it is better not to even try. This might even be activating the wrong conda installation creating chaos in the target computer! |
|
Hi @scarlehoff, thanks for your comment. Indeed we discussed this yesterday during the code meeting. I agree that the conda part is horrible to say the least, but this was something that I wrote in my bash script that was meant to be temporarly and local. So I agree with you that the conda part must be removed alltogether.
I see your point. However, I don't want to fall in the rabbit hole and build complex abstractions for such simple problems. In the end, this is meant to be a simple script that downloads and converts the grids, with some renaming conventions which must be set case-by-case. @andrpie and I will look into this, but at the moment this doesn't have my highest priority.
RE this I just want to ask for clarifications. It's not clear to me how these scripts will be run. Are they meant to be run individually by hand? or they enter an automatised workflow that runs all the scripts for which grids should be "produced". Honestly I haven't thought about this when I wrote the script because this wasn't clear to me. The goal was just having something where I could log the steps so that I didn't forget. |
I hate complex abstractions, I'm happy if you do a simple one :_)
In the ideal world I was thinking the following: I download then pinefarm will read the Then it will run the Then, after the download has finished it automatically runs the After that the So this is why I'd like to separate download and re-organization. The This is just the picture I had in mind. The leading order thing for me is not to repeat the |
Addresses pinefarm#102, i.e. adds a script that downloads and converts the full color grids from ploughshare for each of the 5 dijet datasets.
The scripts work well on my device (MacOS), but of course haven't been tested otherwise.
Thank you @achiefa for the CMS 13TEV script!