search

LCOGT / mop

Microlensing Observation Portal
GNU General Public License v3.0
0 stars 7 forks source link

Priority targets list should not list events that are not alive #111

Open rachel3834 opened 10 months ago

rachel3834 commented 10 months ago

Exclude events with Alive=False.

rachel3834 commented 10 months ago

Nor should it include events flagged as known YSOs etc

rachel3834 commented 9 months ago

Added a method to impose these selection criterion, since it proved tricky to search the target_extras table for two different key words at the same time. This isn't a traditional 'join' filter operation due to the way the extra parameters are stored as optional table rows rather than columns.

rachel3834 commented 9 months ago

Implemented in v4.2.12

KKruszynska commented 7 months ago

This target appears in the Priority target view in the Planetary candidates section.

rachel3834 commented 7 months ago

This was caused by an oversight of mine while I was refactoring for DB query efficiency - while ensuring that targets are not YSOs etc, I accidentally removed the requirement for targets to be alive. :roll_eyes: I've reinstated this constraint in v.4.3.21.

KKruszynska commented 3 months ago

Reopening this. After the big change in the DB, the priority target view has a lot of dead events, dating back to 2020.