search

oturns / geosnap

The Geospatial Neighborhood Analysis Package
https://oturns.github.io/geosnap-guide
BSD 3-Clause "New" or "Revised" License
247 stars 32 forks source link

LTDB reader: adjust for inflation? #53

Closed weikang9009 closed 5 years ago

weikang9009 commented 5 years ago

I might be missing sth, but based on this line:inflate_cols = ["mhmval", "mrent", "hinc"], I think only these three variables are adjusted for inflation while the other economic variables like per capita income incpc are not?

knaaptime commented 5 years ago

uh oh, that's right... I thought I'd caught them all but must have missed some.

I was also thinking about moving the adjust inflation and some of the other private functions into a public util namespace

--

Elijah Knaap, Ph.D. Associate Director Center for Geospatial Sciences University of California, Riverside knaaptime.com | @knaaptime On Feb 28, 2019, 10:02 AM -0800, Wei Kang notifications@github.com, wrote:

I might be missing sth, but based on this line:inflate_cols = ["mhmval", "mrent", "hinc"], I think only these three variables are adjusted for inflation while the other economic variables like per capita income incpc are not? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

weikang9009 commented 5 years ago

Yeah, that could be a good option. As long as we have a consistent interface, I am fine either way.