Skip to content

Conversation

@wence-
Copy link
Contributor

@wence- wence- commented May 23, 2017

Fixes bug reported on mailing list by Wei Pan.

@miklos1
Copy link
Member

miklos1 commented May 23, 2017

What is changing here?

The FunctionSpace dim needs to be part of the cache key!
@wence- wence- force-pushed the mg-transfer-fix branch from 1c21406 to 7690aaf Compare May 23, 2017 15:56
@wence-
Copy link
Contributor Author

wence- commented May 23, 2017

Better?

@miklos1
Copy link
Member

miklos1 commented May 23, 2017

Yes.

Copy link
Member

@thomasgibson thomasgibson left a comment

Choose a reason for hiding this comment

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

Looks fine to me

@wence- wence- merged commit 7690aaf into master Jun 9, 2017
wence- added a commit that referenced this pull request Jun 9, 2017
* mg-transfer-fix:
  multigrid: Fix kernel retrieval
  multigrid: Test buggy kernel cache retrieval
@wence- wence- deleted the mg-transfer-fix branch June 9, 2017 13:21
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.

4 participants