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

Ignore simulated position updates #54

Closed
wants to merge 1 commit into from

Conversation

herver
Copy link

@herver herver commented Jan 29, 2025

Plugins get simulated traffic update even when the box is unticked in Euroscope settings (it's just a display flag). This ensures no simulated traffic update is propagated to the server

@LeoKle
Copy link
Member

LeoKle commented Jan 30, 2025

Do avoid updates from simulated traffic, I did implement the following fix: 836e044.
I think this is the easier and better solution as we prevent the update from entering the DataManager (the update list to process) and we also don't need to extend the Pilot.h struct / model.

The fix has only been deployed to develop as we were awaiting feedback on it. I'll merge it with the next upcoming changes.
Does this work for you?

@herver
Copy link
Author

herver commented Feb 1, 2025

Thanks for the pointer, I missed the fix you implemented. I gave it a try and it seems to be working just fine !

@LeoKle
Copy link
Member

LeoKle commented Feb 1, 2025

No problem, thanks for testing!

@LeoKle LeoKle closed this Feb 1, 2025
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.

2 participants