Skip to content

Conversation

rhshadrach
Copy link
Member

…edge case in HDFStore.append (#60829)

  • Add clearer error messages for datatype mismatch in HDFStore.append. Raise ValueError when nan_rep too large for pytable column. Add and modify applicable test code.

  • Fix missed tests and correct mistake in error message.

  • Remove excess comments. Reverse error type change to avoid api changes. Move nan_rep tests into separate function.

(cherry picked from commit 57340ec)

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.

…edge case in HDFStore.append (pandas-dev#60829)

* Add clearer error messages for datatype mismatch in HDFStore.append. Raise ValueError when nan_rep too large for pytable column. Add and modify applicable test code.

* Fix missed tests and correct mistake in error message.

* Remove excess comments. Reverse error type change to avoid api changes. Move nan_rep tests into separate function.

(cherry picked from commit 57340ec)
@rhshadrach rhshadrach added this to the 2.3 milestone Feb 10, 2025
@mroeschke mroeschke added IO HDF5 read_hdf, HDFStore Error Reporting Incorrect or improved errors from pandas labels Feb 10, 2025
@mroeschke mroeschke merged commit 36ae10d into pandas-dev:2.3.x Feb 10, 2025
51 of 53 checks passed
@mroeschke
Copy link
Member

Thanks @rhshadrach

@rhshadrach rhshadrach deleted the auto-backport-of-pr-60829-on-2.3.x branch February 11, 2025 22:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Error Reporting Incorrect or improved errors from pandas IO HDF5 read_hdf, HDFStore

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants