[CLOSED] sample.nsa_logmstar broken

[albireox]

Issue by bretthandrews
Thursday Jun 30, 2016 at 16:13 GMT
Originally opened as https://github.com/marvin-manga/marvin/issues/1

---

The keyword sample.nsa_logmstar is available to query on, but doesn't work.

[albireox]

Comment by havok2063
Thursday Jul 07, 2016 at 01:19 GMT

---

The sample table is deprecated. At some point we should remove it to avoid confusion and conflict. The NSA parameters are all in sampledb.NSA now. I've modified the SPM SampleModelClasses to include petro_logmass_el and sersic_logmass attributes. These are now usable and filterable by sqlalchemy. If you update the marvin branch of SPM you'll get the changes. These should also be pulled back into the git repo for sampledb.

However, until we can sort out our hybrid property problem, they will continue to be unusable by Marvin Query.

[albireox]

Comment by havok2063
Thursday Jul 07, 2016 at 13:34 GMT

---

Closing this since this particular issue has been fixed and the problem essentially shifted to #4.