Skip to content

feat: validator for ferry trips + bikes allowance#1510

Merged
cka-y merged 4 commits intomasterfrom
issue/1499
Jul 4, 2023
Merged

feat: validator for ferry trips + bikes allowance#1510
cka-y merged 4 commits intomasterfrom
issue/1499

Conversation

@cka-y
Copy link
Copy Markdown
Contributor

@cka-y cka-y commented Jun 20, 2023

Summary:

Add a rule with a WARNING severity level that checks is bike_allowed is present for ferry trips, after the addition in MobilityData/GTFS_Schedule_Best-Practices#55.
Closes #1499

Expected behavior:

missing_bike_allowance warning notice is added to the report if ferry route types don't specify the bikes_allowed value. Valid values are 1 (bikes allowed) and 2 (bikes not allowed). Any value different from 1 or 2 will result in the warning being present in the report.

image

@cka-y cka-y requested a review from isabelle-dr June 20, 2023 18:42
@github-actions
Copy link
Copy Markdown
Contributor

❌ Invalid acceptance test.
New Errors: 1 out of 1430 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1430 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 80 out of 1430 datasets (~6%) are invalid due to code change, which is above the provided threshold of 1%.
Dropped Warnings: 0 out of 1430 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1430 sources (~0 %) are corrupted.
Commit: 6b2aac2
Download the full acceptance test report here (report will disappear after 90 days).
❌ Invalid acceptance test.

@isabelle-dr
Copy link
Copy Markdown
Contributor

Thank you for working on this!
This looks great 👍.

@isabelle-dr
Copy link
Copy Markdown
Contributor

@davidgamez, does this PR have any dependencies with the automatic generation of documentation?

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 3, 2023

❌ Invalid acceptance test.
New Errors: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 80 out of 1435 datasets (~6%) are invalid due to code change, which is above the provided threshold of 1%.
Dropped Warnings: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1435 sources (~0 %) are corrupted.
Commit: 2decc04
Download the full acceptance test report here (report will disappear after 90 days).
❌ Invalid acceptance test.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 3, 2023

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: ./gradlew goJF.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 3, 2023

❌ Invalid acceptance test.
New Errors: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 80 out of 1435 datasets (~6%) are invalid due to code change, which is above the provided threshold of 1%.
Dropped Warnings: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1435 sources (~0 %) are corrupted.
Commit: efb5d30
Download the full acceptance test report here (report will disappear after 90 days).
❌ Invalid acceptance test.

@cka-y cka-y requested review from davidgamez and jcpitre July 3, 2023 18:08
@jcpitre
Copy link
Copy Markdown
Contributor

jcpitre commented Jul 3, 2023

@cka-y I think it would be worthwhile to verify that the failing acceptance tests are consistant with the change in the code and were expected.
In such a situation I would say it's good practice to explain the test failures in a PR comment.

@cka-y
Copy link
Copy Markdown
Contributor Author

cka-y commented Jul 3, 2023

Since we introduced a new warning notice, some acceptance tests are failing.
After examination of the evaluation report, I can confirm that all differences are expected and related to the newly added warning notice.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 3, 2023

❌ Invalid acceptance test.
New Errors: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
Dropped Errors: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
New Warnings: 80 out of 1435 datasets (~6%) are invalid due to code change, which is above the provided threshold of 1%.
Dropped Warnings: 0 out of 1435 datasets (~0%) are invalid due to code change, which is less than the provided threshold of 1%.
0 out of 1435 sources (~0 %) are corrupted.
Commit: baa073c
Download the full acceptance test report here (report will disappear after 90 days).
❌ Invalid acceptance test.

@cka-y cka-y merged commit ec14841 into master Jul 4, 2023
@cka-y cka-y deleted the issue/1499 branch July 4, 2023 16:04
davidgamez pushed a commit that referenced this pull request Jul 5, 2023
* feat: validator for ferry trips + bike allocation

* feat: removed "info" suffix

* feat: fixed formatting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update validator with new Best Practice (bike_allowed present for ferry trips)

4 participants