Starfish-develop / Starfish

Tools for Flexible Spectroscopic Inference
https://starfish.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
68 stars 22 forks source link

Priors Dictionary Issue #142

Closed KLucero09 closed 3 years ago

KLucero09 commented 3 years ago

Hello I have recently been working through the Starfish single-order Spectrum example and have been able to get every step to work except for one: training the priors. I have tried rewriting the dictionary of priors as an array, explicitly stating dtype=object, directly copying and pasting the text from the documentation straight into my script, and many other methods. All of these have errored out, with the direct copy and paste returning "ValueError: setting an array element with a sequence." I am not sure how to work around this error so that I can properly train these priors to the data. It is worth noting that everything else following this step still technically runs, but the priors just are not fine-tuned. Help in explaining what could be going wrong would be great, and please let me know if more information is needed.

mileslucas commented 3 years ago

Hi, thanks for reaching out. Can you please provide a snippet of code that reproduces the issue, as well as noting the version of python and starfish you are using. You can get the Starfish version from Starfish.__version__.

KLucero09 commented 3 years ago

Python version: 3.7.10 Starfish version: 0.3.2 image image

mileslucas commented 3 years ago

This is fixed in version 0.4.0 of Starfish, please update your local version and it should work now.

pip install -U astrostarfish

I'm going to go ahead and close this issue but feel free to reopen it if you run into issues!