search

UDST / synthpop

Synthetic populations from census data
BSD 3-Clause "New" or "Revised" License
100 stars 46 forks source link

Memoryerror pd sum sum #46

Open cvanoli opened 6 years ago

cvanoli commented 6 years ago

-Deleted pd.sum.sum which was giving memory error. Changed it to a for loop summing on every row

coveralls commented 6 years ago

Coverage Status

Coverage increased (+0.07%) to 82.873% when pulling d390e2fd57f676276f7b558675ac6bc95fd8189d on memoryerror_pd_sum_sum into c3588e40657e9e4df3773d656747634f0feb81e3 on master.

janowicz commented 6 years ago

@cvanoli Thanks. Per our existing discussions with @sablanchard, this low-memory change should be made an optional one (e.g. a high-level low_memory parameter that defaults to False), so that those with the requisite memory can avoid the for loop.