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

Prototype fix for CID uniqueness #1

Merged
merged 3 commits into from
Sep 6, 2023

Conversation

bmorris3
Copy link

@bmorris3 bmorris3 commented Aug 29, 2023

PR to spacetelescope#28 in order to overcome some issues with glue ComponentIDs encountered in the binning plugin implementation.

This PR:

  • ensures that the phase viewer uses the right x_att during binning (though maybe that logic should be in the ephemeris plugin on Data add?)
  • moves the unique CID dict to the helper
  • uses the name ephem_component instead of component when the "component" is referring to the name of a periodic event in the system rather than a glue Data component.

@bmorris3 bmorris3 marked this pull request as ready for review August 29, 2023 17:47
Copy link
Owner

@kecnry kecnry left a comment

Choose a reason for hiding this comment

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

Seems to do the trick! 🤞 Thanks!

@kecnry kecnry merged commit a91910a into kecnry:binning-plugin Sep 6, 2023
kecnry added a commit that referenced this pull request Sep 6, 2023
* fix for CID uniqueness
* fix missing phase_comp_lbl reference
* fix for binning test

---------

Co-authored-by: Kyle Conroy <[email protected]>
kecnry added a commit that referenced this pull request Sep 8, 2023
* fix for CID uniqueness
* fix missing phase_comp_lbl reference
* fix for binning test

---------

Co-authored-by: Kyle Conroy <[email protected]>
kecnry added a commit that referenced this pull request Sep 14, 2023
* fix for CID uniqueness
* fix missing phase_comp_lbl reference
* fix for binning test

---------

Co-authored-by: Kyle Conroy <[email protected]>
kecnry added a commit that referenced this pull request Sep 14, 2023
* fix for CID uniqueness
* fix missing phase_comp_lbl reference
* fix for binning test

---------

Co-authored-by: Kyle Conroy <[email protected]>
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