transientskp / tkp

A transients-discovery pipeline for astronomical image-based surveys
http://docs.transientskp.org/
BSD 2-Clause "Simplified" License
19 stars 15 forks source link

Number of forced fits not always correct #493

Closed gijzelaerr closed 2 months ago

gijzelaerr commented 8 years ago

original issue reported here

https://github.com/transientskp/banana/pull/102

gijzelaerr commented 8 years ago

i'm tempted to just replace with with a count() and remove the manual administrated column which somewhere has a mistake in the SQL spagetti. @timstaley what do you think?

gijzelaerr commented 8 years ago

to clarify, I think the forcedfits_count column is an example of premature optimization.

AntoniaR commented 2 months ago

This issue will likely be resolved in the new redesign of the source association and other SQL commands in TraP R7. Closing this issue.