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

Wrong sign in control surface hingLineRotation for port-side wing #906

Open
joergbrech opened this issue Jul 8, 2022 · 1 comment
Open

Comments

@joergbrech
Copy link
Contributor

According to the CPACS documentation the hingeLineRotation of a control surface path is defined to be mathematically positive around the vector pointing from the inner hinge point to the outer hinge point:

The deflection path of the control surface is defined within the hinge line coordinat system. This is defined as follows: The x-hinge coordinate equals the wing x-axis. The y-hinge coordinate equals the hinge line axis (see above; positive from inner to outer hinge point). The z-hinge line is perpendicular on the x-hinge and y-hinge coordinate according to the right hand rule. The rotation of the control surface is defined as rotation around the positive y-hinge line.

For a starboard trailing edge device, a positive hinge line rotation means the device is rotated downwards for descend and for a port wing it would be the other way around: A positive hinge line rotation means the device is rotated upwards for ascend.

If a port-side wing is defined in CPACS (i.e. it is not a mirror of the starboard wing using the symmetry attribute) the hinge line rotation in TiGL has the wrong sign.

I suppose the error is somewhere in CTiglControlSurfaceTransformation::CTiglControlSurfaceTransformation.

Thanks @CLiersch for the find.

@joergbrech
Copy link
Contributor Author

joergbrech commented Sep 30, 2022

I suppose this is related both to the ongoing discussion about symmetry in CPACS, see DLR-SL/CPACS#609 and the discussion on coordinate systems in TiGL, see #911.

If we follow the CPACS definition precisely (which of course we should), we have a different sign of the hingeLineRotation for mirrored wings and non-mirrored wings on the port-side. This could be surprising for a user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant