feat(SpeedTripValidator): check for repeated arrival times

[josh-willis-arcadis]

Checklist

• Appropriate branch selected (all PRs must first be merged to dev before they can be merged to master)
• Any modified or new methods or classes have helpful JavaDoc and code is thoroughly commented
• The description lists all applicable issues this PR seeks to resolve
• The description lists any configuration setting(s) that differ from the default settings
• All tests and CI builds passing
• The description lists all relevant PRs included in this release (remove this if not merging to master)
• Code coverage does not significantly worsen (ideally it improves) (remove this if not merging to master)

Description

Adds an error check for duplicate arrival_times in stop_times.txt for a trip.
Based on the conveyal repo for projects still using that project.

[miles-grant-ibigroup]

This branch is not compatible with the MTC datatools branch, but I did some funky stuff in 29827a0441a5d0c7e50583a63e0df12b8216b7bd to resolve this. We should still merge this PR for now, but until the datatools MTC branch can handle the latest gifs-lib version, we can use my hack commit

[binh-dam-ibigroup]

Please remove the mass import.

[miles-grant-ibigroup]

Looks good!

[br648]

@josh-willis-arcadis once you have two approves you can merge! I don't need to review.

[binh-dam-ibigroup]

@josh-willis-arcadis once you have two approves you can merge! I don't need to review.

@br648 We will leave this one open for now until we bring the MTC datatools on par with the main datatools work.

[josh-willis-arcadis]

@br648 @binh-dam-ibigroup I agree lets keep it open. Github won't allow a PR to be merged until there are 3 approving reviews.

[br648]

Just one nit. Since when has three approves been required?! Anyway, here is the third.

[br648]

Perhaps change the name to tripHasDuplicateArrivalTimesError() and remove public.