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

Vehicle length and weight not considered in autocost #4369

Open
FScholPer opened this issue Oct 30, 2023 · 0 comments · May be fixed by #4370
Open

Vehicle length and weight not considered in autocost #4369

FScholPer opened this issue Oct 30, 2023 · 0 comments · May be fixed by #4370

Comments

@FScholPer
Copy link

Hey all,
currently dimension of cars in autocost are only considered for width and height.

` switch (restriction.type()) {

case AccessType::kMaxHeight:
  return height_ <= static_cast<float>(restriction.value() * 0.01);
case AccessType::kMaxWidth:
  return width_ <= static_cast<float>(restriction.value() * 0.01);`

Vehicle length and weight is missing.

It would be a good feature to also evaluate both of them in autocost like it is in truckcost.
We can set small defaults for the vehicle like:

constexpr float kDefaultAutoLength = 4.0f; // Meters (157,4808 inches)
constexpr float kDefaultAutoWeight = 2.4f; // / Metric Tons (48,000 lbs)

Whats your opinion?

Frank Scholter Peres on behalf of Mercedes-Benz Tech Innovation GmbH.
Provider Information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants