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

[BUG_SP] penultimate point incorrect from AD to CP first WP in reverse #2788

Closed
andreasthu opened this issue Sep 21, 2023 · 5 comments · Fixed by #2792
Closed

[BUG_SP] penultimate point incorrect from AD to CP first WP in reverse #2788

andreasthu opened this issue Sep 21, 2023 · 5 comments · Fixed by #2792
Assignees
Labels
bug Something isn't working
Milestone

Comments

@andreasthu
Copy link

READ FIRST
https://github.com/Courseplay/Courseplay_FS22/wiki
ok

Steps to reproduce
Steps to reproduce the behavior.
mit einem 3punkt Gerät von AD zum 1. Wegpunkt imRückwärtsgang anfahren
z.b. Claas und der Einböck Striegel, aber auch Grubber, Sämaschinen,...

Actual behavior
A clear and concise description of what happen.
wenn AD zu CP übergibt und der 1. Wegpunkt angefahren wird, ist der Vorletzte Wegpunkt bei anfahren versetzt:
fsScreen_2023_09_21_08_26_50
fsScreen_2023_09_21_08_26_57
fsScreen_2023_09_21_08_26_50
fsScreen_2023_09_21_08_27_02
fsScreen_2023_09_21_08_27_09

hier sieht man gut, das der punkt"12" nicht in Linie ist und infolge den Traktor aus der Bahn bringt:
fsScreen_2023_09_21_08_27_12
fsScreen_2023_09_21_08_27_14
fsScreen_2023_09_21_08_27_17
fsScreen_2023_09_21_08_27_20
fsScreen_2023_09_21_08_27_22
fsScreen_2023_09_21_08_27_24

er erfängt sich zwar etwas, aber startet dennoch ca. 1m neben der Spur.
was bei geräten vom 3 oder 4m arbeitsbreite eher ungünstig ist:
fsScreen_2023_09_21_08_27_41

Expected behavior
A clear and concise description of what you expected to happen.
Korrektur des vorletzen Punktes oder deren Löschung

Screenshots
If applicable, add screenshots to help explain your problem.
siehe oben

Courseplay version
Version used:
aktuelles 7.3.1.0

Log
Please attach a log with all debug channels active.
For help with the debug channels,
please check out the readme at the github front page.
log.txt

Map
Map played on.
vulkan islands

Vehicles/ implements involved
Which vehicle or implement?
Mod yes/no ?
claas arion600
einboeck pneumaticstar900
KEINE modfahrzeuge

Relevant script mods
Mods like Autodrive or VCA and so on.
AD +CP

sorry das ich schon wieder arbeit mache
lg
andreas

@andreasthu andreasthu added the bug Something isn't working label Sep 21, 2023
@Tensuko
Copy link
Contributor

Tensuko commented Sep 21, 2023

Und gibt es ein Problem bei der Bearbeitung wenn er fertig ist mit dem Feld?...

@andreasthu
Copy link
Author

Hallo Tensuko.
Hatte es im mittlerweile geschlossenen Thread noch geschrieben, aber da liest ev keiner mehr:
‐-----
edit: (zu deiner Frage oben)
habe "Wenden auf dem Feld" aus weil diese einfach länger dauern, so komme ich mit 1-2 Vorgewenden aus, und auf dieser Map ist rund um die Felder genug Platz um zu wenden (auf der Frühling oder Neu-Minibrunn sieht es da anders aus).

Bei 1-2 vorgewenden dauert "Wende auf dem Feld" einfach ur lange. Überhaupt bei Geräten mit wenig Arbeitsbreite.
Nach dem anheben schiebt der Traktor zurück, wendet dann im Feld und kommt dann ein ganzes Stück im Feld wieder zu stehen um dann schlussendlich wieder zur Feldgrenze zurückzusetzen.
Bei größeren Arbeitsbreiten fällt das dann nicht mehr so ins Gewicht, aber bei 3-6m ist das schon jedesmal ein ordentlicher Weg.
Überhaupt dann, wenn rundherum sowieso genügend Platz zum Wenden ist.

Zum anderen, es gibt kein problem mit der Bearbeitung wenn er fertig ist, aber am Anfang bleibt auch wieder ein Stück unbearbeitet, da er ja schräg ausserhalb der generierten Spur beginnt.

Und ja, es tut mir leid wenn ich euch soviel arbeit mache.
Lg
Andreas

@Tensuko
Copy link
Contributor

Tensuko commented Sep 21, 2023

Wenden auf dem Feld an ist das exakt gleiche Wendemanöver für 3-Punkt Geräte und das dauert nicht länger.

grafik

Aber egal wir gucken das wir das gefixt kriegen.

Ach und doch das liest noch jemand, weil ich krieger über alles eine Mail Benachrichtigung, dauert halt nur bis ich eben in die Mails geguckt habe.

pvaiko added a commit that referenced this issue Sep 21, 2023
Do not add the first waypoint of the fieldwork course
to the alignment course as it is generated to reach it
anyway, no idea why I did that, but it was a problem
when the alignment course ends in reverse.

#2788
@pvaiko pvaiko linked a pull request Sep 21, 2023 that will close this issue
@Tensuko Tensuko added this to the 7.3.1.1 milestone Sep 21, 2023
@andreasthu
Copy link
Author

andreasthu commented Sep 21, 2023

Ah ok, das wusste ich nicht das das bei 3punkt das gleiche manöver ist.
Aber wenn ich dann z.b. den steinsammler, gülllefass oder Miststreuer drann habe sollte ich wenden am Feld dann ja wieder abschalten, und wieder aktivieren bei 3punktgeräten. Was hätte das für einen sinn dann bzw wäre das eine Einstellung die man dauernd ändern müsste?

Ich dachte nur, das bei closed beiträgen das nichtmehr kommt, bzw dachte ich das da selten wer nachschaut.

schönes ackern
Lg
Andreas

edit:
und sorry wegen der extrem großen schrift weiter oben,
hatte das am handy kopiert und eingefügt aber bekomms nicht mehr klein

@Tensuko
Copy link
Contributor

Tensuko commented Sep 21, 2023

Man sollte am besten immer die passende Einstellung für das Gerät verwenden.
Was es ffür einen Sinn hat? Dafür sind die Einstellungen da...
Aktuell halt pro Taktor gespeichert, es gibt Pläne solche Einstellungen pro Gerät zu speichern, aber die stehen aktuell sehr weit unten auf der TODO Liste.

Tensuko added a commit that referenced this issue Sep 23, 2023
- increse skip row for vine generator to 4 #2787 (but the Oxbo harvester is broken...)
- fix for #2788
- Added manure trailer unload target support for shovel mode.
- Added marker to select trailer unload space for shovel mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants