Skip to content

A couple of population improvements #27

@koldunovn

Description

@koldunovn

In the current main, there is a warning:

DtypeWarning: Columns (2) have mixed types. Specify dtype option on import or set low_memory=False.
  pop_dat = pd.read_csv(pop_path)

Can we specify dtype when open the data? Probably will five us some speedup.

Here https://github.com/koldunovn/climsight/blob/e4bf6d07f9f7b0cfbb60d9c123bc33b6665e22de/climsight.py#L498
We load population data once again. Instead I would load it once and pass to function as argument, it will save us one load.

Here https://github.com/koldunovn/climsight/blob/e4bf6d07f9f7b0cfbb60d9c123bc33b6665e22de/climsight.py#L791 we might check if data are not None and save a function call.

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions