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] Drescher 5 Beete (ungerade Zahl) turns in falsche Richtung generiert ? #522

Closed
andreasthu opened this issue Jan 14, 2022 · 17 comments
Labels
suggestion Idea/Feature Request
Milestone

Comments

@andreasthu
Copy link

Steps to reproduce
Drescher, Reihen pro Beet 5 (oder ungerade Zahl, 5, 7, ...)
fsScreen_2022_01_14_05_54_46

Actual behavior
nach dem ersten Beet bzw der 5. Reihe wendet der Drescher nach rechts um eine Reihe zuviel und muss dann anschließend immer nach rechts wenden um das Beet zu machen. somit ist das Abladerohr "immer in der Frucht"
fsScreen_2022_01_14_05_55_44

fsScreen_2022_01_14_05_56_01

fsScreen_2022_01_14_05_56_32

fsScreen_2022_01_14_05_56_44

Expected behavior
der Drescher sollte eine Reihe früher einlenken um am Ende der Reihe dann wieder nach links lenken zu können. somit wäre das Abladerohr wieder immer auf der abgeernteten Seite.

Screenshots
siehe oben

Courseplay version
Version used: aktuelles master/main (4f0ebb1)

Map
Haute Beyleron

Vehicles/ implements involved
Claas Trion + Corio 1275 C

Relevant script mods
CP

lg
andreas

@andreasthu andreasthu added the bug Something isn't working label Jan 14, 2022
@Tensuko
Copy link
Contributor

Tensuko commented Jan 14, 2022

Das liegt an der Aufteilung und sieht für mich erst mal nicht nach 'fährt in die falsche Bahn' aus.

@Tensuko Tensuko changed the title [BUG_SP] Drescher 5 Beete (ungerade Zahl) wendet nach erstem Beet in falsche Reihe [BUG_SP] Drescher 5 Beete (ungerade Zahl) turns in falsche Richtung generiert ? Jan 14, 2022
@andreasthu
Copy link
Author

andreasthu commented Jan 14, 2022

dann schau mal auf das letzte Bild:
hier wendet der Drescher nach rechts, d.h. das Abladerohr ist in der Frucht.
fsScreen_2022_01_14_05_56_44

würde er nach der 5. Reihe eine spur früher einlenken:
fsScreen_2022_01_14_05_55_44

würde er hier rauskommen wo er gerade reinfahren will
fsScreen_2022_01_14_05_56_44
und könnte nach links lenken, in die Bahn von wo er jetzt gerade kommt und hätte das Abladerohr auf der Gedroschenen seite
lg
andreas

@andreasthu
Copy link
Author

andreasthu commented Jan 14, 2022

hier noch der kurs vergrößert:
ist
das erste Beet (in gelb) ist korrekt, der Drescher wendet nach dem durchstich immer nach links, somit ist das Rohr immer Frei.
aber am ende des ersten Beetes fährt er eine Bahn zu weit (in weiß), daher muss er am ende des Durchstiches nach rechts lenken und somit ist das Rohr das ganze Beet auf der Fruchtseite und nie frei zum abtanken.

so sollte es hingegen sein:
sollte
wenn er hingegen eine Bahn früher einlenken würde, könnte er danach immer eine Linkswendung machen und das Abtankrohr wäre immer frei zum Abtanken.

lg
andreas

@Tensuko
Copy link
Contributor

Tensuko commented Jan 14, 2022

Das ändert nichts daran, dass der turn richtig funktioniert, nur der generierte Kurs falsch ist.
Es wird immer von grün (turn start) zu rot (turn end) gewendet.
Ich weis, dass du das wohl nicht oder nur schwer sehen kannst.
Daher ist es ein Generatorproblem und nicht vom turn.

@Tensuko
Copy link
Contributor

Tensuko commented Jan 14, 2022

Wir brauchen von genau dem kurs das logfile wenn der Kurs generiert wurde.

@Tensuko Tensuko added the waiting for user User answer needed label Jan 14, 2022
@andreasthu
Copy link
Author

andreasthu commented Jan 14, 2022

Hallo Tensuko.
Ja sry, der Turn ist ja eh richtig, hab mich ev unklar ausgedrückt.
Der Drescher fährt genau den generierten kurs nach.
Der Kurs sollte aber anders generiert werden.
Damit der Drescher auch im zweiten Beet nach links wendet und damit das Rohr immer innen ist.
Lg
Andreas

@pvaiko pvaiko added suggestion Idea/Feature Request and removed bug Something isn't working waiting for user User answer needed labels Jan 14, 2022
@pvaiko
Copy link
Contributor

pvaiko commented Jan 14, 2022

@andreasthu hier gibt es zwei Tabellen:

local rowOrderInLands = counterclockwise and

die die Reihenfolge angeben, abhängig von Uhrzeigersinn und Reihen pro Beet.

Ich habe keine Zeit drüber nachzudenken aber kann es sein, dass beim ungeraden Zahlen bei den Restreihen müssen wir den Urzeigersinn ändern? In deinem Fall nehmen wir counterclockwise, 5 für das Teil oben und dann counterclockwise, 4, aber clockwise 4 wäre das Richtige.

Wenn du mir ein Regel findest der mit allen Kombinationen funktioniert, kann ich implementieren.

@andreasthu
Copy link
Author

andreasthu commented Jan 14, 2022

hallo pvaiko
kann ich mir mal anschauen.
nach welchem system sind die tabellen aufgebaut, bzw wie kann ich die lesen/verstehen?
was mir da grade einfällt, wenn es mehr als zwei beete am fekd gibt, 3, 4 oder mehr, wie sich das dann weiter auswirkt.
werd mir das mal ansehen.
im ls19 gingen nur gerade anzahlen (2, 4, 6,... ) einzustellen

