-
Notifications
You must be signed in to change notification settings - Fork 87
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
add topo to extra #1261
add topo to extra #1261
Conversation
👋 Hi, I'm @docu-mentor, an LLM-powered GitHub app Simply create a new comment in this PR that says: @docu-mentor run and I will start my analysis. I only look at what you changed @docu-mentor run doc/ README.md In this example, I'll have a look at all files contained in the "doc/" |
Hello @valer1435! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found: There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻 Comment last updated at 2024-03-01 08:19:26 UTC |
All PEP8 errors has been fixed, thanks ❤️ Comment last updated at |
add extra dependencies in unit test config
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1261 +/- ##
==========================================
+ Coverage 79.82% 79.85% +0.02%
==========================================
Files 150 146 -4
Lines 10322 10028 -294
==========================================
- Hits 8240 8008 -232
+ Misses 2082 2020 -62 ☔ View full report in Codecov by Sentry. |
Moved ripser to extra-dependencies