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

Added import to banksim_combine_banks #5036

Merged
merged 1 commit into from
Feb 7, 2025
Merged

Added import to banksim_combine_banks #5036

merged 1 commit into from
Feb 7, 2025

Conversation

MarcoCusinato
Copy link
Contributor

This fixes a missing numpy import to the pycbc_banksim_combine_banks.

  • The author of this pull request confirms they will adhere to the code of conduct

@MarcoCusinato MarcoCusinato self-assigned this Feb 7, 2025
@spxiwh
Copy link
Contributor

spxiwh commented Feb 7, 2025

I wonder how this worked before?? Thanks for the patch!

@MarcoCusinato
Copy link
Contributor Author

I wonder how this worked before?? Thanks for the patch!

It did not :)

@MarcoCusinato MarcoCusinato merged commit 7368e24 into gwastro:master Feb 7, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@spxiwh spxiwh spxiwh approved these changes

@titodalcanton titodalcanton Awaiting requested review from titodalcanton

@pannarale pannarale Awaiting requested review from pannarale

Assignees

@MarcoCusinato MarcoCusinato

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MarcoCusinato @spxiwh