search

socialfoundations / folktables

Datasets derived from US census data
MIT License
240 stars 19 forks source link

Improved load_acs #35

Closed baraldian closed 1 year ago

baraldian commented 1 year ago

Improved loading function using pandas read_csv only instead of StrinIO

mrtzh commented 1 year ago

Thanks for your contribution!

jenno-verdonck commented 6 months ago

Hi, I just noticed that this pull request also remove the usage of the density parameter. This seems to be a parameter to take a sample from the data. Can this be added back?

baraldian commented 6 months ago

Hi, I just noticed that this pull request also remove the usage of the density parameter. This seems to be a parameter to take a sample from the data. Can this be added back?

Hi, I restored that feature in this pull request. Sorry for the mistake.

(I made 2 pull request because I found a more efficient solution, I closed the worst)