LSSTDESC / rail_pzflow

Rail algorithms that depend on PZFlow
MIT License
0 stars 0 forks source link

Update pzflow to pzflow_nf in import #14

Closed OliviaLynn closed 2 months ago

OliviaLynn commented 2 months ago

In my local environment, this fixed pzflow-related import error when running rail.stages.import_and_attach_all()

Will hopefully address a similar pzflow-related error when running Useful_Utilities.ipynb in the rail smoke test

OliviaLynn commented 2 months ago

The KeyError: 'aliases' is something I've seen elsewhere in RAIL today, in relation to the new ceci v2 release. I am assuming that is what is causing the unit test failures here.

torluca commented 2 months ago

Qianjun and I will merge this pull request. The aliases problem is because of ceci v2 as Olivia already pointed out. This is an issue that is presenting itself in different packages and will be fixed for each and every one since ceci v2 is still in development.