Skip to content

Fix OIBH to Bahregan and remove duplicate OI20 - #235

Merged
alexander-san merged 1 commit into
mwgg:masterfrom
thomasgobinfcmlabs:fix-oibh-bahregan
Aug 16, 2026
Merged

alexander-san merged 1 commit into
mwgg:masterfrom
thomasgobinfcmlabs:fix-oibh-bahregan

Conversation

@thomasgobinfcmlabs

@thomasgobinfcmlabs thomasgobinfcmlabs commented Aug 16, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #234.

OIBH is Bahregan, not Bastak. The row carried the correct iata value and the wrong name, state, lat, lon and elevation. OI20 held the real Bahregan data under an OurAirports placeholder code. This PR moves that data onto OIBH and removes OI20, which then duplicates it.

Two entries change. The file drops from 29,308 to 29,307 rows. No other row moves.

Evidence

ICAO Doc 7910 is the authority for location indicators. Edition 132 gives this in the OI - IRAN (ISLAMIC REPUBLIC OF) block:

O I B B     BUZ    BUSHEHR
O I B H*    IAQ    BAHREGAN
O I B I*    PGU    ASALOYEH

ICAO assigns OIBH to Bahregan. The IATA code IAQ sits on the same row. That edition holds no entry for Bastak, in the OI decode block or in the alphabetical location index.

SkyVector, FlightAware, Wikidata Q3448885, OpenStreetMap way/733568330 and both the English and Persian Wikipedia articles all agree. Issue #234 gives the full table.

OurAirports is the source of the old pairing. Several airport directories repeat it, but they all share the OurAirports lineage.

Field notes

  • city as Emam Hasan follows the house style of the file, which strips diacritics. OurAirports spells the municipality "Imam Hassan".
  • lat and lon come from the old OI20 row. They agree with SkyVector, Wikidata and OpenStreetMap to a few hundred metres. The Wikipedia infobox gives 50|58|33|E, which is a typo for 50|16|33|E, so I did not use it.
  • elevation stays at the 20 ft of the old OI20 row. SkyVector gives 32 ft instead. No source settles it, so I kept the value already in the file. Tell me if you prefer 32.

Checks

  • .github/scripts/valid_json.py passes.
  • .github/scripts/sort.py output is byte-identical to the committed file.
  • The diff touches two entries and no others.

@alexander-san

Copy link
Copy Markdown
Collaborator

Nice thanks

@alexander-san
alexander-san merged commit 2473bd8 into mwgg:master Aug 16, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OIBH is Bahregan, not Bastak: name, state, lat, lon and elevation are wrong, and OI20 duplicates it

2 participants