WarcraftReamls.com
  FAQFAQ    SearchSearch    MemberlistMemberlist    UsergroupsUsergroups   RegisterRegister 
  ProfileProfile    Log in to check your private messagesLog in to check your private messages    Log inLog in 
Potential problem with Escalation 5.3 patch.

 
Post new topic   Reply to topic    WarcraftRealms.com Forum Index -> CensusPlus UI Mod Bugs
View previous topic :: View next topic  
Author Message
bringoutyourdead
Forums Admin & general flunky


Joined: 07 Nov 2008
Posts: 697
Location: Silicon Valley
WR Updates: 7,153,254
bringoutyourdead WR Profile

PostPosted: Tue May 21, 2013 7:00 pm    Post subject: Potential problem with Escalation 5.3 patch. Reply with quote

Besides the TOC needing an update
I just noticed on my second pass of census runs today that I'm seeing odd results.
/who is returning 50 results instead of the default 49 max
I need to see if this is a valid change on Blizzards part.. if so some tweaks need to be made to the CensusPlus code since the marker to trigger sub-queries will be broken with that change.
Back to top
View user's profile Send private message
bringoutyourdead
Forums Admin & general flunky


Joined: 07 Nov 2008
Posts: 697
Location: Silicon Valley
WR Updates: 7,153,254
bringoutyourdead WR Profile

PostPosted: Tue May 21, 2013 8:22 pm    Post subject: Reply with quote

Yes there is a problem
When using wholib library all results are being returned as complete even when there is an overflow state and the flag returned should be "not complete".
Back to top
View user's profile Send private message
1974ER
Epic Censi


Joined: 07 Nov 2008
Posts: 729

WR Updates: 24,508,830
1974ER WR Profile

PostPosted: Tue May 21, 2013 10:57 pm    Post subject: Reply with quote

EU is down for applying the patch, so I can't check anything at the moment. Even so, thank you for the heads up. This means that for now, the results will be way too low, as the 90s count will max out at 50? And most bigger factions will show faulty results for lower brackets, too?

Thank you for all the hard work, please keep us informed on future developments of this topic.
Back to top
View user's profile Send private message
Fett



Joined: 01 Nov 2008
Posts: 50

WR Updates: 908,855
Fett WR Profile

PostPosted: Wed May 22, 2013 2:56 am    Post subject: Reply with quote

He's already got a patched version up on Curse. It is reporting a newer version number, so last I tried I was unable to upload data collected with that version. For now I'm just collecting data files until the site will accept them again.
Back to top
View user's profile Send private message
1974ER
Epic Censi


Joined: 07 Nov 2008
Posts: 729

WR Updates: 24,508,830
1974ER WR Profile

PostPosted: Wed May 22, 2013 3:52 am    Post subject: Reply with quote

Thanks for heads up, Fett. Replaced and also going stockpile mode still.
Back to top
View user's profile Send private message
bringoutyourdead
Forums Admin & general flunky


Joined: 07 Nov 2008
Posts: 697
Location: Silicon Valley
WR Updates: 7,153,254
bringoutyourdead WR Profile

PostPosted: Wed May 22, 2013 12:38 pm    Post subject: Reply with quote

Yes I had to up the version number to stop people from using the 6.0.0 version since it fails to properly see all the available people to census.

Unfortunately Metalbeast was working on warcraftrealms last night.. and I didn't have a local copy of his contact info. I have sent messages to two of his contact address requesting that he update the valid version number for uploading.

The baseline problem is that Blizzard changed the way they respond to the /who request.. previously it would return a max of count of 50 but only show 49 to indicate that there were more matches then what could be shown. As of 5.0.3 it now has a max count of 50 and shows 50.

The old baseline straight api coding of CensusPlus has a hard coded limit of 49, which allowed me to quickly get an update out... but we now again clash with any other addon that queries Blizzards who subsystem.

The wholib library version fails because it can't as currently coded see that there is a overflow condition. All queries return as valid and complete.
Back to top
View user's profile Send private message
bringoutyourdead
Forums Admin & general flunky


Joined: 07 Nov 2008
Posts: 697
Location: Silicon Valley
WR Updates: 7,153,254
bringoutyourdead WR Profile

PostPosted: Thu May 23, 2013 2:47 pm    Post subject: Reply with quote

Good news of sorts...
next patch for censusplus is awaiting QA testing and 3rd party update.

A solution for the Libwho problem has been found and verified.. just waiting for code maintainer to update as needed.

A fix for the CensusPlus button being up during Pet Battle has been coded and is going to QA testing.

These updates will NOT require a CensusPlus version number push.. and you won't need to grab the update unless you are annoyed by the CensusPlus button or have other addons that use Blizzards /who subsystem (e.g. WIM)

As to when the update will happen.. who knows? it isn't under my control.
Back to top
View user's profile Send private message
FuxieDK



Joined: 22 May 2008
Posts: 472
Location: Copenhagen, DK
WR Updates: 2,683,372
FuxieDK WR Profile

PostPosted: Thu May 23, 2013 10:45 pm    Post subject: Reply with quote

bringoutyourdead wrote:
A solution for the Libwho problem has been found and verified.. just waiting for code maintainer to update as needed.
Have the guy making LibWho been made aware of this?

It's an external Lib, made by someone else..
_________________
Doing census on various servers Wink
Back to top
View user's profile Send private message
bringoutyourdead
Forums Admin & general flunky


Joined: 07 Nov 2008
Posts: 697
Location: Silicon Valley
WR Updates: 7,153,254
bringoutyourdead WR Profile

PostPosted: Fri May 24, 2013 11:28 am    Post subject: Reply with quote

I have a ticket open on the project page.
the libwho has three maintainers listed... two of which are still active

I'm going to be passive for the Long weekend (U.S.A. has Memorial Day Holiday) and if I don't see anything, I'll give a direct tickle on main coders main public email account to check on status.. The ticket should have already sent an email heads up... but I see that maintainer has been busy updating other addons under her control.
Back to top
View user's profile Send private message
Display posts from previous:   
Post new topic   Reply to topic    WarcraftRealms.com Forum Index -> CensusPlus UI Mod Bugs All times are GMT - 6 Hours
Page 1 of 1

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
WarcraftRealms.com  


Powered by phpBB © 2001, 2005 phpBB Group