Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issues and questions about cases with no pointing model #234

Open
orionlee opened this issue Aug 21, 2021 · 0 comments
Open

Issues and questions about cases with no pointing model #234

orionlee opened this issue Aug 21, 2021 · 0 comments

Comments

@orionlee
Copy link
Contributor

  • there is a bug in initializing pointing model in TargetData. Details below.
  • It'd be great if there is some discussion / guidance on how the absence of pointing model would affect constructed lightcurves. It looks like pointing model is not available for tpfs created from TESSCut, so it seems to be a common case.

Bug:

Line 218 below assumes source has pointing attribute.

  • But currently, a source created from TESSCut does not have pointing attribute, generating exception.
  • Thus the attempt to load one in line 221 is never executed.
  • Users would not know anything, because the exception is silently caught (without, say, issuing a warning).

try:
if source.pointing is not None:
self.pointing_model = source.pointing
else:
self.pointing_model = load_pointing_model(source.pm_dir, source.sector, source.camera, source.chip)
except:
self.pointing_model = None

  • The bug possibly possibly has no material effect for now, at least for TESSCut-based source, because loading of pointing model in line 221 will not succeed anyway (returning None), as source.pm_dir is None.
  • I am not sure if the bug might affect some other cases (where source.pointing does not exist and source.pm_dir is not None).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant