Include nyx_version flag in bookkeeper

[jchavesmontero]

Include this flag for consistency with future versions of the Nyx file. Could you first run src/lace/scripts/compute_nyx_emu_cosmo.py and then Tutorial_bookkeeping.ipynb?

If it works, I will update the readme as well

[andreufont]

Could you take care of this

[andreufont]

Sorry, wrong button...

I noticed that this is a "draft" pull request?

[jchavesmontero]

Yes, I made a draft because I wanted to check that it worked. I am assuming it does, so before any push, I wanted to update the readme. Let me do it and I will turn it into a real PR

[andreufont]

@lauracabayol - could you take care of the review? Otherwise I'll do it on Monday

[jchavesmontero]

I have already updated the readme, waiting for your review

[andreufont]

@lauracabayol this one

[lauracabayol]

It is raising an error related with the NYX_PATH: (DESIenv6) [lcabayol@tdm003 LaCE]$ python src/lace/scripts/compute_nyx_emu_cosmo.py Traceback (most recent call last): File "/nfs/pic.es/user/l/lcabayol/DESI/LaCE/src/lace/scripts/compute_nyx_emu_cosmo.py", line 72, in main() File "/nfs/pic.es/user/l/lcabayol/DESI/LaCE/src/lace/scripts/compute_nyx_emu_cosmo.py", line 16, in main nyx_fname = os.environ["NYX_PATH"] + "modelsNyx" + nyx_version + ".hdf5" File "/data/astro/scratch/lcabayol/anaconda3/envs/DESIenv6/lib/python3.10/os.py", line 680, in getitem raise KeyError(key) from None KeyError: 'NYX_PATH'

[lauracabayol]

Everything working smoothly after addting the Nyx_path as environment variable

[jchavesmontero]

Thanks a lot @lauracabayol!