Repository navigation
Conversation
Assisted-by: Claude Code:claude-opus-5-5
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
|
…o int64 Rename the test to match the PR number and cover nested, mixed-depth fields so that both shared and extra index levels are checked. Assisted-by: Claude Code:claude-opus-5-5
Member
|
🤖 AI text below 🤖 I pushed two commits to this branch: e252a87 – install pandas on Windows CI. pandas was excluded on
1a70b39 – fixes and test.
|
Contributor
|
The documentation preview is ready to be viewed at https://preview.awkward-array.org/PR4492/ |
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ak.to_dataframecan produce int32 index levels on Windows while pandas constructs the expected levels as int64, failingtest_3694_to_dataframe_align_multindex.py::test_merge_option.Create row coordinates as int64 independently of the repeat-count dtype, and explicitly restore merged index levels as int64. Keep the existing platform-compatible repeat counts, including the int32 conversion on 32-bit platforms. This addresses 32-bit Windows as well as 64-bit Windows without changing dependency requirements.
Related: #4458 removes the Windows-wide repeat-count cast when dropping NumPy 1 support. This fix keeps that compatibility path and separates index widths from repeat-count widths; it also replaces the platform-dependent
astype(int)when restoring merged levels.Validation: 23 targeted dataframe tests passed, including simulated Windows 32-bit and 64-bit branches and all merge modes. Ruff and whitespace checks passed. Native Windows execution remains pending in CI; simulated platform flags do not substitute for a native 32-bit runtime.
AI assistance was used to implement and validate this contribution.