Skip to content

Fix DataFrame concatenation warning#40

Merged
gargrahul merged 1 commit intomainfrom
ahmed22002/dataframe-warning
Oct 6, 2025
Merged

Fix DataFrame concatenation warning#40
gargrahul merged 1 commit intomainfrom
ahmed22002/dataframe-warning

Conversation

@ahmed-bsod
Copy link
Copy Markdown
Contributor

Motivation

Fix DataFrame's FutureWarning: The behavior of DataFrame concatenation with empty or all-NA entries is deprecated. In a future version, this will no longer exclude empty or all-NA columns when determining the result dtypes. To retain the old behavior, exclude the relevant entries before the concat operation.

Technical Details

Skip concatenation if perf.csv is empty

@ahmed-bsod ahmed-bsod force-pushed the ahmed22002/dataframe-warning branch from 4164dd3 to 7fc0173 Compare September 22, 2025 11:22
@ahmed-bsod ahmed-bsod force-pushed the ahmed22002/dataframe-warning branch from 7fc0173 to ca92ff0 Compare October 2, 2025 20:29
@ahmed-bsod
Copy link
Copy Markdown
Contributor Author

Passed the CI tests

@ahmed-bsod ahmed-bsod requested review from coketaste and gargrahul and removed request for gargrahul October 3, 2025 11:32
Copy link
Copy Markdown
Collaborator

@coketaste coketaste left a comment

Choose a reason for hiding this comment

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

LGTM

@gargrahul gargrahul merged commit ce54310 into main Oct 6, 2025
@coketaste coketaste deleted the ahmed22002/dataframe-warning branch April 9, 2026 15:02
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.

3 participants