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

Fixed an index bug in DirectViewParallelizedModel #82

Merged
merged 2 commits into from
Sep 8, 2016

Conversation

ihincks
Copy link
Collaborator

@ihincks ihincks commented Sep 7, 2016

This bug caused most models to always run in serial mode.

@coveralls
Copy link

coveralls commented Sep 7, 2016

Coverage Status

Coverage remained the same at 45.192% when pulling f919c68 on ihincks:fix-serial-threshold into de46ca5 on QInfer:master.

@coveralls
Copy link

coveralls commented Sep 7, 2016

Coverage Status

Coverage remained the same at 45.192% when pulling f919c68 on ihincks:fix-serial-threshold into de46ca5 on QInfer:master.

@scasagrande
Copy link
Collaborator

🚢

@cgranade
Copy link
Collaborator

cgranade commented Sep 8, 2016

Well, that explains a bit. Thanks for catching this, merging in now!

@cgranade cgranade merged commit ea90440 into QInfer:master Sep 8, 2016
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.

None yet

4 participants