search

pysal / access

Classical and novel measures of spatial accessibility to services
https://pysal.org/access/
BSD 3-Clause "New" or "Revised" License
21 stars 13 forks source link

`sphinx` enforced to install w/ packages #11

Closed darribas closed 4 years ago

darribas commented 4 years ago

I've noticed that the requirements pin to sphinx, is this needed for the functionality to run? If not, it might make installs easier to not require sphinx (particularly a specific version).

knaaptime commented 4 years ago

there's already this line in requirements_docs.txt where we typically include sphinx, so the pin could be updated to 2.4 in that file and removed from the general package requirements

JamesSaxon commented 4 years ago

Ah, thanks Eli for this solution -- I did not know that these were separated. I'll try to get to it shortly.

Yeah, the conda-forge crew also noted the "heavy" sphinx dependency. I will move it.

On Wed, Aug 19, 2020 at 12:21 PM eli knaap notifications@github.com wrote:

there's already this line https://github.com/pysal/access/blob/bd3f9dbd8b35633c7fd1c45b78c28456b4ef7a17/requirements_docs.txt#L1 in requirements_docs.txt where we typically include sphinx, so the pin could be updated to 2.4 in that file and removed from the general package requirements

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pysal/access/issues/11#issuecomment-676556138, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADBJHPNF5FKSIRYSL3V3EMDSBQCX3ANCNFSM4P2HRLZQ .