I have put the aggregate estimate for NI here. I have not addressed all of the tasks in the ticket, see #356, this will be done in another ticket.
As part of this ticket, I have downloaded a new package to convert latitude and longitude coordinates to BNG. For this, you need to install the package convertbng, which is also in the requirements.txt.
I would redownload all NI datasets to run this ticket as the majority of NI data has changed quite a bit over recent weeks.
I still need to write some methodological changes here and will edit here when this has been done.
Closes or fixes
Fixes #
Code
[ ] Requirements My/our code matches the requirements of the ticket
[ ] Functionality: New functions meet requirements in issue ticket
[ ] Compliant Code Code is as PEP 8 compliant as I can humanly make it
[ ] Code runs The code runs on my machine
[ ] Clean Code
[ ] Code has been linted (use your favourite linter)
Pull Request submission
I have put the aggregate estimate for NI here. I have not addressed all of the tasks in the ticket, see #356, this will be done in another ticket.
As part of this ticket, I have downloaded a new package to convert latitude and longitude coordinates to BNG. For this, you need to install the package convertbng, which is also in the requirements.txt.
I would redownload all NI datasets to run this ticket as the majority of NI data has changed quite a bit over recent weeks.
I still need to write some methodological changes here and will edit here when this has been done.
Closes or fixes
Fixes #
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.