Skip to content

Conversation

@fangliu117
Copy link
Collaborator

The original code still prints the list of row indices even when print_nan_locations is  False. It also prints the indices twice when it is True (message + summary), which clutters the log if tens of thousands of rows are missing, dumping all indices to the console can overwhelm CI log.

I have refactored the function code by commenting out the duplicated line that prints the index list inside the summary block, so keeps logs concise, preserves full detail when requested.
It affects only the console output—it does not change what the function returns. Therefore this unit test assertion remains valid.

@fangliu117 fangliu117 merged commit 2373c39 into dev May 12, 2025
16 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