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

Move observations to cpu in PyroConverter #1659

Merged
merged 2 commits into from
Apr 16, 2021
Merged

Conversation

fonnesbeck
Copy link
Contributor

Implements fix suggested in #1529 to allow traces on the GPU to be moved back to the CPU.

Closes #1529

@codecov
Copy link

codecov bot commented Apr 10, 2021

Codecov Report

Merging #1659 (882e3bb) into main (f98c5fc) will not change coverage.
The diff coverage is n/a.

❗ Current head 882e3bb differs from pull request most recent head 6b76eb2. Consider uploading reports for the commit 6b76eb2 to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1659   +/-   ##
=======================================
  Coverage   90.83%   90.83%           
=======================================
  Files         108      108           
  Lines       11720    11720           
=======================================
  Hits        10646    10646           
  Misses       1074     1074           
Impacted Files Coverage Δ
arviz/data/io_pyro.py 96.24% <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f98c5fc...6b76eb2. Read the comment docs.

@canyon289 canyon289 merged commit ab1277d into arviz-devs:main Apr 16, 2021
utkarsh-maheshwari pushed a commit to utkarsh-maheshwari/arviz that referenced this pull request May 27, 2021
* Move observations to cpu in PyroConverter

* Updated CHANGELOG with fix
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.

from_pyro fails when MCMC is run on GPU
3 participants