Skip to content
This repository has been archived by the owner on Dec 14, 2023. It is now read-only.

remove map_security_sectors() once its purpose is shifted to 2DegreesInvesting/data_preparation #351

Open
cjyetman opened this issue Nov 27, 2020 · 2 comments
Open

remove map_security_sectors() once its purpose is shifted to 2DegreesInvesting/data_preparation #351

cjyetman opened this issue Nov 27, 2020 · 2 comments
Assignees
@cjyetman

Comments

@cjyetman
Copy link
Member

Once 2DegreesInvesting/data_preparation/issues/14 has been resolved, we should remove map_security_sectors(), whose purpose should be fully superseded by whatever is implemented over there.
@cjyetman cjyetman self-assigned this Nov 27, 2020
@cjyetman
Copy link
Member Author

This is also relevant to the refactoring happening in #247

@cjyetman
Copy link
Member Author

cjyetman commented Dec 7, 2020

also development in https://github.com/cjyetman/data_preparation/tree/input-data-validation to add validation functions and make the data_preparation code more robust is relevant

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@cjyetman cjyetman

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cjyetman