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

radar graphics glitch when labels turned on #814

Closed
honeypool opened this issue Nov 1, 2017 · 10 comments
Closed

radar graphics glitch when labels turned on #814

honeypool opened this issue Nov 1, 2017 · 10 comments
Assignees
Labels
HOTFIX Emergency bugfix which must be pushed directly to the production site
Milestone

Comments

@honeypool
Copy link

Browser: google chrome Version 61.0.3163.100 (Official Build) (32-bit)

Airport: any

Approximate play time before issue: i was playing for few days ~one year ago

Issue Description

Some airports crashes radar graphics when i turn on fix and runway labels. Marks appear not matching with terrain, and things get worse when i try to move radar with right mouse button- everything starts glitching including rotating lines and labels, sometimes it just freezes. The only thing that keeps moving adequately in any case is airplane marks. I found that happening on particular airports, i have checked some of them:

Dublin- OK
shannon- OK
Seattle- OK

luisville- crashes
albuquerque - crashes
El Paso- crashes
Portland- crashes
Tampa-crashes
Tucson-crashes
Ted stevens- crashes
Boston-crashes

If i open/refresh page on normal airport->works fine
if i turn on labels on normal airport->works fine
if i switch to glitchy airport after normal airport->works fine without labels
if i turn on labels on glitchy airport->radar crashes
if i turn off labels on glitchy airport->it keeps glitching
if i switch to normal airport with crashed radar graphics->keeps glitching
if i switch to normal airport, then refresh page->works fine again

radar graph crashes
.

@n8rzz
Copy link
Member

n8rzz commented Nov 1, 2017

Thanks for the detailed bug report! We just deployed v5.6.0 last night, so there may be some bugs (though we fixed the ones we saw in testing). This new version completely changes how things are drawn on the canvas. In our tests, performance was much improved but it seems you've found some issues with it.

Two additional questions for you:

  1. are there any console errors when you see the glitchy drawings? (ctrl + alt + i or cmd + alt + i) to open dev tools, then navigate to console tab
  2. are any of the airports experiencing this issue labeled as WIP airports?

@n8rzz n8rzz added the BUGFIX Defect of the simulator, airport, or aircraft label Nov 1, 2017
@Fechulo
Copy link
Member

Fechulo commented Nov 1, 2017

I'll answer those questions.
Surprisingly, no console errors and it occurs with the vast majority of airports, wip and non wip.
I'm surprised we didn't notice this in testing

@n8rzz
Copy link
Member

n8rzz commented Nov 1, 2017

@Fechulo @chpoklv humor me a moment, but can you both try again after clearing your cache? shift + f5 or cmd + shift + r on mac.

I didn't see any issues with albuquerque or tuscon after some quick pan and zoom tests

@honeypool
Copy link
Author

have tried shift+f5 and chrome incognito window, same problem. found this glitch even on android chrome
screenshot_2017-11-01-16-07-05

@n8rzz
Copy link
Member

n8rzz commented Nov 1, 2017

Hmm 🤔

Based on those screenshots it looks like some funkiness with translations. In this latest version, where those happen has moved and is designed to be much more modular. A stale cache seems like the issue here. Especially since I'm not seeing it at all on my end in prod or staging.

But, from what I'm reading here, you are not seeing the same results. that makes me 😞

@Fechulo are you able to reproduce this as well?

@erikquinn erikquinn added HOTFIX Emergency bugfix which must be pushed directly to the production site and removed BUGFIX Defect of the simulator, airport, or aircraft labels Nov 1, 2017
@erikquinn erikquinn added this to the v5.6.0 milestone Nov 1, 2017
@n8rzz
Copy link
Member

n8rzz commented Nov 1, 2017

I can repro this locally now. This is a major issue that we will be looking into immediately and will be pushed through as quick as possible.

@n8rzz n8rzz self-assigned this Nov 1, 2017
@n8rzz n8rzz modified the milestones: v5.6.0, v5.7.0 Nov 1, 2017
@n8rzz n8rzz mentioned this issue Nov 1, 2017
@n8rzz
Copy link
Member

n8rzz commented Nov 1, 2017

@chpoklv we have identified the issue and I have opened a PR for review. Since you were the original reporter, could you take a look at the review app for this PR?

https://openscope-dev-pr-815.herokuapp.com/

This version should have the issue you reported fixed. If you still see the issue there, report back here and let us know.

@honeypool
Copy link
Author

I see this issue fixed now.

I notice another small bug with graphics: heading scale around the radar screen is drawn only after you move a screen with right mouse button, i guess it should appear when you select the aircraft or maybe it should be visible all the time?

@n8rzz
Copy link
Member

n8rzz commented Nov 2, 2017

@chpoklv could you open a new issue for that one? I'm pretty sure I know why that's happening, but I would consider it a low priority (for me) at the moment.

Either way, open a new issue for it and I'll add some notes so anyone could go in and fix it.

@erikquinn erikquinn added the has pull request Add this label when an issue has a PR which will resolve it label Nov 2, 2017
@erikquinn
Copy link
Member

Resolved by #815. Closing.

@n8rzz n8rzz removed the has pull request Add this label when an issue has a PR which will resolve it label Nov 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HOTFIX Emergency bugfix which must be pushed directly to the production site
Projects
None yet
Development

No branches or pull requests

4 participants