Describe the bug
The generated XML declares two fields per sample, FlightModel* and Pilot* (for example relay/src/update.rs:222-223). The relay fills both with the same number, cloning the single value it received (relay/src/state.rs:584-602).
The duplication originates in the relay, not the plugin. The plugin reads both the flightmodel and pilot datarefs and displays both, but transmits only the eight pilot values (xplane_plugin/src/pilotdatasync-xp11.cpp:418-428). The relay then fabricates the missing half.
Expected behavior
FlightModel* carries the flightmodel value and Pilot* carries the pilot value, so the two can be compared. Being able to see instrument versus true state is the reason there are two fields.
Additional context
The relay already has a generic branch that pairs values correctly (relay/src/state.rs:605-624), but it is unreachable because the 8-field branch matches first. The likely fix is widening the plugin payload to 16 paired values. Yaw is a known exception: no pilot yaw dataref exists, so the plugin deliberately uses the flightmodel value (xplane_plugin/src/pilotdatasync-xp11.cpp:138-139).
Found while writing the setup documentation in #138 / #178.
Describe the bug
The generated XML declares two fields per sample,
FlightModel*andPilot*(for examplerelay/src/update.rs:222-223). The relay fills both with the same number, cloning the single value it received (relay/src/state.rs:584-602).The duplication originates in the relay, not the plugin. The plugin reads both the flightmodel and pilot datarefs and displays both, but transmits only the eight pilot values (
xplane_plugin/src/pilotdatasync-xp11.cpp:418-428). The relay then fabricates the missing half.Expected behavior
FlightModel*carries the flightmodel value andPilot*carries the pilot value, so the two can be compared. Being able to see instrument versus true state is the reason there are two fields.Additional context
The relay already has a generic branch that pairs values correctly (
relay/src/state.rs:605-624), but it is unreachable because the 8-field branch matches first. The likely fix is widening the plugin payload to 16 paired values. Yaw is a known exception: no pilot yaw dataref exists, so the plugin deliberately uses the flightmodel value (xplane_plugin/src/pilotdatasync-xp11.cpp:138-139).Found while writing the setup documentation in #138 / #178.