search

NNPDF / reportengine

A framework for declarative data analysis
https://data.nnpdf.science/validphys-docs/guide.html
GNU General Public License v2.0
1 stars 2 forks source link

Version available in pypi doesn't work correctly in Mac OS #54

Closed scarlehoff closed 1 year ago

scarlehoff commented 1 year ago

I haven't pinpointed the problem, but the collection over experiments in NNPDF was not working correctly (and so I was getting a mismatch of arrays).

Installing (with pip install .) from this repository directly fixed the problem.

If I have time I'll try to debug it a bit more (as far as I can see no package was updated and the version of report engine is the same). For the time being I'm leaving this issue here as information for other users.

scarlehoff commented 1 year ago

Just got bitten in the ass again because of this. If we move to python only this needs to be fixed.

Zaharid commented 1 year ago

Don’t believe I had seen this issue. Is it just that the version is older? Should not have to do with mac or the like.

scarlehoff commented 1 year ago

No, the package version seemed to be the same when I opened the issue. As said originally, once I have time I will try to debug a bit more... if when I have time I haven't forgotten (again).

This is for the pip package, and with nnpdf things and dependencies not installed with conda but pip only. No libNNPDF or apfel, in case that makes a difference somewhere in the pipeline.

Zaharid commented 1 year ago

I pushed an updated package. Maybe that will fix things.

scarlehoff commented 1 year ago

Seems to work now, thanks!