zlsa / atc

https://openscope.co/
342 stars 107 forks source link

EIDW bugs in 2.1.8 #714

Closed scott-j-smith closed 7 years ago

scott-j-smith commented 7 years ago

Hi All,

I don't know if these bugs are reported previously (didn't come up when I searched for EIDW), or if they are being fixed in 3.0.0, but I noticed 2 things that are incorrect:

  1. Inbound flights arriving from the southwest, following the OSGAR..DIRUM.. arrival route are by default flying through restricted airspace, but the restriction is they need to be above FL240, and they arrive at FL100, so they will always be in violation. This does not appear to be a point penalty at least.

  2. Inbound flights from the south (whether through OSGAR..DIRUM.. or VATRY..SORIN..) eventually follow a route that takes them back outside the control area by default (..SUGAD..DW704..DW705..), and there is a 20 point penalty for this.

Thanks, Scott

eliuuk commented 7 years ago

3.0.0 does not fix anything to do with any of the airports, but probably what's going on here are 2 things:

Fechulo commented 7 years ago

@scott-j-smith I've been meaning to fix these issues and others at EIDW for a while, but I'm waiting for v3 to be released before making any changes. These are the things I'd like to change/improve:

n8rzz commented 7 years ago

FWIW, there is no need to wait for v3 to make these changes. v3 leaves all the json files and [most] of the logic for rendering untouched. Any changes made to the data in the json files should carry over just fine to v3.

The only exception is, as I've called out a few times and will document once v3 is merged, is how airports are loaded. But that is a relatively minor change and should have little affect.

n8rzz commented 7 years ago

And for the FL010 spawning altitude, I believe that is still a thing, however, I have added a placeholder issue in my fork so I will remember to revisit it. In the v3.1 code (yes, I'm already hard at work on that stuff too) I have drastically simplified how aircraft spawn. So this may be mostly painless to implement in the future.

Fechulo commented 7 years ago

This should be fixed once #732 is merged.

erikquinn commented 7 years ago

The ATC repository is being migrated to it's new home at https://github.com/openscope/openscope, and thus, all issues are being closed. If this is still an issue with the latest version of the sim (accessible at http://www.openscope.co), or is a feature you still think we are lacking, please reopen the issue at the new repo.

Please note that the vast majority of these issues have been copied to the new repository, or else are covered by other issues created there. See the below screenshot for what it looks like when your issue is known in the new repo:

image

Thank you!

Closing this issue.