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

Make some tensors buffers in pathwise sampling code #1821

Closed
wants to merge 1 commit into from

Conversation

Balandat
Copy link
Contributor

@Balandat Balandat commented May 9, 2023

Fixes an issue when moving a SamplePath from GPU to GPU.

I wasn't super thorough to go through all of the code and every eventuality, but this at least makes the issue reported in #1820 go away.

@facebook-github-bot facebook-github-bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label May 9, 2023
@facebook-github-bot
Copy link
Contributor

@Balandat has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

Copy link
Member

@esantorella esantorella left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@Balandat Balandat force-pushed the fix_sample_path_dev_transfer branch from b320204 to e057876 Compare May 9, 2023 15:18
@facebook-github-bot
Copy link
Contributor

@Balandat has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@codecov
Copy link

codecov bot commented May 9, 2023

Codecov Report

Merging #1821 (720d234) into main (c90d640) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 720d234 differs from pull request most recent head 0ad2cfb. Consider uploading reports for the commit 0ad2cfb to get more accurate results

@@            Coverage Diff            @@
##              main     #1821   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          170       170           
  Lines        14911     14915    +4     
=========================================
+ Hits         14911     14915    +4     
Impacted Files Coverage Δ
botorch/sampling/pathwise/features/maps.py 100.00% <100.00%> (ø)
botorch/sampling/pathwise/paths.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Fixes an issue when moving a SamplePath from GPU to GPU reported in pytorch#1820
@Balandat Balandat force-pushed the fix_sample_path_dev_transfer branch from e057876 to 0ad2cfb Compare May 9, 2023 15:32
@facebook-github-bot
Copy link
Contributor

@Balandat has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@Balandat merged this pull request in 1b36cc4.

@Balandat Balandat deleted the fix_sample_path_dev_transfer branch May 18, 2023 03:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed Do not delete this pull request or issue due to inactivity. Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants