Contributions
Fixes I have submitted to open-source projects
Fixes I submitted to open-source tools I use, mostly Windows-specific.
| Project | Change | State | Root cause |
|---|---|---|---|
| XGBoost | PR #12583 | Merged | train() and cv() crashed on Windows whenever an evaluation metric came back NaN, because MSVC prints that as -nan(ind) and the parser did not expect it. |
| DuckDB Python | PR #626 | Closedfixed upstream in #686 | DataFrameReader.load inserted the raw file path directly into the SQL, so any path containing a slash, a space or a drive colon failed to parse.Maintainer evertlammerts cherry-picked my commit into #686 for v2.0. |
| Hugging Face Hub | PR #4950 | Closedfixed upstream in #4978 | The Windows long-path check ran before the temporary-file suffix was added, so a long download path produced a temporary file that could not be created.Wauplin closed it in favour of #4978, thanking me for the report and PR. The fix shipped in v1.33.0. |
| Tauri | PR #16060 | Merged | The bundler cached its 32-bit NSIS and WiX tools below System32, and WOW64 redirected them somewhere they did not exist, so no installer could be built under the Windows SYSTEM account. |
| Electron Packager | PR #1962 | Openapproved | If the --out path differed from the app directory only in the case of the drive letter, the output folder was not recognised and the previous build was packed into the new archive.Approved by MarshallOfSound on 10 October 2026. Merge pending. |
| ONNX Runtime | PR #32707 | Openapproved | write_calibration_table opened its output with no explicit encoding, so a tensor name outside the Windows code page aborted the whole quantization export.Approved by xadupre on 21 September 2026. |
| PyTorch | PR #197834 | Open | The C++ extension loader wrote main.cpp and build.ninja with no explicit encoding, so on Windows any non-ASCII character in the source or the build path failed the build before the compiler ran.An automated reviewer requested changes on 30 October 2026. No human review yet. |
| Tauri | PR #16058 | Open | tauri info panicked instead of reporting when a package manager returned a version string that is not valid semver.No review yet. |
| Polars | Issue #29406 | Answered | Installing polars-cloud silently switched the index type to UInt64, because polars-runtime-64 takes precedence over the 32-bit runtime once both are present.The maintainer says this is by design. |