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

Add pl# and ch# to parameter names when fitting multiple planets and/or channels #643

Open
1 task done
kevin218 opened this issue Apr 5, 2024 · 0 comments
Open
1 task done

Comments

@kevin218
Copy link
Owner

kevin218 commented Apr 5, 2024

Instrument

Light curve fitting (Stages 4-6)

What happened?

The current naming scheme is confusing when fitting multiple planets over multiple channels.

While we're at it, we should either:

  1. Update the damped oscillator parameter names to avoid duplicate keys, or
  2. Rewrite the code in s6_Spectra.py to only add parameter names if they are a perfect match.

Error traceback output

No response

What operating system are you using?

No response

What version of Python are you running?

No response

What Python packages do you have installed?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@github-actions github-actions bot added the LC Fit label Apr 5, 2024
@taylorbell57 taylorbell57 added this to To do in Stage 6: Spectra Plotting via automation Apr 5, 2024
@taylorbell57 taylorbell57 added this to To do in Road to v1.0 via automation Apr 5, 2024
@taylorbell57 taylorbell57 added this to To do in Stage 5: Light Curve Fitting via automation Apr 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

4 participants