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

Problems with newer FlightGear versions #306

Closed
HHS81 opened this issue Nov 28, 2018 · 9 comments
Closed

Problems with newer FlightGear versions #306

HHS81 opened this issue Nov 28, 2018 · 9 comments

Comments

@HHS81
Copy link
Owner

HHS81 commented Nov 28, 2018

According mailing-list JSBSim is now more verbose about some things. That means: our c182s isn`t working anymore with FlightGear versions from today and following.

https://sourceforge.net/p/flightgear/mailman/message/36479072/

I get an error like that:
bug

No idea what it means.....

@HHS81
Copy link
Owner Author

HHS81 commented Nov 28, 2018

and also described here with possible solution: c172p-team/c172p#1241

@HHS81
Copy link
Owner Author

HHS81 commented Nov 28, 2018

I could solve this by deleting the product-tags.
Still left following:

In file E:/AirCraftAddon/c182s/Systems/cabin-heat_and_air.xml: line 366
only has one argument which makes it a no-op.
Its argument will be evaluated but will not be applied to the result.

In file E:/AirCraftAddon/c182s/c182s.xml: line 831
only has one argument which makes it a no-op.
Its argument will be evaluated but will not be applied to the result.

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 28, 2018

Those two are invalid JSBSim tag pairs. I'm mobile right now but can look when I get home.
See Dev mil list...

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 28, 2018

The heat one is waiting for @onox to sort out to make sure we get it right.

@HHS81
Copy link
Owner Author

HHS81 commented Nov 28, 2018

@wlbragg Ah, I see. So <sum> needs at least two arguments.

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 28, 2018

The other one may be an unnecessary product tag.
After looking at it, yes it is the <sum> with only one property.
Which is either a mistake and is missing the other property/s. Or is correct and just needs the "sum" tag removed.

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 28, 2018

So <sum> needs at least two arguments.

Yes.

@wlbragg
Copy link
Collaborator

wlbragg commented Nov 28, 2018

@onox verified we can remove the <sum> tag pair from the
E:/AirCraftAddon/c182s/Systems/cabin-heat_and_air.xml: line 366
error warning as well.

HHS81 added a commit that referenced this issue Dec 9, 2018
Signed-off-by: Heiko Schulz <Heiko.H.Schulz@gmx.net>
@HHS81
Copy link
Owner Author

HHS81 commented Dec 9, 2018

All fixed in Master!

@HHS81 HHS81 closed this as completed Dec 9, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants