Rewrite in Lua for FlyWithLua Plugin by samrq · Pull Request #13 · schenlap/winwing_fcu

samrq · 2025-02-19T18:50:12Z

Removed dependency on libusb.

Implemented the functionality using Lua for better integration with FlyWithLua.

Tested on the default A330-300 model, with the following results:

[Almost] All buttons, LCDs, and LEDs are functioning correctly.

Some LCDs related to autopilot alt ctrl require further testing.

Fix some format problem for spd and hdg

schenlap · 2025-02-20T07:25:15Z

Hi,

thank you for your pr. Everything that make is easier to use for all people is welcome.
On what platform did you test it?

There are a few things in it I already began working:

Use mcu for laminar A330 and other: https://github.com/schenlap/winwing_fcu/tree/aircraft_plugin
Dropping libusb: https://github.com/schenlap/winwing_fcu/tree/hid_test

Why did you drop support for EFIS L/R and Toliss airplanes? Switching to lua should be one pull-request and new features a extra one. Dropping features can not be merged.

samrq · 2025-02-20T16:04:18Z

HI
Thanks for your reply.
I tested it on my M-chip Macbook with Xp12 and FlyWithLua NG+ v2.8.9.
no extra dependencies was introducd, so I think it could run on mac/win/linux as long as you installed flywithlua.
I'm not intend to remove toliss airplane and efls module. I just build and test this script with what I have :).
I'm not very farmiliar with pr procedure and i hope it won't bother you

schenlap · 2025-02-21T10:34:38Z

I'm not very farmiliar with pr procedure and i hope it won't bother you

No. Maybe flywithlua is a good idea. Do you want wo work on it further or should I add Toliss an EFIS?

schenlap · 2025-02-21T10:42:54Z

+cache_data["autopilot_spd"] = 0
+cache_data["autopilot_spd_is_mach"] = 0
+cache_data["autopilot_hdg_mag"] = 0
+cache_data["autopilot_alt"] = 0
+cache_data["autopilot_vs"] = 0
+cache_data["autopilot_fpa"] = 0
+cache_data["autopilot_ap1"] = 0
+cache_data["autopilot_ap2"] = 0
+cache_data["autopilot_athr"] = 0
+cache_data["autopilot_appr"] = 0
+cache_data["autopilot_loc"] = 0
+cache_data["autopilot_spd_window"] = 0
+cache_data["autopilot_fpa_window"] = 0
+cache_data["autopilot_hdg_window"] = 0
+cache_data["autopilot_trkfpa"] = 0
+cache_data["autopilot_alt_mode"] = 0 


is cache_data needed in lua? What I had seen it is possible to map variables direct to datarefs.

cache_data is used to check whether the values of these datarefs have changed. Otherwise, the LCD and LED would be redrawn in every loop

Perhaps I should rename this variable to clarify its purpose

schenlap · 2025-02-21T10:45:29Z

+        set_button_assignment(821, "sim/autopilot/vertical_speed_down")
+        set_button_assignment(822, "sim/autopilot/vertical_speed_up")


shouldn't it be done during init?

this code is outdated
I have indeed attempted to directly bind buttons to achieve these functionalities, aiming to minimize the reliance on more low-level interfaces, but it did not take effect. I am still working on resolving this issue. Moreover, I am uncertain whether the button IDs might change across different machines, which could potentially lead to additional development and configuration efforts

schenlap · 2025-02-21T10:45:32Z

+        set_button_assignment(821, "laminar/A333/autopilot/fpa_decrease")
+        set_button_assignment(822, "laminar/A333/autopilot/fpa_increase")


shouldn't it be done during init?

schenlap · 2025-02-21T10:48:53Z

+    local button = {}
+    for index = 0,32 do
+        local mask = 2^index
+        local bit = math.floor(event_bit/mask)
+        local bit_last = math.floor(last_event_bit/mask)
+        if (bit ~= bit_last) then
+            if (bit % 2 == 1) then
+                local id = index+1
+                button_press_event_list[index]=1
+            end
+        end
+    end
+    last_event_bit = event_bit


is this necessary when using hid? Why not work with button assignment instead?

schenlap · 2025-02-21T10:49:26Z

+function find_fcu()
+    for i = 1,NUMBER_OF_HID_DEVICES do
+        local device = ALL_HID_DEVICES[i]
+        if ((device.vendor_id == 16536) and (device.product_id == 47888) )


please in hex :-)

samrq · 2025-02-21T11:13:49Z

I'm not very farmiliar with pr procedure and i hope it won't bother you

No. Maybe flywithlua is a good idea. Do you want wo work on it further or should I add Toliss an EFIS?

If you're interested and have some free time, feel free to add any features you want to this script. After all, my work also relies on what you've done before. Maybe when I get new device and aircraft models, I can work on related stuff too, but I’m not sure about that yet.