lg
andreas

@pvaiko
Copy link
Contributor

pvaiko commented Jan 14, 2022

{3, 4, 2, 5, 1}, heisst z.B. dass es 5 Reihen gibt, und erst wird Reihe 3, dann 4, dann 2, usw bearbeitet.

im ls19 gingen nur gerade anzahlen (2, 4, 6,... ) einzustellen

Deshalb ist das problem noch nie aufgetaucht. Wir können es auch wieder einstellen.

@Tensuko
Copy link
Contributor

Tensuko commented Jan 14, 2022

Brauchen wir denn wirklich 3 5 7 usw ? Hat das große Vorteile ?

@andreasthu
Copy link
Author

Hallo ihr zwei
ich habe mir mal einige gedanken gemacht.
@Tensuko
der vorteil bei 357... ist der das der Verbindungsweg zwischen Beet 1 und Beet 2 kürzer ist, das ist eigentlich schon alles. siehe auf der skizze (4) und (3).

20220115_094938

auf der skizze hab ich bei (1) angefangen.
wenn man immer 5 Reihen/Beet will, muss man jedes 2. Beet die Reihenfolge tauschen 32415 -> 34251 ->32415 -> 34251 ...
hat dann zw Beet 1->2 einen kurzen Verbindungsweg, zw. Beet 2->3 dafür einen langen , 3->4 wieder kurz, 4->5 wieder lang,...

wenn man aber nach 5 Reihen im ersten Beet eine wegnimmt (2) auf 4 Reihen/Beet oder eine dazugibt (3) auf 6 Reihen/Beet dann hat man immer einen kurzen Verbindungsweg.

aber eventuell ist es besser wieder wie im ls19 nur 2 4 6 8 ... Beete zuzulassen.
lg
andreas

@Tensuko
Copy link
Contributor

Tensuko commented Jan 15, 2022

Wow! Da hast du dir ja richtig gedanken gemacht.
Das ist echt super!

Ich glaube auch das es das einfachste ist, erst mal wieder auf gerade Zahlen zurück zu gehen.
Aber vlt kann man das hier als Referenz offen lassen, weil das ist wirklich gut gemacht.

@rohne83
Copy link
Contributor

rohne83 commented Jan 15, 2022

Das ist mir auch schon aufgefallen. Workaround wäre halt erstmal die Anzahl der Bahnen wieder wie im LS19 auf gerade Zahlen zu beschränken. Grundsätzlich interessant wäre es aber schon irgendwann auch wieder ungerade Zahlen zuzulassen sofern die dann auch richtig abgearbeitet werden (siehe Andreas' Skizzen).

Bin mir nicht sicher ob es so gut ist, die Arbeitsrichtung an der Uhrzeigersinn-Einstellung festzumachen. Vielmehr sollte eher allein die Rohrseite entscheidend sein (übrigens nicht nur bei Beeten sondern auch bei spiralförmig wo wir damit ja immer noch Probleme haben). Erstes Vorgewende wird im RL zB tatsächlich meistens gegen den Uhrzeigersinn gedroschen, also mit Rohr über der Frucht. Im 19er ließ sich das mit CP zumindest bei Drescherverbänden auch recht problemlos umsetzen. Klappt aber halt nur wenn die Vorgewende-Richtung keinen Einfluss auf die Arbeitsrichtung in den Beeten/Spiralen hat.

Btw, ist es gewollt dass es hier keine Diskussions-Sektion mehr gibt? Wollte nämlich auch mal noch ein paar Sachen grad zum Thema Dreschen mit euch und der Community teilen. Ob und was davon dann tatsächlich auch mal sich in CP als Funktionalität wiederfindet, steht auf einem anderen Blatt. Deswegen wollte ich auch kein Issue für aufmachen, aber interessant wäre es sicherlich.

@pvaiko
Copy link
Contributor

pvaiko commented Jan 15, 2022

Bin mir nicht sicher ob es so gut ist, die Arbeitsrichtung an der Uhrzeigersinn-Einstellung festzumachen.

Wenn du den Code anguckts wird klar dass es hier nicht um die Uhrzeigersinn-Einstellung aus dem Generator geht. Uhrzeigersinn hier ergibt sich genau aus der Rohrseite.

@rohne83
Copy link
Contributor

rohne83 commented Jan 15, 2022

Danke für die Aufklärung. Dann bin ich ja beruhigt ;)

@Tensuko
Copy link
Contributor

Tensuko commented Jan 15, 2022

Btw, ist es gewollt dass es hier keine Diskussions-Sektion mehr gibt?

Ist offen.

@andreasthu
Copy link
Author

Bin gespannt was da noch alles rauskommt.
Wenn noch was gebraucht wird, lasst es mich wissen.
Lg
Andreas

@Tensuko Tensuko added this to the 7.1.1.6 milestone Nov 2, 2022
Tensuko added a commit that referenced this issue Nov 6, 2022
- fixed a bug where combines called full unloaders.
- threshold can now be lowered to 40% for auger wagons with a low max fill level when weight limit is enabled.
- add option (keybind) for course editor to delete all waypoints untill the end.
- fix for #2044
- fix for #2045
- fix for #2037
- fix for #828 #1306 #2054
- fix for #522 #2060
- partial fix for #1949
@Tensuko Tensuko closed this as completed Nov 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
suggestion Idea/Feature Request
Projects
None yet
Development

No branches or pull requests

4 participants