Skip to content
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

MobilityData Validator CSV Parsing Issue with large GTFS files #524

Open
philip-cline opened this issue Mar 24, 2023 · 3 comments
Open

MobilityData Validator CSV Parsing Issue with large GTFS files #524

philip-cline opened this issue Mar 24, 2023 · 3 comments
Assignees

Comments

@philip-cline
Copy link
Contributor

philip-cline commented Mar 24, 2023

Observed behavior

MobilityData Validation Result can run into CSV parsing issues when validating the MBTA's GTFS feed. Running the same feed with the MobilityData Validator locally does not produce the same result.

Expected behavior

Validator should not produce the parsing issue.

Steps to reproduce the problem

Validate MBTA's latest GTFS feed. Observe the following errors:
image

Any special notes on configuration used

None

Version of datatools-server and datatools-ui if applicable (exact commit hash or branch name)

Latest validator branch, dev server.

@miles-grant-ibigroup
Copy link
Contributor

I think we can probably get around this by increasing the timeout between report generation time and report collection time but should we maybe consider making the time dynamic based on file size? What do you think @philip-cline

@philip-cline
Copy link
Contributor Author

I think we should make it dynamic based on file size personally. Wouldn't want to wait the time it takes to validate MBTA for the results to come back for a tiny feed like Golden Gate

@miles-grant-ibigroup
Copy link
Contributor

Please let me know how #525 works for you!

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 a pull request may close this issue.

2 participants