barahona-research-group / PyGenStability

PyGenStability: Multiscale community detection with generalized Markov Stability
https://barahona-research-group.github.io/PyGenStability/
GNU General Public License v3.0
32 stars 11 forks source link

update installation in readme #92

Closed d-schindler closed 10 months ago

d-schindler commented 11 months ago

What happened to the contrib module? Is it not an extra anymore?

codecov-commenter commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (783ff73) 100.00% compared to head (72b8382) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #92 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 7 7 Lines 642 642 ========================================= Hits 642 642 ``` | [Flag](https://app.codecov.io/gh/barahona-research-group/PyGenStability/pull/92/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=barahona-research-group) | Coverage Δ | | |---|---|---| | [pytest](https://app.codecov.io/gh/barahona-research-group/PyGenStability/pull/92/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=barahona-research-group) | `100.00% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=barahona-research-group#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

arnaudon commented 11 months ago

maybe let's keep both installations, clone and pypi.

arnaudon commented 11 months ago

What happened to the contrib module? Is it not an extra anymore?

it is installed by default in the main package, as it is not a dependency

d-schindler commented 11 months ago

maybe let's keep both installations, clone and pypi.

But then the section get's very long, especially when we go through all the extras again. We could present installation via git, but skip the extras for that one?

d-schindler commented 11 months ago

I added a (slightly modified) description of installation via cloning, have a look!

d-schindler commented 10 months ago

@arnaudon , can we merge this?

arnaudon commented 10 months ago

Sure, it looks good!