search

NNPDF / pinefarm

Generate PineAPPL grids from PineCards
https://pinefarm.readthedocs.io
GNU General Public License v3.0
1 stars 0 forks source link

Hide away imports and info selection #67

Closed scarlehoff closed 5 months ago

scarlehoff commented 5 months ago

This PR hides away dependencies, imports and makes the selection of the tool not rely on all tools being available (in the sense of importable).

To be completely honest, I think we should still remove dependencies (or make more dependencies optional) since for heavy tools such as Matrix we potentially want to be able to run pinefarm just to "transpile" the pinecard into a matrix runcard, so ideally that part would run with python 3.6 even. But that would require too many changes, so for now it is >=3.9. I've added the >= 3.8 marker to nnpdf since it might be the only one stopping 3.8, but I'd say we want either to go all the way down to 3.6 or stay at 3.9. 3.7/3.8 are useless since I haven't seen any computers left in that middle ground*.

*the lcm is in 3.5 but that's too much

felixhekhorn commented 5 months ago

To be completely honest, I think we should still remove dependencies (or make more dependencies optional)

this could be done - but I'm not sure you want to do it here and I don't know how many we can actually strip ... the only obvious candidate is pygit2: used to install pineappl CAPI -> should be replaced by pre-compiled binaries and then dropped completely.

What else did you want to move?

scarlehoff commented 5 months ago

Nono, I don't want to do it here since it would require testing. It was just a thought (and at the moment we won't deal with the cluster running so you can always prepare the runcards in your personal computer and then submit it to a cluster)