Juan M. Cruz-Martinez

Results 164 comments of Juan M. Cruz-Martinez
trafficstars

> Versions imply a sort order. Here we want incompatible branches as you say below. I said version but I realise now I meant builds https://docs.conda.io/projects/conda-build/en/latest/resources/variants.html > See about mutex...

since we can install now directly with pip from the repository, running from a branch in e.g., a cluster is trivial

But actually, this is now closed by #2305 :)

We might want to add some scripting to automatize some of the benchmarking that the [escience](https://github.com/NNPDF/nnpdf/issues?q=is%3Aissue+is%3Aopen+label%3Aescience) people did.

I would honestly skip old the optionals (other than the reproducible weight initialisation maybe) and focus instead on polishing the stuff that's already there*. For instance, these problems with tf...

This is now finished, with only this item still to be merged (but finished otherwise) https://github.com/NNPDF/nnpdf/pull/1976

I'll try to review this tomorrow and leave it ready for you to have a second look and merge it in case you need to touch n3fit as discussed @RoyStegeman...

I'm going to merge this since the tests are passing and it is rebased on top of master (which means it is probably fixing something that has changed since the...

Because when they were separated it didn't really matter and it is decoupled from training / validation (the idea of diagonalising is to be able to do the split removing...

> I don't fully understand The chi2 (should not) depend of the diagonalization, since the total covmat is only used to report the total chi2, nobody cared about diagonalising that...