Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

Impossible maneuver #3421

Closed
GeorgeEarslight opened this issue Mar 16, 2019 · 10 comments
Closed

Impossible maneuver #3421

GeorgeEarslight opened this issue Mar 16, 2019 · 10 comments
Assignees

Comments

@GeorgeEarslight
Copy link
Contributor

v138

CP_3421_138

Course F18ToDO, maneuver from 2nd to the 3rd row.
https://we.tl/t-KYktLW2To3

@Tensuko
Copy link
Contributor

Tensuko commented Mar 17, 2019

A File of 2MB you can easy drag and drop into the message window and upload here on github.... No need to use weird upload pages.

@GeorgeEarslight
Copy link
Contributor Author

@Tensuko wetransfer a weird upload page? You need to rethink that or ask your work colleagues about it.

@Tensuko
Copy link
Contributor

Tensuko commented Mar 17, 2019

why my work colleagues ? you don't know anything about me or my work...
And I never heard about that wetransfer so....

@Messy71
Copy link

Messy71 commented Mar 18, 2019

@GeorgeEarslight if you’ve created your course in HUD ( on that same settings page from the picture) try creating it with the advanced course generator ( in the same settings page you have the gear icon on the top right, near the generate and save course buttons) see if that works.
The HUD course generation it’s outdated and at the moment the devs are advising the use of the advanced course generator.

@GeorgeEarslight
Copy link
Contributor Author

@GeorgeEarslight if you’ve created your course in HUD ...
The HUD course generation it’s outdated and at the moment the devs are advising the use of the advanced course generator.

@Messy71 It's on the advanced generator. If you look closely, I've also used the skip rows option, which is only available on the advanced mode.

@pvaiko
Copy link
Contributor

pvaiko commented Mar 18, 2019

@GeorgeEarslight that upload site is indeed disturbing, not going to download anything from there. Please upload to GitHub.

@GeorgeEarslight
Copy link
Contributor Author

@pvaiko Is this a joke?
https://www.alexa.com/siteinfo/wetransfer.com
https://reviews.financesonline.com/p/wetransfer/
https://www.trustpilot.com/review/www.wetransfer.com
I'm genuinely curious about the reason you find it disturbing unless you're just toying.

Anyhow, here's the zip.
CP_3421_138.zip

@pvaiko
Copy link
Contributor

pvaiko commented Mar 18, 2019

No joke, I clicked on it twice both times I got a full page stupid animated ad in my face.

I'm genuinely curious about the reason to upload a file anywhere else other than the platform (GitHub) we are already using to track issues.

@GeorgeEarslight
Copy link
Contributor Author

I never got any ads from there, perhaps due to my adblocker.
As for my reason, is because in my use case it's just one extra click and I've been using it for work and non-work related big file transfers for years, aka it's working for me.

@vertigoogle
Copy link

Maybe you employer wants to hear about which external databases you feed with work-related data ;)

@pvaiko pvaiko self-assigned this Mar 22, 2019
pvaiko pushed a commit that referenced this issue Mar 24, 2019
Omitted `turnDiameter` x offsets from the first quarter turn. 
Appears to now generate the intended course for 180-90-90 and 90-90-180 maneuvers.
pvaiko added a commit that referenced this issue Mar 24, 2019
Omitted `turnDiameter` x offsets from the first quarter turn.
Appears to now generate the intended course for 180-90-90 and 90-90-180
maneuvers.
@pvaiko pvaiko closed this as completed Mar 24, 2019
pvaiko added a commit that referenced this issue May 27, 2019
Reverted "Fix for lane transition maneuvers #3421 (#3459)"

This reverts commit 4de84e9.

Fixed it properly, that commit worked by accident but the real problem
was that the original code assumed that we never skip a row and used
to work width instead of the delta X.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants