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 a default plotting utility before running fits #349

Open
sblunt opened this issue Nov 10, 2023 · 2 comments
Open

Add a default plotting utility before running fits #349

sblunt opened this issue Nov 10, 2023 · 2 comments
Assignees
Labels
easy enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@sblunt
Copy link
Owner

sblunt commented Nov 10, 2023

It's pretty common for people to mistype astrometry, or otherwise mess up the formatting in some way. This week alone, I talked to three people who had this issue! I think this could be decreased if orbitize! were to by default plot all the data before starting a fit.

@sblunt sblunt added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers easy labels Nov 10, 2023
@DTCupcakes
Copy link
Collaborator

I've added a method in the system class in the new branch to plot data given in ra/dec and sep/pa (converted to ra/dec). I plan to add functionality to separate different objects/planets in the plot by colour.

Would like to know if any of the following functionality would be useful:

  • Additional plots for sep/pa/RV
  • A way to separate data on the plot by instrument

@sblunt
Copy link
Owner Author

sblunt commented Jul 23, 2024

Great! Can you make a pull request? I'll try to check it out this week.

@DTCupcakes DTCupcakes linked a pull request Jul 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
easy enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants