Skip to content

fix: Address warnings, error on malformed TSV contents#2240

Merged
effigies merged 4 commits intobids-standard:maint/1.10.1from
effigies:fix/pandas_deprecation
Oct 30, 2025
Merged

fix: Address warnings, error on malformed TSV contents#2240
effigies merged 4 commits intobids-standard:maint/1.10.1from
effigies:fix/pandas_deprecation

Conversation

@effigies
Copy link
Copy Markdown
Collaborator

@effigies effigies commented Oct 29, 2025

Closes #2010 following pattern used in bids-standard/pybids#1136.

Closes #2238.

@effigies effigies requested a review from erdalkaraca as a code owner October 29, 2025 17:56
@effigies effigies added schema-code Updates or changes to the code used to parse, filter, and render the schema. exclude-from-changelog This item will not feature in the automatically generated changelog labels Oct 29, 2025
@effigies effigies requested a review from yarikoptic October 29, 2025 17:57
@effigies effigies force-pushed the fix/pandas_deprecation branch from 938969e to 3e01f2c Compare October 29, 2025 17:57
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 29, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.79%. Comparing base (6d002b2) to head (3e01f2c).

Files with missing lines Patch % Lines
...ols/schemacode/src/bidsschematools/render/utils.py 80.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2240      +/-   ##
==========================================
- Coverage   82.85%   82.79%   -0.06%     
==========================================
  Files          20       20              
  Lines        1621     1639      +18     
==========================================
+ Hits         1343     1357      +14     
- Misses        278      282       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@effigies effigies force-pushed the fix/pandas_deprecation branch from 3e01f2c to 963531d Compare October 30, 2025 01:21
@effigies effigies changed the title fix: Opt in to pandas 3.0 behaviors, when possible fix: Address warnings, error on malformed TSV contents Oct 30, 2025
@effigies effigies changed the base branch from master to maint/1.10.1 October 30, 2025 01:21
@effigies effigies force-pushed the fix/pandas_deprecation branch from 963531d to 69db614 Compare October 30, 2025 01:22
@effigies effigies force-pushed the fix/pandas_deprecation branch from 69db614 to 42223a0 Compare October 30, 2025 12:30
Copy link
Copy Markdown
Collaborator

@yarikoptic yarikoptic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @effigies . I test driven it (both context handler and decorator versions) and have not see warnings popping up any longer -- life will be so much better now!!!!

@effigies effigies merged commit 9e62f05 into bids-standard:maint/1.10.1 Oct 30, 2025
17 of 18 checks passed
@effigies effigies deleted the fix/pandas_deprecation branch October 30, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exclude-from-changelog This item will not feature in the automatically generated changelog schema-code Updates or changes to the code used to parse, filter, and render the schema.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

extra warning during build with recent panda

2 participants