schenlap · 2025-02-21T12:47:55Z

I tested you script now on my linux system. It is very slow. After turning a knob it takes 3 seconds to change the value, this is not usable. Does it work better on your setup?

samrq · 2025-02-21T13:14:15Z

I tested you script now on my linux system. It is very slow. After turning a knob it takes 3 seconds to change the value, this is not usable. Does it work better on your setup?

This script runs smoothly on my MacBook. I haven't tried it on a Linux platform yet, but I think it might be related to the HID interface. The hid_read in the script is set to a blocking mode with a 10ms timeout., if the device not respond, the loop will be blocked. You could use logMsg in on_button_event to see what slow down the loop. I'll try to fix this issue later

update :
i found a bug in last commit
** do_often(find_fcu())**
it should be
do_often("find_fcu()")

try to use the lasted commit and see if the promble still exist
sorry for this silly mistake
quite confused it still works in my computer!

schenlap · 2025-02-21T16:40:22Z

No, it did not change anything. I added logMsg in line 159 (in function on_button_event) and I see it get's read more often than I get the button event. After more than 20 calls I see that I presses a button.

Another problem is that often it does not connect, I get this output:

[001173] FlyWithLua Info: Start loading script file X-Plane 12/Resources/plugins/FlyWithLua/Scripts/fcu_driver.lua
[001174] FlyWithLua Info: Finished loading script file X-Plane 12/Resources/plugins/FlyWithLua/Scripts/fcu_driver.lua
[001175] FlyWithLua Info: All script files loaded successfully.
[001176] FlyWithLua Info: Loading time for all scripts is 0.012008 sec.
[001177] FlyWithLua Error: Wrong arguments to function hid_read().

samrq · 2025-02-21T19:13:11Z

No, it did not change anything. I added logMsg in line 159 (in function on_button_event) and I see it get's read more often than I get the button event. After more than 20 calls I see that I presses a button.

Another problem is that often it does not connect, I get this output:
[001173] FlyWithLua Info: Start loading script file X-Plane 12/Resources/plugins/FlyWithLua/Scripts/fcu_driver.lua
[001174] FlyWithLua Info: Finished loading script file X-Plane 12/Resources/plugins/FlyWithLua/Scripts/fcu_driver.lua
[001175] FlyWithLua Info: All script files loaded successfully.
[001176] FlyWithLua Info: Loading time for all scripts is 0.012008 sec.
[001177] FlyWithLua Error: Wrong arguments to function hid_read().

Can you check which processes are opening the FCU deivce at the same time ？
stop other processes that try to access this device

well. On my Mac, if I don’t run any plugins, X-Plane will recognizes the FCU just fine. When I press buttons, I can see button icon blink in the joystick settings page. But as soon as a plugin invoked hid_open, the device stops being registered in X-Plane, and the icon doesn't blink anymore when I press button.

Similarly, if I run sudo python3 ./test_endpoint.py after starting X-Plane, I will see this message in the X-Plane console:
UNREGISTER Joystick device: WINWING FCU-320 - VID:16536 PID:47888.

The issue you reported makes me suspect that the plugin and another process are competing for the device, causing intermittent data reception (like packet loss).

To be honest, my knowledge of HID devices is limited, and I don’t have a Linux device on hand. l'm sorry to say that I can't provide too much help. Everything I’ve said so far is just a guess，I hope this can give you some ideas to solve the problem, rather than leading you in the wrong direction

samrq · 2025-02-22T08:55:01Z

Try using the latest commit
I just removed hid_read and use builtin button support
hid_write only for led and lcd
and you may need to set the FCU_BUTTON_BEGIN depends on you own computer's setting.

you can get this id by click Plugin - FlyWithLua -FlyWithLua Macros - show joystick button numbers and then press "SPD MACH" on fcu(!! stop this plugin before you do this. it will conflict)

schenlap · 2025-02-23T08:02:10Z

When starting the macro for joystick button I get:

FlyWithLua Error: Can't execute Lua chunk. The chunk who failed is: DO_EVERY_FRAME_CHUNK
FlyWithLua Debug Info: The Lua stack contains the following elements:
...esources/plugins/FlyWithLua/Internals/button_sniffer.lua:8: attempt to call global 'button' (a table value)

and Lua stopped! at the top rigth corner.

samrq · 2025-02-23T08:21:20Z

When starting the macro for joystick button I get:

FlyWithLua Error: Can't execute Lua chunk. The chunk who failed is: DO_EVERY_FRAME_CHUNK
FlyWithLua Debug Info: The Lua stack contains the following elements:
...esources/plugins/FlyWithLua/Internals/button_sniffer.lua:8: attempt to call global 'button' (a table value)

