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

minor: trending function add cycle 3 routine WFS PIDs #878

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

mperrin
Copy link
Collaborator

@mperrin mperrin commented Jul 8, 2024

Very minor PR: Adds the cycle 3 WFS program IDs to the list inside wavefront_time_series_plotwavefront_time_series_plot`. This has only the trivial effect of changing the color of the dot used to plot those datapoints.

@mperrin
Copy link
Collaborator Author

mperrin commented Jul 24, 2024

@nflagey-stsci I'm wondering if you could step in to provide a Github review and approval of this fairly simple and trivial PR, please, since Marcio is away this week?

Copy link
Collaborator

@obi-wan76 obi-wan76 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. We should think of a way to add the relevant sensing programs without hard coding their PIDs.

@obi-wan76 obi-wan76 merged commit cab6ef4 into spacetelescope:develop Jul 31, 2024
7 checks passed
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

Successfully merging this pull request may close these issues.

2 participants