Skip to content

fix: remove method= and downcast= from fillna (removed in pandas 3.0)#1776

Merged
SFJohnson24 merged 1 commit into
cdisc-org:mainfrom
filippsatverily:filipps/pandas3-remove-fillna-params
Jun 23, 2026
Merged

fix: remove method= and downcast= from fillna (removed in pandas 3.0)#1776
SFJohnson24 merged 1 commit into
cdisc-org:mainfrom
filippsatverily:filipps/pandas3-remove-fillna-params

Conversation

@filippsatverily

Copy link
Copy Markdown
Contributor

Removes the method= and downcast= parameters from fillna() in DatasetInterface, PandasDataset, and DaskDataset. Both parameters were deprecated in pandas 2.2.0 and removed in pandas 3.0. No callers in the codebase pass these arguments — they were only present in the method signatures.

Tested scenarios:

  • Full pytest suite: 1746 passed, 11 skipped, 0 failed (pandas 2.3.3, dask 2025.12.0)
  • Ran validation on CDISC_Pilot_Study_v4_FIXED.json: 201 SUCCESS, 6 SKIPPED, 0 errors

@filippsatverily filippsatverily marked this pull request as ready for review June 22, 2026 21:26
@filippsatverily

Copy link
Copy Markdown
Contributor Author

@SFJohnson24 another commit from #1745

@SFJohnson24 SFJohnson24 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

PR correctly removes parameters--passes unittests, test suite (#1779) and pilot study.

@SFJohnson24 SFJohnson24 merged commit bd00e7c into cdisc-org:main Jun 23, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants