Closed james-westwood 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.
@nkshaw23 I think this is ready to go. I just ran the code and it seems fine now.
Order to merge pull requests: 1) #360 2) #361 3) This one: #366
Pull Request submission
Main changes include:
Note: No additional data or dependencies needed to run this code.
Please check the data output - train_highly_serviced_stops.csv (and the df and the feather). Is this what we actually want?
Closes or fixes
Fixes #355
Code
Documentation
Any new code includes all the following forms of documentation:
parameters
andreturns
for all major functions N/AData
Testing