Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: dataset comparison default names #1159

Merged
merged 1 commit into from
Nov 21, 2022
Merged

Conversation

alexbarros
Copy link
Contributor

Change the default titles of the reports for Dataset {char} where char is a capital letter in the alphabetic order.

@codecov-commenter
Copy link

Codecov Report

Base: 90.45% // Head: 90.44% // Decreases project coverage by -0.00% ⚠️

Coverage data is based on head (b7a398a) compared to base (c9850c2).
Patch coverage: 85.71% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1159      +/-   ##
===========================================
- Coverage    90.45%   90.44%   -0.01%     
===========================================
  Files          181      181              
  Lines         5509     5516       +7     
===========================================
+ Hits          4983     4989       +6     
- Misses         526      527       +1     
Flag Coverage Δ
py3.8-ubuntu-latest-pandas 90.44% <85.71%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ng/report/structure/variables/render_timeseries.py 93.10% <ø> (ø)
src/pandas_profiling/compare_reports.py 86.79% <85.71%> (-0.08%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alexbarros alexbarros merged commit c508c63 into develop Nov 21, 2022
@alexbarros alexbarros deleted the fix/compare_dataset_names branch November 21, 2022 16:44
aquemy pushed a commit that referenced this pull request Nov 22, 2022
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.

4 participants