Closed paigeh-fsa closed 1 year ago
I've added in a couple lines that will replace any comma in the census NI dataframe (for population) and will make these columns integers.
Fixes #356
Any new code includes all the following forms of documentation:
parameters
returns
requirements.txt
The author has responded to my review and made changes to my satisfaction.
Insert detailed comments here!
These might include, but not exclusively:
Your suggestions should be tailored to the code that you are reviewing. Be critical and clear, but not mean. Ask questions and set actions.
Pull Request submission
I've added in a couple lines that will replace any comma in the census NI dataframe (for population) and will make these columns integers.
Closes or fixes
Fixes #356
Code
Documentation
Any new code includes all the following forms of documentation:
parameters
andreturns
for all major functionsData
Testing
Peer Review Section
requirements.txt
Final approval (post-review)
The author has responded to my review and made changes to my satisfaction.
Review comments
Insert detailed comments here!
These might include, but not exclusively:
Your suggestions should be tailored to the code that you are reviewing. Be critical and clear, but not mean. Ask questions and set actions.