-
Notifications
You must be signed in to change notification settings - Fork 17
Open
Description
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.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels