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.

Sign up for GitHub

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

Jump to bottom

fix: alias name kko to FCA in get_correlator_inputs #60

Merged
merged 3 commits into from
Jun 4, 2023
Merged

fix: alias name kko to FCA in get_correlator_inputs #60

merged 3 commits into from
Jun 4, 2023

Conversation

aelanman
Copy link
Contributor

@aelanman aelanman commented Jun 1, 2023

Querying the layout db for "KKO" or "kko" should return the KKO inputs. Currently only accepting pco.

Also adds support for GBO.

@aelanman aelanman requested a review from aaronpearlman June 1, 2023 21:20
@aaronpearlman
Copy link
Contributor

Looks good

@aaronpearlman aaronpearlman requested a review from jrs65 June 3, 2023 21:17
@aaronpearlman
Copy link
Contributor

@aelanman I think you can merge this PR into the master branch

@aelanman aelanman merged commit 9e99c30 into master Jun 4, 2023
@aelanman aelanman deleted the kko_alias branch June 4, 2023 19:54
@aelanman
Copy link
Contributor Author

aelanman commented Jun 4, 2023

Thanks @aaronpearlman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aaronpearlman aaronpearlman aaronpearlman approved these changes

@jrs65 jrs65 Awaiting requested review from jrs65

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aelanman @aaronpearlman