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

Websocket management issues? #13

Closed
BigCow74 opened this issue Mar 28, 2022 · 4 comments
Closed

Websocket management issues? #13

BigCow74 opened this issue Mar 28, 2022 · 4 comments
Assignees

Comments

@BigCow74
Copy link

Apologies for opening an issue for something I'm not sure is an issue, but I did not know how else to ask this question or contact you.

Do you know if this CJ4 implementation suffers from the performance degradation bug that FBW recently fixed? Link below.

flybywiresim/aircraft#6947

@dementedmonkey
Copy link
Owner

This will only be an issue if you install this mod and fly the CJ4 without running the websocket server.

It's probably a bigger issue for the A320 since it's a part of the base mod and you might not be using the external MCDU, where this mod is specifically for the external MCDU, you wouldn't install it if you weren't going to use it.

I see what FBW is doing, I'll see if I can put a similar feature on the Sim setup screen in the CJ4.

@BigCow74
Copy link
Author

Thanks. Yes, my question was for the situation where I have your CJ4 mod in my community folder, but decide to not launch the corresponding server app. Also, CJ4 flights are typically shorter so less likely to arise I suppose.

Thanks for the mod, it's very useful in the CJ4.

@dementedmonkey dementedmonkey self-assigned this Mar 30, 2022
@dementedmonkey dementedmonkey added this to the 0.4.0 milestone Mar 30, 2022
@dementedmonkey
Copy link
Owner

It looks like the underlying issue will be fixed by MSFT:

https://forums.flightsimulator.com/t/sim-update-10-beta-release-notes-1-27-9-0/528823

Fixed a bug where Javascript WebSockets were not closed and released properly when connection failed

Repository owner deleted a comment from frankracis-work Sep 22, 2022
@dementedmonkey
Copy link
Owner

SU10 is released, and the issue shows as fixed

https://forums.flightsimulator.com/t/release-release-notes-1-27-21-0-sim-update-10/543606

Since there's no longer a need to work around their bug, I will close this ticket.

@dementedmonkey dementedmonkey removed this from the 0.4.0 milestone Sep 22, 2022
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

2 participants