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

Parallelize Bayesian inference #73

Merged
merged 32 commits into from
Feb 2, 2024
Merged

Parallelize Bayesian inference #73

merged 32 commits into from
Feb 2, 2024

Conversation

ma-sadeghi
Copy link
Collaborator

No description provided.

@ma-sadeghi
Rename apply_heuristic_rules to filter_implausible_circuits for c…
3bd3648 
…larity
@ma-sadeghi
Expose hard-coded parameters as argument in perform_full_analysis
828794e
@ma-sadeghi
Fix docstrings
702a03b
@ma-sadeghi
filter_implausible_circuits shouldn't modify dataframe columns
4d0db1b
@ma-sadeghi
Fix docstrings
f2487d0
@ma-sadeghi
Fix pep8
ea31a93
@ma-sadeghi
Fix pep8
6b95fcd
@ma-sadeghi
Fix pep8
74f032f
@ma-sadeghi
Add validate_circuits_dataframe helper to utils
13c0052
@ma-sadeghi
perform_bayesian_inference now returns exit code to handle MCMC fai…
85690b1 
…lure
@ma-sadeghi
Rename Numpyro models for clarity
bb935da
@ma-sadeghi
Fix pep8
136f9a8
@ma-sadeghi
Add ruff guideline to pyproject.toml
d06250d
@ma-sadeghi
Fix pep8
f79b3f0
@ma-sadeghi
Fix type hint syntax
c815190
@ma-sadeghi
Fix error handling in perform_bayesian_inference function
0566f53
@ma-sadeghi
Add test circuits to assets
b11b85d
@ma-sadeghi
Add load_test_circuits helper to load toy circuits
1b393a0
@ma-sadeghi
No need to set numpy random seed, jax is enough
f28b126
@ma-sadeghi
Fix usage of jax random seeds when parallel processing
19dbab3
@ma-sadeghi
Toy circuits were in the wrong assets folder
41609c9
@ma-sadeghi
Add unit tests for inference (serial and parallel)
73a3509
@ma-sadeghi
Refactor of circuit function to be jittable!
613c659
@ma-sadeghi
Don't intrude user's with rich tracebacks!
f977ff4
@ma-sadeghi
Fix incorrectly used jax random seeds
bb0c53a
@ma-sadeghi
Fix bug in circuit validation where required_columns were being overr…
498d995 
…iden
@ma-sadeghi
Add parallelized version of inference
aa81b47
@ma-sadeghi
Log warning when no circuits survive the post filteration
5eebbe1
@ma-sadeghi
Use the parallelized version throughout the package
5d4eaec
@ma-sadeghi
Fix pep8
8d33c78
@ma-sadeghi ma-sadeghi added the enhancement New feature or request label Feb 2, 2024
@ma-sadeghi ma-sadeghi merged commit 5f815a9 into develop Feb 2, 2024
3 of 4 checks passed
@ma-sadeghi ma-sadeghi deleted the feature/parallelize-bi branch February 2, 2024 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ma-sadeghi