-
Notifications
You must be signed in to change notification settings - Fork 101
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: added rider categories feature #1995
Conversation
📝 Acceptance Test Report📋 Summary✅ The rule acceptance has passed for commit 5468ee7 📊 Notices ComparisonNew Errors (1 out of 1824 datasets, ~0%) ✅Details of new errors due to code change, which is less than the provided threshold of 1%.
Dropped Errors (0 out of 1824 datasets, ~0%) ✅No changes were detected due to the code change. New Warnings (0 out of 1824 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Warnings (1 out of 1824 datasets, ~0%) ✅Details of new errors due to code change, which is less than the provided threshold of 1%.
🛡️ Corruption Check0 out of 1824 sources (~0 %) are corrupted. ⏱️ Performance Assessment📈 Validation TimeAssess the performance in terms of seconds taken for the validation process.
|
Metric | Dataset ID | Reference (s) | Latest (s) | Difference (s) |
---|---|---|---|---|
Average | -- | 470.03 MiB | 461.78 MiB | ⬇️-8.25 MiB |
Median | -- | 334.07 MiB | 335.92 MiB | ⬆️+1.85 MiB |
Standard Deviation | -- | 785.20 MiB | 759.80 MiB | ⬇️-25.39 MiB |
Minimum in References Reports | ro-vrancea-consiliul-judetean-vrancea-gtfs-1984 | 38.00 MiB | 68.95 MiB | ⬆️+30.96 MiB |
Maximum in Reference Reports | gb-unknown-uk-aggregate-feed-gtfs-2014 | 11.14 GiB | 11.00 GiB | ⬇️-142.56 MiB |
Minimum in Latest Reports | us-massachusetts-marthas-vineyard-transit-authority-gtfs-420 | 48.28 MiB | 39.42 MiB | ⬇️-8.86 MiB |
Maximum in Latest Reports | gb-unknown-uk-aggregate-feed-gtfs-2014 | 11.14 GiB | 11.00 GiB | ⬇️-142.56 MiB |
📝 Acceptance Test Report📋 Summary✅ The rule acceptance has passed for commit f53dc74 📊 Notices ComparisonNew Errors (0 out of 1824 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Errors (0 out of 1824 datasets, ~0%) ✅No changes were detected due to the code change. New Warnings (0 out of 1824 datasets, ~0%) ✅No changes were detected due to the code change. Dropped Warnings (0 out of 1824 datasets, ~0%) ✅No changes were detected due to the code change. 🛡️ Corruption Check0 out of 1824 sources (~0 %) are corrupted. ⏱️ Performance Assessment📈 Validation TimeAssess the performance in terms of seconds taken for the validation process.
📜 Memory Consumption
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary:
Closes #1992
Added
Rider Categories
feature.Please make sure these boxes are checked before submitting your pull request - thanks!
gradle test
to make sure you didn't break anything