`search_lightcurve(cadence="short")` includes FFI-based data for TESS sector 56+ / Cycle 5+ about lightkurve HOT 4 CLOSED

One expedient way to fix it is to change the short cadence threshold from 300 seconds to 200 seconds. Would there be any unintended side effects?

from lightkurve.

Hi @orionlee , good point. I tend to think people who use the 'short cadence' flag do actually want the postage stamp products, not FFI data. I will think on how we want to handle this. Thanks for the suggestion!

from lightkurve.

BTW, the issue is also applicable to search_targetpixelfile(), because TESS-SPOC HLSP also releases TPFs, in addition to lightcurve files. For example,

> print(lk.search_targetpixelfile("TIC261136679", mission="TESS", sector=61, cadence="short"))

#     mission     year   author  exptime target_name distance  proposal_id 
                                     s                 arcsec               
--- -------------- ---- --------- ------- ----------- -------- -------------
  0 TESS Sector 61 2023      SPOC     120   261136679      0.0 G05144_G05155
  1 TESS Sector 61 2023 TESS-SPOC     200   261136679      0.0           N/A

from lightkurve.

I've put in PR #1410 that implements this fix (ie, short cadence is now <200 rather than < 300 so that FFI data is not returned.

from lightkurve.