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

Fix marginal counts for pulse jobs. Fixes #6430. #6486

Closed
wants to merge 3 commits into from

Conversation

rithikaadiga
Copy link
Contributor

Fixes #6430

Code fix checks if creg_sizes is an attribute in the object and executes related code only if it is.

…s if creg_sizes is an attribute in the object and executes related code, only if it is.
@rithikaadiga rithikaadiga requested a review from a team as a code owner May 30, 2021 22:22
@CLAassistant
Copy link

CLAassistant commented May 30, 2021

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@taalexander taalexander 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 to me, would you mind adding a test?

@taalexander
Copy link
Contributor

Is there an update on this @rithikaadiga?

@rithikaadiga
Copy link
Contributor Author

@taalexander I'll add that test right away. Also troubleshooting the issues with the Lint. Apologies for the delay.

@taalexander
Copy link
Contributor

@rithikaadiga any update on this? This is almost ready to be merged and just requires a quick test.

@1ucian0
Copy link
Member

1ucian0 commented Mar 31, 2022

Closing this one as stalled. @rithikaadiga , if you are back, let's reopen it!

@1ucian0 1ucian0 closed this Mar 31, 2022
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.

marginal_counts is broken for pulse results
4 participants