Skip to content

Adding VHHK and WSJC to RPHI#389

Open
HacobAllon wants to merge 13 commits into
vacs-project:mainfrom
HacobAllon:main
Open

Adding VHHK and WSJC to RPHI#389
HacobAllon wants to merge 13 commits into
vacs-project:mainfrom
HacobAllon:main

Conversation

@HacobAllon
Copy link
Copy Markdown

Description

Addition of VHHK and WSJC with a reshuffle of RPHI.json

Checklist

  • Dataset files are in the correct dataset/{FIR}/ directory.
  • I have verified the data is correct.
  • If adding a new FIR: CODEOWNERS entry added and maintainer team noted in PR description.
  • If contributing from a fork: "Allow edits by maintainers" is enabled.

@HacobAllon HacobAllon requested a review from a team as a code owner May 8, 2026 10:17
Copilot AI review requested due to automatic review settings May 8, 2026 10:17
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds Hong Kong (VHHK) and San Jose (WSJC) ACC entries to the RPHI profile and reorganizes Manila (MNL) control/FMP key placement.

Changes:

  • Expanded Manila (MNL) CTR sector presets and adjusted FMP labeling.
  • Replaced the previous “CONTROL” page content with an “ACC” page containing VHHK + WSJC entries.
  • Updated the ACC page layout configuration (rows/keys).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +398 to 457
"label": ["ACC"],
"page": {
"rows": 5,
"rows": 3,
"keys": [
{
"label": ["MNL", "CTR"],
"station_id": "MNL_CTR"
"label": ["HKG", "U CTR"],
"station_id": "VHHK_U_CTR"
},
{
"label": ["MNL", "2 CTR"],
"station_id": "MNL_2_CTR"
"label": ["HKG", "S CTR"],
"station_id": "VHHK_S_CTR"
},
{
"label": [""]
"label": ["HKG", "W CTR"],
"station_id": "VHHK_W_CTR"
},
{
"label": [""]
"label": ["HKG", "C CTR"],
"station_id": "VHHK_C_CTR"
},
{
"label": ["FMP"],
"station_id": "MNL_FMP"
"label": ["HKG", "D CTR"],
"station_id": "VHHK_D_CTR"
},
{
"label": ["MNL", "N CTR"],
"station_id": "MNL_N_CTR"
"label": ["HKG", "V CTR"],
"station_id": "VHHK_V_CTR"
},
{
"label": ["MNL", "NE CTR"],
"station_id": "MNL_NE_CTR"
},
{
"label": ["MNL", "NW CTR"],
"station_id": "MNL_NW_CTR"
},
{
"label": ["MNL", "N1 CTR"],
"station_id": "MNL_N1_CTR"
"label": ["WSJC", "8 CTR"],
"station_id": "WSJC_8_CTR"
},
{
"label": [""]
},
{
"label": ["MNL", "C CTR"],
"station_id": "MNL_C_CTR"
},
{
"label": ["MNL", "CN CTR"],
"station_id": "MNL_CN_CTR"
},
{
"label": ["MNL", "CE CTR"],
"station_id": "MNL_CE_CTR"
"label": [""]
},
{
"label": ["MNL", "CW CTR"],
"station_id": "MNL_CW_CTR"
"label": [""]
},
{
"label": ["MNL", "W CTR"],
"station_id": "MNL_W_CTR"
"label": [""]
},
{
"label": ["MNL", "S CTR"],
"station_id": "MNL_S_CTR"
"label": [""]
},
{
"label": ["MNL", "SE CTR"],
"station_id": "MNL_SE_CTR"
"label": [""]
},
{
"label": ["MNL", "SW CTR"],
"station_id": "MNL_SW_CTR"
"label": [""]
},
{
"label": ["MNL", "S1 CTR"],
"station_id": "MNL_S1_CTR"
"label": [""]
},
{
"label": ["MNL", "CS CTR"],
"station_id": "MNL_CS_CTR"
"label": ["ASEA", "FSS"],
"station_id": "ASEA_FSS"
}
Comment thread dataset/RP/profiles/RPHI.json
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