Merged
Conversation
Bumps [requests](https://github.com/psf/requests) from 2.32.5 to 2.33.0. - [Release notes](https://github.com/psf/requests/releases) - [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md) - [Commits](psf/requests@v2.32.5...v2.33.0) --- updated-dependencies: - dependency-name: requests dependency-version: 2.33.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
When a C3D org has duplicate session properties, normalize_columns() raises a Polars DuplicateError and skips the entire project. This fixes both collision scenarios (unnest collision and clean-name collision) by suffixing duplicates with _2, _3, etc. instead of crashing. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Drop duplicate columns and log a warning instead of adding uninformative _2/_3 suffixes. Applies to both unnest collisions (property field vs top-level column) and clean-name collisions (different raw names that normalize to the same snake_case). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…rrors When API responses contain all-null columns, Polars infers them as Null dtype. This causes errors in downstream consumers (e.g. Fabric pipelines) when later rows contain actual string values. Cast any Null-typed columns to Utf8 (String) as a safe default in coerce_types(). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The top-level "hmd" field has inconsistent types across projects (null/int/string) causing Polars schema inference failures. Strip it before DataFrame construction and source the value from the reliable c3d.device.hmd_type property instead, aliasing it back as "hmd" for downstream compatibility. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
nccanderson
approved these changes
Apr 13, 2026
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
2 participants
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.
added documentation for both users and contributors