search

pysal / region

A library for Spatially-Explicit Regionalization
BSD 3-Clause "New" or "Revised" License
14 stars 16 forks source link

add verbose option to max_p #21

Closed knaaptime closed 5 years ago

knaaptime commented 5 years ago

this should only happen when verbose=True https://github.com/pysal/region/blob/a787cd6f42db906fe542a76723df9851243458ff/region/max_p_regions/heuristics.py#L70

knaaptime commented 5 years ago

there are todos later in the file that reference removing the print statements. I could just get rid of them in a PR instead of adding a verbose option?

ljwolf commented 5 years ago

Fine w/ me On Wed, 31 Oct 2018 at 04:59 eli knaap notifications@github.com wrote:

there are todos later in the file that reference removing the print statements. I could just get rid of them in a PR instead of adding a verbose option?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/pysal/region/issues/21#issuecomment-434561459, or mute the thread https://github.com/notifications/unsubscribe-auth/ACJY842NlO8GkzdHpZVgnyBrtLNdMG6rks5uqS4hgaJpZM4YC6YH .

--

Levi John Wolf

Lecturer in Quantitative Human Geography | University of Bristol

Fellow | Center for Spatial Data Science, University of Chicago

Steering Committee | Jean Golding Institute for Data Science, University of Bristol

Director | Q-Step Program, School of Geographical Sciences, University of Bristol

ljwolf.org