-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
[bug] recursively include examples in distribution script #2591
Conversation
* [vulkan] Add Vulkan API * fix * support osx * [skip ci] enforce code format Co-authored-by: Taichi Gardener <[email protected]>
* [vulkan] Add per kernel info structs * trace
This reverts commit f5f5ab2.
Ailing, If this one-liner change looks good then we can merge it to fix #2588. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thanks @victoriacity !
Btw since now we are testing in install mode, we should be able to test another example other than minimal so that similar bug can be caught in the future. I'm okay with adding the test in this PR or a followup :D |
Sounds good! I would be inclined to directly adding the test into this PR. |
Related issue = close #2588
Note: this will still not include CHI examples in PyPI distribution.