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

Check validity of instances #51

Closed
leonlan opened this issue Feb 5, 2023 · 2 comments
Closed

Check validity of instances #51

leonlan opened this issue Feb 5, 2023 · 2 comments
Labels
enhancement New feature or request long-term

Comments

@leonlan
Copy link
Member

leonlan commented Feb 5, 2023

Except for some edge weight format and types, the current code does not check if the instance data are correct. For example, time windows can be checked for early <= late. Or that all data sections should have enough entries.

As of now, the library is just really about reading existing instances, so there should be no need to checking validity of the instances. But at some point it might be useful to include this as well, combined with #34 and #14.

@leonlan
Copy link
Member Author

leonlan commented Feb 6, 2023

We should be slightly careful with this, because it depends on the specific problem formulation to define whether an instance is valid or not.

@leonlan
Copy link
Member Author

leonlan commented Feb 10, 2023

I have decided that this and #27 is not something we will support. Our library focuses on input/output of VRPLIB instances. Validating the instance content is outside the scope of this library.

@leonlan leonlan closed this as completed Feb 10, 2023
This was referenced Feb 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request long-term
Projects
None yet
Development

No branches or pull requests

1 participant