and Lua stopped! at the top rigth corner.

I know what happend

it's a global variable button in fcu_driver.lua and it shadows the "button" function in button_sniffer.lua :(

fixed.

sry. I'm a fresher in lua too

schenlap · 2025-02-23T08:36:45Z

Thanks, the macro works now
It ist much besser now, but far from my python script (which is also not optimal). Changing the heading from 0 to 180 takes about a minute with lua.

samrq · 2025-02-23T08:56:13Z

Thanks, the macro works now It ist much besser now, but far from my python script (which is also not optimal). Changing the heading from 0 to 180 takes about a minute with lua.

Okay, maybe different platforms require different implementation approaches. If I have time later, I'll see if I can deploy a Linux system on my old laptop to reproduce and resolve this issue. I've uploaded a demo video showing the running effect on a Mac platform. Thank you for your review over these past few days.

schenlap · 2025-02-23T10:35:30Z

Your video looks very good.
I made one on my setup: https://tmpsend.com/mJ96sU7l

alha847 · 2025-04-11T15:26:27Z

Hi,

great work from you!

Just to let you know, I am trying to extend the lua script from samrq to work with EFIS R and EFIS L for the default LR A333. LEDs, switches and buttons are working right now, the two knobs and the LCD is still to do.

alha847 · 2025-05-04T12:39:26Z

Hi,

great work from you!

Just to let you know, I am trying to extend the lua script from samrq to work with EFIS R and EFIS L for the default LR A333. LEDs, switches and buttons are working right now, the two knobs and the LCD is still to do.

Hi, I have done some work on my script and published a preliminary version: https://github.com/alha847/winwing_fcu_efis_lr

Maybe its helpful for you.

init alt100/1000 from ww fcu

单若琦 and others added 7 commits February 14, 2025 03:27

a333

1e4851d

fix default A330-300

e0b8ca9

rewrite in lua as plugin

a46e312

test annunciator in A333

420d94c

reset

9fcddef

reset

c17deb9

remove

05f322a

samrq changed the title ~~Rewrite Script Using Lua for FlyWithLua Plugin~~ Rewrite in Lua for FlyWithLua Plugin Feb 19, 2025

samrq closed this Feb 20, 2025

loop find

15557f1

schenlap reopened this Feb 21, 2025

schenlap requested changes Feb 21, 2025

View reviewed changes

schenlap marked this pull request as draft February 21, 2025 12:48

samrq added 2 commits February 21, 2025 23:38

set hid nonblock

7744c43

fix bug

cf59146

remove hid_read use hid_write for led and lcd only

3362535

rename button and change FCU to a local variable to avoid conflict

7c085f9

samrq added 2 commits February 23, 2025 16:26

add local scope

6db7f62

refactor btn

6265909

add a demo video

e372b2d

alha847 added 5 commits April 16, 2025 22:08

Added automatic detection of winwing fcu, efis l, efis r

8bdf4d2

init alt100/1000 from ww fcu

b748998

Resolved merge conflict from init_switch

fc35d13

Updated hid_open after merge

faf7bc6

Added init of EFIS R switches

dadb285

Merge pull request #1 from alha847/init_switch

f9f623c

init alt100/1000 from ww fcu

schenlap mentioned this pull request Jun 27, 2025

Convert scripts to be executed through XPPython schenlap/winwing_mcdu#4

Closed

		set_button_assignment(821, "sim/autopilot/vertical_speed_down")
		set_button_assignment(822, "sim/autopilot/vertical_speed_up")

		set_button_assignment(821, "laminar/A333/autopilot/fpa_decrease")
		set_button_assignment(822, "laminar/A333/autopilot/fpa_increase")

Uh oh!

Conversation

samrq commented Feb 19, 2025

Uh oh!

schenlap commented Feb 20, 2025

Uh oh!

samrq commented Feb 20, 2025

Uh oh!

schenlap commented Feb 21, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

samrq commented Feb 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

schenlap commented Feb 21, 2025

Uh oh!

samrq commented Feb 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

schenlap commented Feb 21, 2025

Uh oh!

samrq commented Feb 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

samrq commented Feb 22, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

schenlap commented Feb 23, 2025

Uh oh!

samrq commented Feb 23, 2025

Uh oh!

schenlap commented Feb 23, 2025

Uh oh!

samrq commented Feb 23, 2025

Uh oh!

schenlap commented Feb 23, 2025

Uh oh!

alha847 commented Apr 11, 2025

Uh oh!

alha847 commented May 4, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

samrq commented Feb 21, 2025 •

edited

Loading

samrq commented Feb 21, 2025 •

edited

Loading

samrq commented Feb 21, 2025 •

edited

Loading

samrq commented Feb 22, 2025 •

edited

Loading