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 #4814 - Correct lots of runway definitions #4815

Merged
merged 16 commits into from
Jul 23, 2023

Conversation

AliceFord
Copy link
Collaborator

@AliceFord AliceFord commented Jul 17, 2023

Fixes #4814

Summary of changes

  • Fix error 1 - runway order incorrect
  • Fix error 2 - newline at end of file
  • Fix error 3 - runway definitions backwards
  • Fix error 4 - actual runway issues

Screenshots (if necessary)

@AliceFord AliceFord self-assigned this Jul 17, 2023
@AliceFord AliceFord added the bug Bugs or errors label Jul 17, 2023
@AliceFord
Copy link
Collaborator Author

Just a preface: it is entirely possible that some parts of this PR are wrong, I haven't checked every file. If you find an issue (or worse, a string if issues), let me know and I'll sort it out.

@AliceFord AliceFord marked this pull request as ready for review July 17, 2023 20:40
@AliceFord AliceFord marked this pull request as draft July 17, 2023 20:42
@AliceFord
Copy link
Collaborator Author

Already found some issues, I'll fix tomorrow morning.

@AliceFord AliceFord marked this pull request as ready for review July 18, 2023 05:34
@AliceFord AliceFord marked this pull request as draft July 22, 2023 06:15
@AliceFord AliceFord marked this pull request as ready for review July 22, 2023 08:04
Copy link
Collaborator

@Robbo599 Robbo599 left a comment

Choose a reason for hiding this comment

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

you have removed all military airfield runways these are to be kept

Airports/EGNE/Runway.txt Outdated Show resolved Hide resolved
@AliceFord
Copy link
Collaborator Author

you have removed all military airfield runways these are to be kept

Ahhhhh thats very annoying that they aren't covered my the EGTT/EGPX MATS c6, I'll go through manually and back all the ones that fulfil the requirements.

@AliceFord
Copy link
Collaborator Author

AliceFord commented Jul 23, 2023

Another question @Robbo599, many of the military aerodromes don't have their ATIS's defined when the aerodrome has an ATIS service, but the ATIS frequencies are outside of the normal vatsim VHF range. Should they be added or not?

@Robbo599
Copy link
Collaborator

Another question @Robbo599, many of the military aerodromes don't have their ATIS's defined when the aerodrome has an ATIS service, but the ATIS frequencies are outside of the normal vatsim VHF range. Should they be added or not?

most military units use UHF freqs so can't be used on VATSIM so no point defining them. we have a few alternate VHF ones defined for the more commonly used airfields.

Airports/EGXE/Runway.txt Outdated Show resolved Hide resolved
Copy link
Collaborator

@PLM1995 PLM1995 left a comment

Choose a reason for hiding this comment

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

Only looked at deletions - will need to go through the corrections separately.

This is gonna make the runway selector so much cleaner!

.github/CHANGELOG.md Outdated Show resolved Hide resolved
Airports/EGCM/Runway.txt Outdated Show resolved Hide resolved
Airports/EGCM/Runway.txt Outdated Show resolved Hide resolved
Airports/EGLW/Runway.txt Outdated Show resolved Hide resolved
@PLM1995 PLM1995 changed the title Fixes #4814 - Lots of runway definitions incorrect Fixes #4814 - Correct lots of runway definitions Jul 23, 2023
@PLM1995
Copy link
Collaborator

PLM1995 commented Jul 23, 2023

Shall we cover #4846 in this PR too?

EDIT: Just saw someone has already claimed it, never mind.

@PLM1995 PLM1995 added the enhancement For new or improved features label Jul 23, 2023
Airports/EGAC/Runway.txt Outdated Show resolved Hide resolved
@PLM1995
Copy link
Collaborator

PLM1995 commented Jul 23, 2023

OK so just compilation errors now. The following need active runway definitions removing from https://github.com/VATSIM-UK/UK-Sector-File/blob/main/UK.rwy I think as their runways are no longer defined.

  • EGHO
  • EGHR
  • EGNC
  • EGSU

@AliceFord
Copy link
Collaborator Author

OK so just compilation errors now. The following need active runway definitions removing from https://github.com/VATSIM-UK/UK-Sector-File/blob/main/UK.rwy I think as their runways are no longer defined.

  • EGHO
  • EGHR
  • EGNC
  • EGSU

I didn't even know UK.rwy was a thing! Fixed.

Copy link
Collaborator

@PLM1995 PLM1995 left a comment

Choose a reason for hiding this comment

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

A P P R O V E D

Thanks @AliceFord

@AliceFord
Copy link
Collaborator Author

A P P R O V E D

Thanks @AliceFord

Thank god (thanks to you and the other reviewers as well)

@PLM1995
Copy link
Collaborator

PLM1995 commented Jul 23, 2023

OK so just compilation errors now. The following need active runway definitions removing from https://github.com/VATSIM-UK/UK-Sector-File/blob/main/UK.rwy I think as their runways are no longer defined.

  • EGHO
  • EGHR
  • EGNC
  • EGSU

I didn't even know UK.rwy was a thing! Fixed.

I think the Active_Runway.txt files are actually ignored by the compiler as far as I can tell and it just uses UK.rwy, perhaps another something to add to the look at list...

@Robbo599
Copy link
Collaborator

nice work!

@PLM1995 PLM1995 merged commit d259fef into VATSIM-UK:main Jul 23, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs or errors enhancement For new or improved features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Lots of runway definitions incorrect
3 participants