Closed paigeh-fsa closed 1 year ago
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 have imported the disability dataset for NI and the proportions of disabled/non-disabled people that are being served/unserved are now part of the NI script.
You will require a new data file in Sync: data/disability_status/qs303_ni.csv to run the script.
I also created a new function
disab_dict
(may want to change the name) indata_transform
to cut out a whole chunk of copied code across all three scripts. It creates the dataframe for those who are served/unserved for each SA and puts it into a dataframe for that local authority/local government district inside the disability dictionary created. This was created to adhere to DRY across all three scripts and will cut down a lot of the process in the main script when we merge.Closes or fixes
Fixes #
Code
Documentation
Any new code includes all the following forms of documentation:
parameters
andreturns
for all major functionsData
Testing