Skip to content

Conversation

@jackye1995
Copy link
Contributor

No description provided.

jackye1995 and others added 2 commits January 22, 2026 20:41
…ly (lance-format#5793)

In Rust nightly, __cpuid was changed to be a safe function, making the
unsafe block unnecessary. This causes a warning that fails CI when -D
warnings is set (e.g., via setup-rust-toolchain action).

Adding #[allow(unused_unsafe)] allows the code to compile without
warnings on both stable (where unsafe is required) and nightly (where
it's no longer needed).

Also add RUSTFLAGS="-D warnings" to the linux-build CI job to catch such
issues early.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
This PR will expose blob handling APIs to python so that users can just
scan all blobs as binary.

---

**Parts of this PR were drafted with assistance from Codex (with
`gpt-5.2`) and fully reviewed and edited by me. I take full
responsibility for all changes.**

---------

Co-authored-by: Will Jones <willjones127@gmail.com>
@github-actions github-actions bot added bug Something isn't working python labels Jan 23, 2026
@github-actions
Copy link
Contributor

Review Summary

This is a cherry-pick PR for v2.0.0-rc.2 that brings two previously merged commits:

P1 Issue Found

Typo in type hint (python/python/lance/dataset.py:697):

The Literal type hint uses "blobs_description" (singular), but everywhere else uses "blobs_descriptions" (plural):

  • Documentation at line 790: "blobs_descriptions"
  • Validation set at line 4802: {"all_binary", "blobs_descriptions", "all_descriptions"}
  • Rust enum: BlobsDescriptions

This means type checkers will flag the correct value as invalid. Should be:

blob_handling: Optional[
    Literal["all_binary", "blobs_descriptions", "all_descriptions"]
] = None,

No Other Issues

The rest of the changes look correct:

  • CI fix for RUSTFLAGS="-D warnings" is appropriate
  • The #[allow(unused_unsafe)] fix is the correct approach for stable/nightly compatibility
  • Test coverage is adequate for the new blob handling feature
  • Validation logic properly matches between Python and Rust layers

…rmat#5759)

Closes lance-format#5762

- Replace `@torch.jit.script` with `@torch.compile` in distance.py (6
functions)
- Add `torch>=2.0` version constraint
- Add filterwarning for PyTorch inductor's use of deprecated
`load_module()`

- [x] `pytest python/tests/torch_tests/test_distance.py` passes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Jan 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Xuanwo and others added 4 commits January 22, 2026 23:15
Pandas 3 just released.  This caused some tests to fail.
This PR will try to make tests more stable

---

**Parts of this PR were drafted with assistance from Codex (with
`gpt-5.2`) and fully reviewed and edited by me. I take full
responsibility for all changes.**
@jackye1995 jackye1995 merged commit e1ab5c4 into lance-format:release/v2.0 Jan 23, 2026
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants