Missing skill or step out of reach

[mebymyself]

When response is "Missing skill or step out of reach for vehicle" how to know which job is causing this error?
Is it possible to add a JobId to the error message?

[jcoupey]

That would be useful additional information indeed. It would not be a problem since the error results from a check where we have both vehicle and job ranks at hand:

vroom/src/algorithms/heuristics/heuristics.cpp

Lines 925 to 928 in e6a92c0

	if (!input.vehicle_ok_with_job(v, job_rank)) {
	throw InputException("Missing skill or step out of reach for vehicle " +
	std::to_string(vehicle.id) + ".");
	}

Happy to review a PR adding this.

[mebymyself]

@jcoupey I tried to submit pull request but seems like I am missing permissions.
I have created patch with all changes, Would you apply it, please?

patch.zip

[jcoupey]

Thanks for trying to change this!

By default, you don't have push permission to the repo, so the usual workflow is to push your patch to a branch on your own fork, then create the PR from there.

[mebymyself]

@jcoupey Hello, is it possible to give some priority to this issue? The PR is submitted
Thanks

[jcoupey]

Done in #952