locations.geojson POC phase2: End-to-end partial support of json data#1810
locations.geojson POC phase2: End-to-end partial support of json data#1810
Conversation
jcpitre
added
the
do not merge
|
This contribution does not follow the conventions set by the Google Java style guide. Please run the following command line at the root of the project to fix formatting errors: |
jcpitre
changed the title
jcpitre
changed the title
There was a problem hiding this comment.
Could you add class comments to help understanding the purpose of this abstract class?
There was a problem hiding this comment.
Added class comments.
There was a problem hiding this comment.
Same here, class comment could be useful.
There was a problem hiding this comment.
Added class comments.
There was a problem hiding this comment.
What's the benefit of pulling TableStatus out from GtfsTableContainer? Just curious, coz nothing is changed in this enum class.
There was a problem hiding this comment.
I would say it's not essential but it's not bad to have a file dedicated to one enum.
📝 Acceptance Test Report📋 Summary✅ The rule acceptance has passed for commit 4f9e3db 📊 Notices ComparisonNew Errors (0 out of 1575 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Errors (0 out of 1575 datasets, ~0%) ✅No changes were detected due to the code change. New Warnings (0 out of 1575 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Warnings (0 out of 1575 datasets, ~0%) ✅No changes were detected due to the code change. 🛡️ Corruption Check0 out of 1575 sources (~0 %) are corrupted. ⏱️ Performance Assessment📈 Validation TimeAssess the performance in terms of seconds taken for the validation process.
|
|
@jcpitre One note re: the test notice of So there are two options:
|
Yes, I put anything related to location_groups.txt in comments in the validator class so we hopefully don't forget. |
jcpitre
removed
the
do not merge
📝 Acceptance Test Report📋 Summary✅ The rule acceptance has passed for commit 78eee27 📊 Notices ComparisonNew Errors (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Errors (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. New Warnings (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Warnings (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. 🛡️ Corruption Check0 out of 1588 sources (~0 %) are corrupted. ⏱️ Performance Assessment📈 Validation TimeAssess the performance in terms of seconds taken for the validation process.
|
📝 Acceptance Test Report📋 Summary✅ The rule acceptance has passed for commit 3dcfc5c 📊 Notices ComparisonNew Errors (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Errors (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. New Warnings (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Warnings (0 out of 1588 datasets, ~0%) ✅No changes were detected due to the code change. 🛡️ Corruption Check0 out of 1588 sources (~0 %) are corrupted. ⏱️ Performance Assessment📈 Validation TimeAssess the performance in terms of seconds taken for the validation process.
|
4 participants
Summary:
Builds on the poc written by @davidgamez (#1805), related to locations.geojson
What it does:
Geojsonin the name)Not covered:
To be done:
This is NOT ready to merge yet.
I modified an existing dataset that has flex data to create an error where the id of a feature in
locations.geojsonis the same as an id instops.txt.browncounty-mn-us--flex-v2-broken.zip
Here is a capture of the sample notice in the report:
Please make sure these boxes are checked before submitting your pull request - thanks!
gradle testto make sure you didn't break anything