Skip to content

Conversation

@BrettMayson
Copy link
Member

@BrettMayson BrettMayson commented Jun 8, 2019

When merged this pull request will:

  • Add additional speed units

    • km/h
    • mph
    • kn
    • m/s
  • Add additional altitude units

    • meters (m)
    • feet (f)
  • Display altitude to 1 decimal place below 10

@commy2
Copy link
Contributor

commy2 commented Jun 9, 2019

Is this worth it to have two (or more) PFH running each frame?
Can this be optimized to have no PFH running in case the settings match Arma default units?

@BrettMayson
Copy link
Member Author

It could be modified to use no PFH at the cost of complication. When in a land vehicle there will be a single PFH running, in aircraft there will be 2.

@commy2
Copy link
Contributor

commy2 commented Jun 9, 2019

Is the control actually deleted when leaving the vehicle, therefore is there the PFH only running when inside the vehicle and it stops when leaving the vehicle?

@Max255PL
Copy link
Contributor

Since this modifies the altitude display already, would it be possible to have both ASL and AGL measurements shown at the same time in all aircraft?
This was a feature back in ACE2 (along with a UI variometer), random pic showing this: https://i.ytimg.com/vi/oLnRu10thKY/maxresdefault.jpg
(ASL middle right)

@Max255PL
Copy link
Contributor

Found a bug with this, when controlling UAVs the UI is still getting the player position instead of the drone. Result as expected.

@commy2
Copy link
Contributor

commy2 commented Aug 20, 2019

All vehicle ACE_player should be cameraOn.

@BrettMayson
Copy link
Member Author

Since this modifies the altitude display already, would it be possible to have both ASL and AGL measurements shown at the same time in all aircraft?
This was a feature back in ACE2 (along with a UI variometer)

This PR is not really modifying the UI, and there is unfortunately not enough space to add these in. It's something I'd be interested in adding but it might be better suited for an aircraft orientated mod like AWEsome since it will require modifying the look of the UI to add room. At the very least it should be a different PR if it is to be added to ACE.

@jonpas jonpas modified the milestones: 3.13.0, 3.14.0 Dec 7, 2019
@BrettMayson BrettMayson requested a review from jonpas October 4, 2021 23:55
@BrettMayson BrettMayson requested a review from commy2 April 11, 2022 15:08
@PabstMirror PabstMirror modified the milestones: 3.15.1, Ongoing May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/enhancement Release Notes: **IMPROVED:**

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants