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

Fixes #5551 - Improve next sector prediction for ScTMA arrs Montrose -> Talla #5555

Merged
merged 4 commits into from
Jan 22, 2025

Conversation

hazzas-99
Copy link
Contributor

Fixes #5551

Summary of changes

Added guest ownership to Deancross North sector for the Talla, ScTMA Bandbox and Scottish Low positions for PH/PF/PK inbounds, to improve performance of next sector prediction from Montrose to Talla.

Screenshots

Not yet tested.

@hazzas-99 hazzas-99 added the bug Bugs or errors label Jan 6, 2025
@PLM1995 PLM1995 changed the title Fixes #5551 - Improved next sector prediction for ScTMA arrs Dcs -> Talla Fixes #5551 - Improve next sector prediction for ScTMA arrs Dcs -> Talla Jan 7, 2025
@Lucak1011
Copy link
Collaborator

I can test if you'd like me to, Harry. What's the best way of going about this? Create a sbx file with routes that should go through the mentioned sectors?

@BenWalker01
Copy link
Collaborator

@Lucak1011 We had a go with #5554, and couldn't get it to work. Perhaps it will in this case, but given the similar nature of the fix I'm not sure. To answer your question, a SBX file would be the easiest, I think that's what Peter did when testing.

@hazzas-99 hazzas-99 changed the title Fixes #5551 - Improve next sector prediction for ScTMA arrs Dcs -> Talla Fixes #5551 - Improve next sector prediction for ScTMA arrs Montrose -> Talla Jan 13, 2025
@BenWalker01
Copy link
Collaborator

@hazzas-99 I had to update the agreement from Montrose to Talla as in #5554 to get the COPX to work.

image
@Lucak1011 As above, I think this now works if you want to test it?

@hazzas-99
Copy link
Contributor Author

@BenWalker01 yeah that makes sense. AGPED 260 is slightly above profile anyway so makes sense FIXBEFORE works better.

@PLM1995 PLM1995 merged commit 06352b0 into main Jan 22, 2025
4 checks passed
@PLM1995 PLM1995 deleted the issue-5551 branch January 22, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs or errors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incorrect next sector prediction for ScTMA inbounds from ScAC South to Talla
5 participants