Skip to content

Weather Addon#28

Open
Pontiac51 wants to merge 27 commits intomracko:masterfrom
Pontiac51:master
Open

Weather Addon#28
Pontiac51 wants to merge 27 commits intomracko:masterfrom
Pontiac51:master

Conversation

@Pontiac51
Copy link

@Pontiac51 Pontiac51 commented Jan 29, 2021

Hi,

as discussed I have added weather information as this is usually not available in MSFS unless you have ATIS or a glass cockpit with that information.

I have included the following parameters on the "Other" page:

Sea level pressure in inHg and mbar
Ambient pressure in inHg and mbar
Ambient temperature in °C and °F
Ambient visibility in meters (according to WMO standard) and miles

2 compass roses indicate wind direction and velocity in kts
First compass rose is north orientated
Second compass is heading orientated

Wind components (aircraft lateral and longitudinal aka crosswind, head- and tailwind) in kts

I hope you find this useful!

Pontiac51 and others added 27 commits January 27, 2021 10:24
simconnect_thread_func

line 313-317 added "Weather" fetch
added weather 
to initialisation line 107ff
to getSimulatorData line 694ff
to displayData line 776ff
added waether
to line 19ff
temperature description changed to °C
added aircraft wind x and z
added wind x and z
fixed aircraft wind direction
@mracko
Copy link
Owner

mracko commented Jan 29, 2021

Thanks for the pull request. I'll have a look at it over the next couple of days. Just so that you know - I'll be updating the app with a new map/controls split-screen mode for tablets in landscape mode. From what I've seen, this shouldn't have compatibility issues with your pull request. Maybe the menu_other.html will require manual fixing, but nothing complicated.

One important question: I've noticed that you've added 5 new vars for SimConnect to pull. I've been very cautious with adding new vars because with each addition the response time for data going from the sim to the app gets worse. I'm planning on adding other essential controls like flaps, gear, spoilers, trim wheel etc so I'm worried about having too many variables that would make the app painfully slow. Did you compare the response times of your version with the original?

Thanks!

@Pontiac51
Copy link
Author

Pontiac51 commented Jan 29, 2021

I did not specifically test this, but I was quite happy with the performance. We could leave one or two variables out if it is a concern. The ambient pressure and the sea level pressure (pressing B does do the same) are not really required, it is just a bonus.

Can’t wait to see the new version.

Cheers!

@Pontiac51
Copy link
Author

Would it be possible to pull only the variables the page has to display? That should help. A friend of mine suggested to use Websockets, but that would be a bigger overhaul.

@mracko
Copy link
Owner

mracko commented Jan 30, 2021

As far as I have seen, the performance/latency problem is due to Python SimConnect. Switching to native C++ SimConnect would solve the issue, but I have no C++ experience.

I have updated the master to v1.5 which add the landscape mode. You can switch to the landscape mode in the Others tab. I'll look at your code some time next week and give you feedback.

@mracko
Copy link
Owner

mracko commented Feb 5, 2021

I've been busy finishing v1.5 this week but I'll have a look at your weather additions in the next couple of days.

@mracko
Copy link
Owner

mracko commented Mar 30, 2021

Hi @Pontiac51 ! Just want to let you know that I haven't forgotten about the weather info. I've made a new Data tab where this info would make perfect sense. I haven't implemented it yet because I don't want to include too many variables unnecessarily. I'm thinking of adding a third thread for additional SimConnect vars to keep the latency under control.

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

Successfully merging this pull request may close these issues.

3 participants