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.

Sign up for GitHub

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

Jump to bottom

[BUG_SP] Lua Error beim Kurs generieren #417

Closed
MST050706 opened this issue Jan 7, 2022 · 27 comments
Closed

[BUG_SP] Lua Error beim Kurs generieren #417

MST050706 opened this issue Jan 7, 2022 · 27 comments
Assignees
@pvaiko
Labels
bug Something isn't working

Comments

@MST050706
Copy link

Steps to reproduce
Kurs generieren

Actual behavior
Wenn ich einen Kurs generiere dann kommt ein Lua error und es bewegt sich nur die Map im Hintergrund und es wird kein Kurs generiert. Mit der ersten Version ging dies Fehlerfrei.

Expected behavior
Das der Kurs normal generiert wird.

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

Courseplay version
Version used: V7.0.0.16

Log
log.txt

Map
Osina

Vehicles/ implements involved
Mod yes

Relevant script mods
VCA
@MST050706 MST050706 added the bug Something isn't working label Jan 7, 2022
@Tensuko
Copy link
Contributor

Tensuko commented Jan 7, 2022

Welche Version hat auf der Map noch funktioniert ?

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

Die aller erste

@MST050706
Copy link
Author

log.txt
Nocjmal die Log mit allen Debug Channels. Hatte ich vergessen sorry ;)

@Tensuko
Copy link
Contributor

Tensuko commented Jan 7, 2022

Und was ist für dich die aller erste ?

@MST050706
Copy link
Author

Die die ihr für den 22er releast habt.
v7.0.0.1

@Tensuko
Copy link
Contributor

Tensuko commented Jan 7, 2022

Da liegen 15 Versionen zwischen.... keine Idee ab wann es nicht mehr ging ?

@MST050706
Copy link
Author

Hatte es nie aktualisiert.

@MST050706
Copy link
Author

Also hab nur die 2 Versionen gespielt. kann es dir leider nich sagen.

@MST050706
Copy link
Author

UPDATE: Bei V7.0.0.10 gehts zu letzt

@Tensuko
Copy link
Contributor

Tensuko commented Jan 7, 2022

Ah super, danke dir fürs probieren!

@Tensuko Tensuko removed the waiting for user User answer needed label Jan 7, 2022
@pvaiko
Copy link
Contributor

pvaiko commented Jan 7, 2022

Das heisst, diese Map hat weniger als 10 cm zwischen Felder ...

@MST050706
Copy link
Author

Jo kein Problem ging ja schnell mit dem Testen.
Ahh ok aber das Feld geht ja noch in v7.0.0.10 und in 0.11 nicht mehr? Kann man das irgendwie ändern oder könnt ihr das in einer Version fixen?

@Geradsralf
Copy link

Geradsralf commented Jan 8, 2022 •
edited
Loading

Habe die gleiche Beobachtung gemacht. Habe große Arbeitsbreiten bei kleinen Feldern im Verdacht, den Fehler zu erzeugen. In 7_0_0_12 und 7_0_0_16 kann z.B. mit Düngerstreuer oder Spritze kein Vorgewende auf den Feldern 20, 64 und 65 und wahrscheinlich noch mehreren (Elmcreek) erstellt werden. In mindesten einer älteren Version war das auch schon mal als Fehler, kann aber nicht mehr genau sagen in welcher.
Wenn min. 1 Vorgewende ausgewählt ist, rastet die Map an den Mauszeiger beim Klick auf "Kurs generieren". Nächster Klick löst die Map wieder.
Stellt man dann Vorgewende auf 0, wird eine Route mit 1 Geraden oder 1 mal hoch und runter generiert. Dann klappt es.
Einen Log kann ich bei Bedarf beim nächsten mal liefern.

@MST050706
Copy link
Author

Ja genau sowas habe ich auch beobachtet

@Tensuko
Copy link
Contributor

Tensuko commented Jan 8, 2022

@Geradsralf
Mit 42m ein kleines Feld bearbeiten und dann noch mit Vorgewende macht man auch nicht... Was will man dann noch in der mitte ?

@Geradsralf
Copy link

Ja ist korrekt, passiert meistens bei der Abwicklung mehrerer Aufträge hintereinander wenn man mehrfach mit den gleichen Einstellungen einen Kurs generiert.
Grundsätzlich gibt es dann aber trotzdem immer Probleme mit dem Kurs ohne Vorgewende.
Mit der Feldspritze auf Feld 20 wurde z.B. ein Kursgeneriert, der weit außerhalb des Feldes die Wende vorsah. Die ging natürlich auf Grund der Bäume schief.
Auf 64 oder 65 mit dem Streuer wird eine einzige Bahn generiert, die einmal mit eingeschaltetem Streuer gefahren wird und dann aus irgendeinem Grund zurück gefahren wird ohne Streuer und noch einmal entlang des Kurses ohne Streuer gefahren wurde, bis schließlich am Endpunkt gehalten wird.
Da die Updates aber zum Glück schneller kamen als ich spiele, hatte ich noch keinen erneuten Auftrag zum Streuen auf dem Feld.

