Skip to content

MNT: Address deprecation warning in CI#864

Open
DimitriPapadopoulos wants to merge 2 commits intonipreps:masterfrom
DimitriPapadopoulos:delim_whitespace
Open

MNT: Address deprecation warning in CI#864
DimitriPapadopoulos wants to merge 2 commits intonipreps:masterfrom
DimitriPapadopoulos:delim_whitespace

Conversation

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor

FutureWarning: The 'delim_whitespace' keyword in pd.read_csv is deprecated and will be removed in a future version. Use ``sep='\s+'`` instead

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 17, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.13%. Comparing base (6f8243e) to head (cc61922).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #864   +/-   ##
=======================================
  Coverage   67.13%   67.13%           
=======================================
  Files          54       54           
  Lines        6849     6849           
  Branches      852      852           
=======================================
  Hits         4598     4598           
  Misses       2036     2036           
  Partials      215      215           

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

@effigies
Copy link
Copy Markdown
Member

Is there any chance you've actually built an image you could screenshot? If not, I can put it on my todo list.

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor Author

DimitriPapadopoulos commented Apr 18, 2024

No, unfortunately no screenshot.

@DimitriPapadopoulos
Copy link
Copy Markdown
Contributor Author

Merge #979 first to fix CI.

	Passing `palette` without assigning `hue` is deprecated
	and will be removed in v0.14.0. Assign the `x` variable
	to `hue` and set `legend=False` for the same effect.
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