search

crflynn / pypistats.org

PyPI downloads analytics dashboard
https://pypistats.org/
138 stars 10 forks source link

Sort and dedupe requirements #23

Closed Zac-HD closed 3 years ago

Zac-HD commented 4 years ago

The current PyPIstats listing does not distinguish at all between requirements, which can be duplicated. https://pypistats.org/packages/hypothesis provides a good example - pytz is listed four times!

Since working out how to display named sets of optional dependencies and environment markers for conditional dependencies is rather complicated, I settled for simply de-duplicating and sorting the existing list for clarity.

This fixes #17.

Zac-HD commented 4 years ago

I should also say: thanks so much for the site! It's great to satisfy general curiosity about what everyone else is using, and as a maintainer it's a fantastic tool when we're deciding what versions to support :smile:

Zac-HD commented 4 years ago

Before:

Requires: attrs | sortedcontainers | enum34 | django | dpcontracts | lark-parser | numpy | pandas | pytest | python-dateutil | pytz | pytz | python-dateutil | pytz | django | dpcontracts | lark-parser | numpy | pandas | pytest | pytz

After:

Requires: attrs | sortedcontainers | enum34 (python_version == "2.7") Optional dependencies: dateutil: python-dateutil django: pytz | django dpcontracts: dpcontracts lark: lark-parser numpy: numpy pandas: pandas pytest: pytest pytz: pytz

Zac-HD commented 4 years ago

@crflynn - any chance that this will be merged? If you have any suggestions I'd be happy to hear them :smile:

Zac-HD commented 3 years ago

Closing this pull request because the site is down (persistent 502 bad gateway), and the project seems dead 😞

hugovk commented 3 years ago

A similar fix has been deployed in https://github.com/crflynn/pypistats.org/issues/31 👍🏼