@Tensuko
Copy link
Contributor

Tensuko commented Jan 8, 2022

@Geradsralf Ich kann mit deinen Informationen rein gar nichts anfangen....
Feld 20 keine Ahnung was du meinst. Welche Fedlsprize, wie sah dein Kurs aus, hast du die Version in der die Feldrandberechnung verbessert wurde usw usw...
Kann bereits alles gefixt sein.
Und wie gesagt, keiner nutzt Vorgewende bei Feldern, wo man höchstens einmal rauf und einmal runter fährt...

@MST050706
Copy link
Author

Um jetzt nochmal auf den eigentlichen Bug zurückkommen. Dafür gibt es afaik kein Fix oder?

@Geradsralf
Copy link

Sorry, aber mein Post von vor ein paar Minuten bezieht sich doch erneut auf meinen Post, 2 Antworten vorher. Da stehen doch Versionen und Karte drin. (Elmcreek)
Version bei dem Fehler mit der Feldspritze war bereits 7.0.0.16.
Die genaue Bezeichnung der Feldspritze muss ich später erst nachschauen.

@Tensuko
Copy link
Contributor

Tensuko commented Jan 8, 2022

@MST050706
Weis nicht, 10cm zwischen den Feldern ist schon etwas wenig, kenne die Karte jetzt nicht und muss sagen wenn das wirklich so ist, sehe ich das Problem eher an der Karte...

@Geradsralf
Aber das alles hat nichts mit diesem Issue zu tun...
Häppchenweise Informationen inmitten eines andere Issues, damit kann man einfach nichts anfangen.
Auch keine Screenshots und logfiles....
Wenn das ein riesen Problem ist, dann mach bitte ein eigenes Issue auf.

pvaiko added a commit that referenced this issue Jan 8, 2022
@pvaiko
Field scanner improvement #417
75701bb 
This can't fix maps like Osina with very messy
field edges but it can handle them better, at least
when two fields which are too close to each other have
a different farmland ID configured.
@pvaiko
Copy link
Contributor

pvaiko commented Jan 8, 2022

So, mehr können wir momentan nicht machen, einige Felder funktionieren jetzt besser, aber die Feldgrenzen sind sehr unsauber auf dieser Map.

@MST050706
Copy link
Author

Danke dir dafür

@F4bl32
Copy link

F4bl32 commented Jan 8, 2022 •
edited
Loading

So, mehr können wir momentan nicht machen, einige Felder funktionieren jetzt besser, aber die Feldgrenzen sind sehr unsauber auf dieser Map.

Seit dem Fix gehen zusammengelegte Felder nicht mehr, da er nur noch die fieldID benutzt und bei zusammengelegten Feldern müssten es mehrere ID's sein vermute ich mal.

@MST050706
Copy link
Author

Das ist doof da ich viele Felder zusammen gelegt habe.

@pvaiko
Copy link
Contributor

pvaiko commented Jan 8, 2022

@F4bl32 dann weiss nicht wie ich das handeln soll.

@MST050706
Copy link
Author

Gut dann lass ich es erstmal. Wenn eine neu Map kommt wird es schon wieder gehen.

pvaiko added a commit that referenced this issue Jan 8, 2022
@pvaiko
Reverting field scanner change for #417
88b4c5e 
Reverting part of 75701bb as it won't work for merged fields.
Tensuko added a commit that referenced this issue Jan 9, 2022
@Tensuko
Push Release
f12298a 
- Another fix for cotton harvester #421
- Allow reverse pathfinding option re-enabled #102
- Field scanner improvement #417
- Set start job on first waypoint as default
- Set default keybinds for open mini hud and start/stop CP Driver
- Keybind for CP now also stops CP
- Disable last waypoint callback for combine headland turn (fix for Claas Dominator)
- Leftover reversing debugs removed #394
- Another imperial unit fix for #128
- Don't show pathfinder debug boxes #426
- Stop chopper only if it is harvesting #411
- Smal turn fix
- Headland to up/down transition fix #340
@Tensuko
Copy link
Contributor

Tensuko commented Jan 9, 2022

Sry but as we said, this might be partly solved in 7.0.0.17
Not much more we can do about it without breaking other things.
Maybe you can try to contact the mapper to fix his Fields.

@Tensuko Tensuko closed this as completed Jan 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pvaiko pvaiko

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@pvaiko @Tensuko @F4bl32 @MST050706 @Geradsralf