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

Rename NAV CRUISE mode to NAV COURSE HOLD #6297

Merged

Conversation

shellixyz
Copy link
Collaborator

@shellixyz shellixyz commented Nov 16, 2020

Suggested by Matey HV

CRUISE 2D -> COURSE HOLD, displayed on the OSD as "CRSH"
CRUISE 3D -> simply CRUISE, displayed on the OSD as "CRUZ"

More logical

@snaewe
Copy link
Contributor

snaewe commented Nov 17, 2020

Am I the only one reading 'CRSH' as CRASH ? Just wondering...

@CertainBot
Copy link

CertainBot commented Dec 22, 2020

Lol, yes thats not making a very optimistic abbreviation :-). I would rather stick with CRS for course and CRZ for cruise.

@shellixyz shellixyz force-pushed the improvement/cruise_to_course branch from 924cfd3 to b559d26 Compare April 6, 2021 18:22
@shellixyz shellixyz force-pushed the improvement/cruise_to_course branch from b559d26 to eecdd46 Compare April 6, 2021 20:07
@shellixyz
Copy link
Collaborator Author

We have voted, it will be CRSH for course hold (old cruise 2D) and CRUZ for cruise mode (old cruise 3D)

@shellixyz shellixyz merged commit 3aadc49 into iNavFlight:master Apr 6, 2021
@shellixyz shellixyz added this to the 3.0 milestone Apr 6, 2021
@shellixyz shellixyz added the Release Notes Add this when a PR needs to be mentioned in the release notes label Apr 6, 2021
@MrD-RC
Copy link
Collaborator

MrD-RC commented Apr 11, 2021

I have to say that when I read this comment page, I thought it was a good idea. It sounded like in configurator someone can either activate Course Hold or Cruise. That made perfect sense.

However, looking at the code, it looks like the box mode has just been renamed. I really don't think this works, especially to people new to iNav.
Cruise + Alt Hold = 3D Cruise made sense. You are taking a Cruise mode, and adding a lock to it to make it 3D.
Course Hold + Alt Hold = Cruise make no sense at all. There is no continuity between the mode names. I think this will really just confuse people. Especially new pilots.

Personally, I saw no problem with Cruise and 3D Cruise. Course Hold and Cruise does make more technical sense, so I'm totally fine with the renaming. But I think they should be two separate box modes, to be clear to users.

@shellixyz
Copy link
Collaborator Author

Cruise doesn't really make sense if it doesn't manage the throttle as it is just holding course. In aviation cruise is used to denote course hold + throttle control / alt hold. I guess we could create a new cruise mode which is just course hold + alt hold but then there will be two ways to do the same thing. It is just dead code. But maybe it is worth it as it might be less confusing for some users, no sure.

@avsaase
Copy link
Member

avsaase commented Apr 11, 2021

@shellixyz I think that would be the most straightforward solution to any confusion.

@MrD-RC
Copy link
Collaborator

MrD-RC commented Apr 11, 2021

@shellixyz I do think that would be the best solution, even if it means a little duplicated functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes Add this when a PR needs to be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants