Skip to content

Adjust vehicle maneuvering parameters for unloading - #1355

Open
shortcircuit412 wants to merge 1 commit into
Courseplay:mainfrom
shortcircuit412:main
Open

shortcircuit412 wants to merge 1 commit into
Courseplay:mainfrom
shortcircuit412:main

Conversation

@shortcircuit412

Copy link
Copy Markdown
Contributor

Increased minimum distance when moving out of the way, safe maneuvering distance, maximum direction difference, and course extension length for better vehicle handling.

  • higher chance to reach specialized harvesters - small working widths and long vehicles. with a bigger turn radius.

Increased minimum distance when moving out of the way, safe maneuvering distance, maximum direction difference, and course extension length for better vehicle handling.

-- when moving out of way of another vehicle, move at least so many meters
AIDriveStrategyUnloadCombine.minDistanceWhenMovingOutOfWay = 5
AIDriveStrategyUnloadCombine.minDistanceWhenMovingOutOfWay = 10

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can be a very big distance, I would love the see an example.

-- when moving out of way of another vehicle, move at most so many meters
AIDriveStrategyUnloadCombine.maxDistanceWhenMovingOutOfWay = 25
AIDriveStrategyUnloadCombine.safeManeuveringDistance = 30 -- distance to keep from a combine not ready to unload
AIDriveStrategyUnloadCombine.safeManeuveringDistance = 35 -- distance to keep from a combine not ready to unload

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine I guess.

AIDriveStrategyUnloadCombine.maxFruitPercent = 10
AIDriveStrategyUnloadCombine.proximitySensorRange = 15
AIDriveStrategyUnloadCombine.maxDirectionDifferenceDeg = 35 -- under this angle the unloader considers itself aligned with the combine
AIDriveStrategyUnloadCombine.maxDirectionDifferenceDeg = 15 -- under this angle the unloader considers itself aligned with the combine

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be a not so good Idea.
It depends on how well the unloader gets to the combine, else it will restart the aligment. For a moving combine not that big of a deal.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Been using that values for some time, unloaders so not crash into harvesters anymore. Overall an improvement for me at least, If you are too close to headlands/field borders or hedgerows one may need to alter course Gen. Lots of issues with non-grain harvesting vanished

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Okay, will give this a try then.

AIDriveStrategyUnloadCombine.maxDirectionDifferenceDeg = 15 -- under this angle the unloader considers itself aligned with the combine
-- Add a short straight section to align with the combine's course in case it is late for the rendezvous
AIDriveStrategyUnloadCombine.driveToCombineCourseExtensionLength = 10
AIDriveStrategyUnloadCombine.driveToCombineCourseExtensionLength = 20

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It needs a lot of space and could be an Issue at the start of a row and in some other situations.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, may be a bit too much. One vehicle lenght should be even more than 20m usually

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.

2 participants