Skip to content

Fix plots bug due to new pandas casting#983

Merged
zerothi merged 3 commits intozerothi:mainfrom
pfebrer:fix_plots
Mar 13, 2026
Merged

Fix plots bug due to new pandas casting#983
zerothi merged 3 commits intozerothi:mainfrom
pfebrer:fix_plots

Conversation

@pfebrer
Copy link
Contributor

@pfebrer pfebrer commented Mar 12, 2026

Pandas has new rules for inferring types: https://pandas.pydata.org/docs/user_guide/migration-3-strings.html

This was making plots fail because None was suddenly parsed to NaN which in some cases was not accepted by the plotting routines.

value = dic[key]
try:
dic[key] = value.where(value.notnull(), other=None)
except:

Check notice

Code scanning / CodeQL

Empty except Note

'except' clause does nothing but pass and there is no explanatory comment.
value = dic[key]
try:
dic[key] = value.where(value.notnull(), other=None)
except:

Check notice

Code scanning / CodeQL

Except block handles 'BaseException' Note

Except block directly handles BaseException.
@zerothi zerothi merged commit 98ea838 into zerothi:main Mar 13, 2026
11 checks passed
@zerothi
Copy link
Owner

zerothi commented Mar 13, 2026

Thanks, fixed!

@codecov
Copy link

codecov bot commented Mar 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.67%. Comparing base (beee684) to head (fbcffc7).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #983   +/-   ##
=======================================
  Coverage   86.66%   86.67%           
=======================================
  Files         413      413           
  Lines       54373    54388   +15     
=======================================
+ Hits        47125    47140   +15     
  Misses       7248     7248           

☔ 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.

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