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

VATPRC TMA Update #340

Merged
merged 1 commit into from
Dec 10, 2023
Merged

Conversation

RudiZhang
Copy link
Contributor

@RudiZhang RudiZhang commented Nov 23, 2023

Description

  • TMA boundaries re-defined

Motivation and Context

  • match ATC sectors, add necessary sub-sector

How to prove the effect of this PR?

Is this a breaking change?

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

  • My change or addition follow the formatting standard of the project.
  • I am on the list of approved contributors.

@RudiZhang RudiZhang changed the title Major Update VATPRC TMA Update Nov 23, 2023
@kmoberg
Copy link
Collaborator

kmoberg commented Nov 24, 2023

This is a pretty massive pull-request changing a lot of files. I know it's not easy, but can you help make this easier for us by documenting what to expect things to look like, according to the policy on How to submit a pull request? That would make our job of verifying this a heck of a lot easier. @RudiZhang

@RudiZhang
Copy link
Contributor Author

This is a pretty massive pull-request changing a lot of files. I know it's not easy, but can you help make this easier for us by documenting what to expect things to look like, according to the policy on How to submit a pull request? That would make our job of verifying this a heck of a lot easier. @RudiZhang

only ZBAA_F ZBAD ZSSS_W ZSSS_S ZWWW_E ZSSS_F ZSSS_PD will be added fully new. All others are re-defined boundaries. The screenshots below are from Aero-Nav which we used for sectorfiles. Exactly the same as this PR.

image
image
image
image
image
image

@RudiZhang
Copy link
Contributor Author

image
image
image
image
image

There are some screenshots for this PR

@kmoberg kmoberg merged commit 1b461bc into vatsimnetwork:main Dec 10, 2023
@RudiZhang RudiZhang deleted the VATPRC-TMA-update branch December 11, 2023 08:02
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