Table View Doesn't Properly Update when Adjusting Altitude in the Vertical View
Issue #582
resolved
When an aircraft performance file is loaded, adjusting the altitude in the side view does not update ‘Leg time’, ‘Cum. time’, ‘Time’, ‘Rem. fuel’, etc. These should be updated if a performance file contains the proper entries in the “climb”, “descent”, and “cruise” fields. The performance file is attached for reference.
Comments (13)
-
-
That is probably a simple fix. Do you need it urgently, i.e. do we need a patch to stable or can it wait till the next “regular” release?
-
-
assigned issue to
Joern Ungermann
-
assigned issue to
-
reporter
It's not urgent, but will be good to have before ACCLIP next year (July-August, 2021).
-
That shouldn’t be a problem
I’ll prepare the fix until end of Octobre for sure. -
This also affects changing flightlevels in TableView.
-
@Jens-Uwe Grooß It seems that MSS has a documentation bug. The code assume altitudes in feet, the documentation says m…
-
@shawn That feature has, obviously, not been tested enough. I will change at least that. Could you maybe verify that a JSON file, with altitudes in feet would work as expected?
-
reporter
Hi Jorn,
Are the altitudes in the aircraft performance JSON file supposed to be in feet? If so, that could definitely be my issue since I have them in meters in my current version (see attachment). If that is the case, then it's likely that I just based our flight profile JSON file off of an outdated one.
-
Hi Shawn, did altitude in feet work?
-
I also looked in our sample files, and they seem fine. Some of our older json files have wrong units, though. We should probably tidy up those that we can get our hands on to avoid such errors in the future.
-
As I think the error was caused by an outdated version of a JSON file using wrong comments I close this issue as no action is necessary on the repository.
-
- changed status to resolved
No action necessary as documentation and code are working properly. An old JSON file with a wrong comment caused the issue.
- Log in to comment
@Joern Ungermann do you want to take this ?