Nyx emulator can not handle different number of emu_params

igmhub / LaCE

Emulator for the one-dimensional Lyman-alpha flux power spectrum. It was used to generate the results shown in https://arxiv.org/abs/2011.15127, https://arxiv.org/abs/2209.09895, and https://arxiv.org/abs/2305.19064.

1 stars 3 forks source link

Nyx emulator can not handle different number of emu_params #26

Closed andreufont closed 1 year ago

andreufont commented 1 year ago

I was hoping this to work: nyx_emu=nn_emulator.NNEmulator(training_set='Nyx23',emu_params=['Delta2_p', 'n_p','mF', 'sigT_Mpc', 'gamma']) but it crashes because in some internal linear algebra we are trying to multiply a matrix of size 6 and one of size 5. Probably because we have hardcoded somewhere that there should always be 6 emu_params?

Let's leave this for post-Durham (or even post-Summer), but good to document this type of issues.

andreufont commented 1 year ago

@lauracabayol - this was fixed, right? If so, we can close the issue

lauracabayol commented 1 year ago

yes!