I have included the urban and rural data for Northern Ireland, and uploaded this to sync. A new folder will have to be created and stored here: data/urban_rural/NI/NI_urb_rur_class.csv
I have gone with the definition that we are making urban >10,000 and rural as <10,000. This means that bands A-D are urban and E-H are rural.
I have also included a new function that does the urban rural classification across all 3 scripts
Closes or fixes
Fixes #316
Code
[x] Requirements My/our code matches the requirements of the ticket
[x] Functionality: New functions meet requirements in issue ticket
[x] Compliant Code Code is as PEP 8 compliant as I can humanly make it
[x] Code runs The code runs on my machine
[x] Clean Code
[ ] Code has been linted (use your favourite linter)
Hi @Antonio-John I have done this now and saved them with different names. When we combine data together in our final output these will have to be removed from the config.
Pull Request submission
I have included the urban and rural data for Northern Ireland, and uploaded this to sync. A new folder will have to be created and stored here: data/urban_rural/NI/NI_urb_rur_class.csv
I have gone with the definition that we are making urban >10,000 and rural as <10,000. This means that bands A-D are urban and E-H are rural.
I have also included a new function that does the urban rural classification across all 3 scripts
Closes or fixes
Fixes #316
Code
Documentation
Any new code includes all the following forms of documentation:
parameters
andreturns
for all major functionsData
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.