HHS81 / c182s

Cessna C182S (1996 model) for FlightGear
GNU General Public License v2.0
27 stars 9 forks source link

Fix flap lever animation #33

Closed gilbertohasnofb closed 8 years ago

gilbertohasnofb commented 8 years ago

After https://github.com/HHS81/c182/issues/26#event-519605842 has been implemented, the lever must be repositioned and its animation must be redone (it should move a little bit sideways after each notch).

HHS81 commented 8 years ago

ah, I didn't know,

gilbertohasnofb commented 8 years ago

Maybe take a look on what onox did on the c172p: https://github.com/Juanvvc/c172p-detailed/commit/fb9a37c3187a1472c871ce9ff61ee043cfc12ef1

HHS81 commented 8 years ago

should be fixed

gilbertohasnofb commented 8 years ago

Heiko, I just checked it and there are some problems still:

HHS81 commented 8 years ago

-the "smooth" move is somewhere burried in the c172p-files, where I can't find it. Anyway, how smooth or instantaneously the flaps lever is moving depends on your hand and force you use in RL. Really something minor for me right now....That can come later. -they used the translate animation for up and down, but it seems the animation should be a rotate animation for lever and indicator according some videos on youtube- so I changed it back, but kept the translate sidewards of the lever -the flap lever in reality is a bit tilted, so I changed this to it

Push comes right now

gilbertohasnofb commented 8 years ago
HHS81 commented 8 years ago
gilbertohasnofb commented 8 years ago

:+1: I will ask onox about the smooth movement then!

onox commented 8 years ago

I simply used a <kinematic> in c172p.xml. See https://github.com/Juanvvc/c172p-detailed/pull/653/files#diff-9eb2c7f8235986e706c0aee4fb47fad6 at the bottom of the page.

gilbertohasnofb commented 8 years ago

Thanks, @onox !

HHS81 commented 8 years ago

@onox Many Thanks! Didn't thought about this, it's a easy one, I will use this soon as I can.

gilbertohasnofb commented 8 years ago

We can later apply this to the parking brakes lever as well ( @onox I think the c172p also needs this to be applied for the parking brakes, right?)

HHS81 commented 8 years ago

Yep, I even think they already did apply this....

HHS81 commented 8 years ago

Should have been fixed now

gilbertohasnofb commented 8 years ago

Really cool animation, Heiko! But if I may be picky, I think the flap isn't really resting in the lowest point of each notch. Try to observe this this: at 10 or 20 degrees, when you lower the flaps you will notice that it first goes a tiny bit down, then right, then down. I think the flap should be resting in the lowest position possible for that notch, so that the movement of lowering the flaps would be only right then down.

HHS81 commented 8 years ago

Now it should be fixed

gilbertohasnofb commented 8 years ago

Strange, retracting the flaps is perfect now, but extending is still a bit strange, there is just a tiny bit of downwards movement before the lateral movement.

HHS81 commented 8 years ago

and now?

gilbertohasnofb commented 8 years ago

Still not. Try this: press Shift+A so that the time compression is exactly at 1/8x (lower values make the flap movement go crazy, but this is enough): now try lowering the flaps from 10 to 20 then raising back. See how the movement when lowering is strange (some unwanted movement) while raising is fine? Same for 20 to 30.

HHS81 commented 8 years ago

I see what you mean, but lower slow motion values (1/16, 1/32, ...) breaks the whole animation. The slower, the more it seems. Andf I guess that's the reason for your strange observation. Bug in FGFS itself, I would say.

gilbertohasnofb commented 8 years ago

No Heiko, I can observe that without any slow motion, but the point is that at 1/8 you can observe better. But at 1x this issue is there!

gilbertohasnofb commented 8 years ago

The slow motion tip was just to make sure we are understanding each other on what I meant :wink:

HHS81 commented 8 years ago

according the table, it should not be the case. You should observe the same issue on the c172p, since I used the same principle

gilbertohasnofb commented 8 years ago

I will test it on the c172p after, but I don't remember seeing any such thing with its flap. But anyway, it's such a minor thing that I'd say we can just leave it as it is.

gilbertohasnofb commented 8 years ago

Heiko, the c172p flap lever movement is much smoother and clearer than ours here, but the difference is just so small that maybe we can just leave it as it is. I leave this up to you, so feel free to close this issue if you want to.

HHS81 commented 8 years ago

I will close it, if the difference is small, I won't spend any time now. Later maybe.....