-
Notifications
You must be signed in to change notification settings - Fork 0
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
Copy headers to prefix/include #19
Conversation
e97dab5
to
9a9e919
Compare
@ELundby45 From what I see in https://conda-metadata-app.streamlit.app/?q=pkgs%2Fmain%2Fwin-64%2Fpybind11-global-2.13.6-py39h214f63a_0.conda, the headers should already be there no? |
I don't think so? I see:
But what I need is:
|
|
a sub-system of pytorch is expecting pip-like include directories. alternative is to patch pytorch to expect more conda-like directory structure. I prefer this solution as it's more broadly applicable, but either will do imho (although patching pytorch will be harder) |
it occurs when you define custom pytorch operators, possibly specific to automatic gradient calculation. The error hasn't cropped up in the general case. |
The thing is that all packages in uou ecosystem look for stuff in |
Basically, my concern is that merging this change could potentially hide very important problems in our ecosystem. |
in the rest of the ecosystem or just pytorch? Yea, probably best to make sure pytorch is looking in the right place at runtime. Otherwise the jit functions might not work somewhere else either. |
Saying ecosystem was intentional. If something in pytorch is not able to find the right files where they should usually be, it could well be an ecosystem problem or a pytorch problem. By "fixing" the problem in pybind11, you risk making the problem bleed into potentially the rest of the ecosystem. |
pybind11 rebuild
Changes
%PREFIX%/include/pybind11
this matches the functionality of the pip package and allows me to remove a hack that was required here: CI SKIP Pkg 6001 win cf pytorch-feedstock#53 (comment)