Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryDate SubmittedLast Update
0001722AI War[All Projects] Bug - Graphical IssueDec 2, 2010 10:26 pmDec 3, 2010 5:01 pm
ReporterToll 
Assigned Tokeith.lamothe 
Severityminor 
StatusresolvedResolutionfixed 
Product Version4.043 
Fixed in Version 
Summary0001722: Panning via middle mouse button got choppy in this version
DescriptionBefore, panning via holding down the middle mouse button was smooth. Now, however, the screen "sticks" for a moment a few times per second, giving an extremely choppy sensation when panning that way.
TagsNo tags attached.
Internal Weight
Attached Files

- Relationships

-  Notes
(0005043)
keith.lamothe (administrator)
Dec 3, 2010 10:19 am

Spent a good part of the morning going back and forth between svn revisions trying to find which one broke this, and it appears to have started in 1094 where I would expect, but it isn't as frequent there as in the head revision. The only ones I've been able to confirm have been during GC collections, which makes everything skip.
(0005044)
Toll (reporter)
Dec 3, 2010 10:38 am

Considering it skips a few times a second on my end, it isn't likely it has to do with the GC. My guess was that it had to do with the new "set the mouse position every frame" part of the code.
(0005055)
keith.lamothe (administrator)
Dec 3, 2010 12:25 pm

That change was reverted out before the release. But there was another change that if anything should make stuff smoother, but I guess it's hard to say.
(0005064)
keith.lamothe (administrator)
Dec 3, 2010 2:52 pm

Ok, it was the world point thing, so I had to revert that to not update unless the actual screen point has updated. I recall having to do this when initially implementing the middle-mouse-scroll (and again when porting it to unity).
(0005065)
x4000 (administrator)
Dec 3, 2010 2:56 pm

Why on earth would that cause such lag? It's a pretty minor update of one struct. What all else is it cascading into updates?
(0005066)
keith.lamothe (administrator)
Dec 3, 2010 3:01 pm

It's not lag, it just makes the middle-mouse-scrolling jump back and forth even when the screen point isn't moving, because the world-point is still updating and the middle-mouse operation is moving the world under the cursor, so it never actually settles.

At least, that's what I ran into both during the initial implementation and the unity port of said.

I'll take another brief look, but I've probably spent 10 hours total on the middle-mouse behavior over the past year and it's some of the strangest, most intractable, "just change something and hope it works this time" things I've ever worked with, including ColdFusion ;)
(0005068)
keith.lamothe (administrator)
Dec 3, 2010 3:18 pm

Ok, I tried some different changes and it seems to not break middle-mouse-scrolling while still updating screen and world mouse coordinates every frame. We'll see if it breaks anything else :)
(0005071)
x4000 (administrator)
Dec 3, 2010 5:01 pm

Ok -- wow. Didn't know it was such a crazy thing!

Issue Community Support
This issue is already marked as resolved.
If you feel that is not the case, please reopen it and explain why.
Supporters: No one explicitly supports this issue yet.
Opponents: No one explicitly opposes this issue yet.

- Issue History
Date Modified Username Field Change
Dec 2, 2010 10:26 pm Toll New Issue
Dec 3, 2010 8:03 am x4000 Assigned To => keith.lamothe
Dec 3, 2010 8:03 am x4000 Status new => acknowledged
Dec 3, 2010 10:19 am keith.lamothe Note Added: 0005043
Dec 3, 2010 10:38 am Toll Note Added: 0005044
Dec 3, 2010 12:25 pm keith.lamothe Note Added: 0005055
Dec 3, 2010 2:52 pm keith.lamothe Note Added: 0005064
Dec 3, 2010 2:52 pm keith.lamothe Status acknowledged => resolved
Dec 3, 2010 2:52 pm keith.lamothe Resolution open => fixed
Dec 3, 2010 2:56 pm x4000 Note Added: 0005065
Dec 3, 2010 3:01 pm keith.lamothe Note Added: 0005066
Dec 3, 2010 3:18 pm keith.lamothe Note Added: 0005068
Dec 3, 2010 5:01 pm x4000 Note Added: 0005071


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker