-
Notifications
You must be signed in to change notification settings - Fork 15
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
Build with NumPy 2 #762
Build with NumPy 2 #762
Conversation
For [sc-53799]. |
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.
@jdblischak here are some failed windows build experiments with similar changes to
The windows feedstock is also broken - https://github.com/TileDB-Inc/tiledb-vcf-feedstock/runs/29369534110 I'll try a few more experiments. Let me know if you see any clues in the error logs above. |
So the errors here are
These are different than the errors that are currently breaking tiledb-vcf-feedstock, which look like:
Though these are also related to libfmt |
f2ab2ac
to
7471a5d
Compare
ff6fa8d
to
5c1ec43
Compare
@jdblischak, @ihnorton as we saw in #778 (comment), the windows build is broken for reasons other that this PR. At this point, I plan to merge PRs with failing windows builds. |
Ok, SGTM. @dudoslav is looking at the windows issues. |
A quick post-mortem on this. I had been very puzzled why the libfmt errors we have observed in tiledb-vcf-feedstock for months all of a sudden appeared in the Windows CI here in the TileDB-VCF repo. I think I have an explanation:
Unfortunately I don't know how to fix this. But if we can figure out the libfmt linking problem, we can solve the Windows builds both in this repo and in the feedstock. Here is how I arrived at this conclusion. I copy-pasted the output of $ diff vcf-win-pass.txt vcf-win-fail.txt | grep cmake
< cmake 3.29.6 h400e5d1_0 conda-forge
> cmake 3.30.5 h400e5d1_0 conda-forge
$ diff vcf-win-pass.txt vcf-win-fail.txt | grep fmt
> fmt 11.0.2 h7f575de_0 conda-forge
$ diff vcf-win-pass.txt vcf-win-fail.txt | grep spdlog
> spdlog 1.14.1 h9f2357e_1 conda-forge
$ grep fmt vcf-win-*
vcf-win-fail.txt: fmt 11.0.2 h7f575de_0 conda-forge
$ diff vcf-win-pass.txt vcf-win-fail.txt | grep tiledb
< m2w64-htslib 1.19 h8a9f4b3_0 tiledb
< tiledb 2.15.4 h616f04a_1 conda-forge
< tiledb-py 0.21.6 py311h37c03f0_0 conda-forge
> m2w64-htslib 1.20 h8a9f4b3_0 tiledb
> tiledb 2.25.0 h19c5691_18 conda-forge
> tiledb-py 0.31.1 py312h195dda3_0 conda-forge
$ diff vcf-win-pass.txt vcf-win-fail.txt | grep pyarrow
< pyarrow 9.0.0 py311h6a6099b_49_cpu conda-forge
< pyarrow-hotfix 0.6 pyhd8ed1ab_0 conda-forge
> pyarrow 17.0.0 py312h7e22eef_1 conda-forge
> pyarrow-core 17.0.0 py312h6a9c419_1_cpu conda-forge
> pyarrow-hotfix 0.6 pyhd8ed1ab_0 conda-forge
$ diff vcf-win-pass.txt vcf-win-fail.txt | grep vs
< vs2015_runtime 14.40.33810 h3bf8584_20 conda-forge
< vs2019_win-64 19.29.30139 he1865b1_20 conda-forge
< vswhere 3.1.4 h57928b3_0 conda-forge
> vs2015_runtime 14.40.33810 h3bf8584_22 conda-forge
> vs2019_win-64 19.29.30139 he1865b1_22 conda-forge
> vswhere 3.1.7 h57928b3_0 conda-forge |
Changes in this PR
numpy
andpyarrow
pinsSee TileDB-Inc/tiledb-vcf-feedstock#138 for conda feedstock experiments using this branch.