Fixed mean_cache data extraction for dKG#2452
Open
yyexela wants to merge 6 commits intocornellius-gp:mainfrom
Open
Fixed mean_cache data extraction for dKG#2452yyexela wants to merge 6 commits intocornellius-gp:mainfrom
mean_cache data extraction for dKG#2452yyexela wants to merge 6 commits intocornellius-gp:mainfrom
Conversation
Contributor
Author
|
Any idea why the unit test is failing when there is no code change? |
Contributor
Author
|
Noticed main fails the same test so I'm assuming my code works and it's another problem. EDIT: Unit tests aren't passing but neither are the ones in main, so I'll assume this change is what's needed for now. |
|
Any news on this PR? I confirm that there is interest in gradient-enabled GPs/BO for several application cases. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Issue: dKG in BoTorch no longer works since I last ran it a few months ago. It appears to be from the
mean_cachegoing from size[num_fantasies, 1, 1, num_data]to[num_fantasies, num_fantasies, num_fantasies, num_data]. What this change does is extract the relevant data ofmean_cachewhile keeping the expected shape for the rest of the code (ie.[num_fantasies, 1, num_data]).Not sure about a unit test for this since it depends on having BoTorch's
qKnowledgeGradientclass, among other dependencies.See meta-pytorch/botorch#2137