Skip to content

Conversation

@HacobAllon
Copy link
Contributor

@HacobAllon HacobAllon commented Nov 14, 2025

RPHI -> MNL
Added MNL NW, NE, SE,SW, W

Description of changes

Changes RPHI -> MNL

Reason and motivation

Adds useful sectors so that pilots know which frequency to contact

Approved contributior?

  • I am on the approved contributers list
  • I have sent an request by email to get approved
  • Someone on the approved contributer list will review this request

RPHI -> MNL 
Added MNL NW, NE, SE,SW, W
@github-actions
Copy link

@github-actions
Copy link

❌ Github login HacobAllon was not found in contributor list. Please read Readme to learn more.

Keep in mind: this message will not update automatically. Please, re-run validate-contributor job when this will be solved.

Copy link
Collaborator

@daniluk4000 daniluk4000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are not approved as contributor, and all checks fail

@HacobAllon HacobAllon marked this pull request as draft November 23, 2025 12:45
@HacobAllon HacobAllon marked this pull request as ready for review November 23, 2025 12:46
@HacobAllon
Copy link
Contributor Author

You are not approved as contributor, and all checks fail

My self and my vACC Director sent an email months ago regarding this. The email should've been sent by @vatphil.com

@HacobAllon
Copy link
Contributor Author

Should be completed now.

Copy link
Collaborator

@daniluk4000 daniluk4000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Image Image

Those are old misaligments. Would it be possible to fix them though?

WBFC and KZAK

@HacobAllon
Copy link
Contributor Author

It weird as thats the decimal code given by AeroGNG 4, 132.53333333333.
Screenshot 2025-11-23 at 3 12 24 pm
same thing with the one with WBFC
Screenshot 2025-11-23 at 3 14 30 pm

@Adrian2k
Copy link
Collaborator

Borders in our dataset must allign.
You can either update the other FIR's borders since it is a small change, or adjust to keep the current border as it has no operational effect for the pilots regardless. Up to you, but must be fixed before we can merge.

@Adrian2k
Copy link
Collaborator

Adrian2k commented Nov 24, 2025

Also we prefer the dataset to be defined as RPHI, with MNL as secondary code - this is how it is done almost everywhere else where possible.

Edit: You must do this change - MN belongs to another country, so it will be mislabeled as belonging to Nicaragua. We also want our dataset alphabetically sorted; this helps maintaining an alphabetical sorting.

@Adrian2k
Copy link
Collaborator

You are not approved as contributor, and all checks fail

My self and my vACC Director sent an email months ago regarding this. The email should've been sent by @vatphil.com

As we also sent out via email to all previous registered contributers, including an vatphil.com email, there was a change of policy 18th July, so all contributers was removed and must read the new policy and apply for access.
No email from any vatphil.com has been recieved after 18th July.

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