User:Stephane Lo Presti/Post-10-25-maintenance feedback

From Guild Wars 2 Wiki
Jump to navigationJump to search

Following the wiki maintenance on 10/25, please report any bug or strange behavior on the wiki here.

All IP addresses reported as internal (again)[edit]

Since the maintenance, all edits are coming from 10.28.100.* addresses instead of the users' actual addresses. We've had a similar issue before.

The difference from before is that ALL anonymous users are showing the internal IPs, rather than it happening at random. This is a serious issue as it completely prevents us from blocking anonymous bots. —Dr Ishmael User Dr ishmael Diablo the chicken.png 07:57, 26 October 2012 (PDT)
I corrected this issue this morning. Justin Lloyd 14:25, 26 October 2012 (PDT)
I noticed that the IPs started looking normal again around lunchtime. Thanks! —Dr Ishmael User Dr ishmael Diablo the chicken.png 14:43, 26 October 2012 (PDT)
I move that we rename him Justin Time. — snogratUser Snograt signature.png 14:45, 26 October 2012 (PDT)

SMW CSS not configured properly[edit]

When previewing a page with semantic data (so that the factbox is shown), the CSS for SMW is not loaded. Instead, the plain text CSSskins/SMW_custom.css is displayed immediately after the <body> tag. —Dr Ishmael User Dr ishmael Diablo the chicken.png 08:18, 26 October 2012 (PDT)

Also, on pages that include semantic tooltips (example: preview Property:Is in trait line), it additionally displays JSskins/SMW_tooltip.jsDr Ishmael User Dr ishmael Diablo the chicken.png 08:48, 26 October 2012 (PDT)

Worked on this with Justin over email, patch has been applied and SMW's CSS/JS load properly now. —Dr Ishmael User Dr ishmael Diablo the chicken.png 16:00, 30 October 2012 (UTC)

signature time zone[edit]

I just noticed that the time zone used in signatures is now PDT, instead of UTC. Was this intended? —Dr Ishmael User Dr ishmael Diablo the chicken.png 16:51, 26 October 2012 (PDT)

  • The maintenance involved moving the wiki from one set of servers to another. The original servers were set to UTC and the new ones are on PDT and the wiki was picking up the system timezone since it wasn't being explicitly set via $wgLocaltimezone. I need to explicitly set it to either UTC (its historical setting), PDT (ArenaNet's timezone), or EDT (timezone of the country's capitol, consistent with how our European wikis are set). From a community perspective, is there any particular reason to stay with UTC? If not, I'd argue for setting it to EDT. Justin Lloyd 18:29, 26 October 2012 (PDT)
UTC. This isn't the USA GW2W, it's the global English-speakers' GW2W — snogratUser Snograt signature.png 18:48, 26 October 2012 (PDT)
Yup, UTC. Partly because it's my own timezone (non-DST), but also because it's the reference point for other timezones: UTC+x instead of PDT+x or EDT+x, or PST+x or EST+x after daylight savings. pling User Pling sig.png 08:04, 28 October 2012 (PDT)
In terms of online games I seldom use any other time system than UTC. It seems natural for me because you communicate with people around the globe. - Yandere Talk to me... 08:17, 28 October 2012 (PDT)
Speaking as someone in the EDT/EST timezone, yeah, UTC please. It's what we're used to, it's a sensible global standard, and given the not-insignificant number of Brits on the wiki, it's a bit of a stretch to call America the 'home' for the English language. Besides, as much as it's a worthless argument 99% of the time, it's what the English Wikipedia uses. - Tanetris 09:22, 28 October 2012 (PDT)
Don't forget all them Aussies, Tane — snogratUser Snograt signature.png 14:22, 28 October 2012 (PDT)
And the Kiwis, of course. Not forgetting India, Pakistan, Nigeria and many other nations who officially use English. — snogratUser Snograt signature.png 14:33, 28 October 2012 (PDT)
I agree, going back to the universal UTC time (redundancies har har) makes sense. —Dr Ishmael User Dr ishmael Diablo the chicken.png 08:20, 29 October 2012 (PDT)

I've set the timezone to UTC. Justin Lloyd 17:07, 29 October 2012 (UTC)

Refresh tab[edit]

There seems to be a new tab next to the watch/unwatch tab for refreshing the page. I don't recall that being there before, and it's redundant with the clock. --JonTheMon 05:48, 29 October 2012 (PDT)

I'd say the tab is better than the clock, since it actually says what clicking it does. pling User Pling sig.png 05:58, 29 October 2012 (PDT)
Fixed with a single CSS line, easy to revert. —Dr Ishmael User Dr ishmael Diablo the chicken.png 05:59, 29 October 2012 (PDT)
Well, it's not completely inappropriate, it's just that it's something unexpected. And it's not integrated properly (square on round). I suppose we could keep it for a clearer way to purge (I would want it renamed as such, though). --JonTheMon 06:41, 29 October 2012 (PDT)
I don't see anything wrong with the redundancy, given the more obvious meaning of the tab. However, why would purge be better than refresh? To an unexperienced reader, "refresh" makes more sense in the context of web browsing, where they might think "purge" means that they'll clear out (blank) the page itself. I can add a bit of JS that would add a tooltip to the link (like all the other tabs have) for even better explanation. —Dr Ishmael User Dr ishmael Diablo the chicken.png 07:39, 29 October 2012 (PDT)
I remember thinking "purge" would do something ghastly before I knew what it did, so yes, "refresh" with explanatory tooltip is the way to go. The clock already has a tooltip, but it does use the word purge, which even there might still be a bit scary. pling User Pling sig.png 07:50, 29 October 2012 (PDT)
"Refresh the server's cached version of this page" How does that sound? I'll get started on the CSS changes to make it fit nicely. —Dr Ishmael User Dr ishmael Diablo the chicken.png 08:08, 29 October 2012 (PDT)

Queries and subcategories[edit]

From smw:Help:Inferencing: "SMW's mechanism of subcategory inferencing can be restricted or disabled by the site administrator." I believe this is restricted or disabled. If you take a look at my sandbox you'll see that I can get a result when searching the Category "Griffons", but not when searching Category "Bestiary", which "Griffons" is a subcategory of. If subcategory inferencing is enabled, then I can search through all pages in all subcategories of the Bestiary and not have to name them all individually. Jauranna 15:55, 30 October 2012 (UTC)

That is configured through $smwgQSubcategoryDepth, which defaults to 10. I don't know why this would be disabled - it's actually quite necessary given the somewhat obtuse categorization scheme in place here. —Dr Ishmael User Dr ishmael Diablo the chicken.png 16:22, 30 October 2012 (UTC)
@Justin: If you could list all of the custom configuration options currently set for SMW, that would be very useful. —Dr Ishmael User Dr ishmael Diablo the chicken.png 16:26, 30 October 2012 (UTC)
$smwgLinksInValues = true;
$smwgUseCategoryHierarchy = false;
$smwgQSubcategoryDepth = 0;
$smwgQPropertyDepth = 10;
$smwgQFeatures = SMW_PROPERTY_QUERY | SMW_CATEGORY_QUERY | SMW_CONCEPT_QUERY | SMW_NAMESPACE_QUERY | SMW_CONJUNCTION_QUERY | SMW_DISJUNCTION_QUERY;
$smwgQConceptFeatures = SMW_ANY_QUERY & ~SMW_DISJUNCTION_QUERY & ~SMW_CONCEPT_QUERY;
Justin Lloyd 16:57, 30 October 2012 (UTC)
That looks okay except for the category settings. I assume you decided to disable that for performance reasons? Unfortunately, like I said above, we will need it, but we should be okay with it limited to a depth of only 4 or 5. —Dr Ishmael User Dr ishmael Diablo the chicken.png 17:32, 30 October 2012 (UTC)
As I recall, these settings were specifically requested by the German GW2 wiki community for their wiki, so we standardized across the board. However, I'm fine with changing them to suit this community's needs (as long as performance isn't noticeably impacted). Just let me know what you specifically want. Justin Lloyd 17:40, 30 October 2012 (UTC)
$smwgUseCategoryHierarchy = true;
$smwgQSubcategoryDepth = 5;
Let's go with that for now. I checked a few base categories (Bestiary, Skills) to see how deep our hierarchy goes, and the deepest I found was 4 levels from the base, so I don't think we'll ever exceed 5. (If we do, then it's probably an indication that the hierarchy needs a restructuring.) —Dr Ishmael User Dr ishmael Diablo the chicken.png 17:55, 30 October 2012 (UTC)
I believe the reason for our request of disabling subcategory inferencing was the fact that it threw database errors on certain queries because SMW (sometimes?) needs to create temporary MySQL tables for that and the database is using a slave architecture where creating tables is only allowed on the master server (which is a sane setting), while SMW will sometimes try to do that on the slaves. It's noted as a "todo" in the relevant source code, I can dig it up if you need. --Tera 18:53, 30 October 2012 (UTC)
Ah, right, Justin had mentioned that issue in our email conversation, but I didn't connect it to subcategories. From a quick Google search, I found this MySQL bug, which ties the issue to the log-bin=mysql-bin configuration setting. A workaround was posted to use binlog_format = ROW, instead of binlog_format = STATEMENT or MIXED, which somehow avoids the bug. —Dr Ishmael User Dr ishmael Diablo the chicken.png 21:27, 30 October 2012 (UTC)
I have no idea about the performance implications of setting the log format to ROW instead of STATEMENT, from skimming over the documentation the row format tends to generate bigger logs, but doesn't mention how much bigger they might be. It's a crude workaround for the underlying problem. --Tera 21:44, 30 October 2012 (UTC)
Yes, there are size and performance tradeoffs between STATEMENT and ROW-based binary logging and it's not a change to be made lightly. I need some time to investigate this more carefully, including looking at the types of statements being performed by MediaWiki. Justin Lloyd 21:49, 30 October 2012 (UTC)
I don't like the fact that SMW even needs to create a temporary table for that :/ I need to find the time to set up a small replication locally and try out if we might be able to do some fixes to the code to make it work without changing MySQL settings, either by telling SMW to always use the master connection or by finding a (performant) way to make it work without a temp table. I know that they are testing out another SQL layer for SMW, but didn't really have the time to see what they have changed yet. [Just got an internal error when posting this.]--Tera 22:03, 30 October 2012 (UTC)

persistent bug[edit]

Since we've got eyes on this page, could someone look at this issue? It is persisting even after the maintenance, and preventing some people from creating accounts. —Dr Ishmael User Dr ishmael Diablo the chicken.png 21:37, 30 October 2012 (UTC)

Good timing. This is due to the recaptcha extension. I'm literally just about to switch over to ConfirmEdit with FancyCaptcha, which we're already using for the DE/FR/ES wikis. It should be just a few more minutes, I think. Justin Lloyd 21:46, 30 October 2012 (UTC)
Trying to edit this page (and probably others) started failing after the change, so further testing is required. Justin Lloyd 22:01, 30 October 2012 (UTC)
This edit indicates the ConfirmEdit change has fixed account creation and edits still work. Justin Lloyd 22:58, 30 October 2012 (UTC)
Confirming that account creation works. - Dvorak 00:45, 31 October 2012 (UTC)

Semantic Forms and Widgets[edit]

Back when we started the whole process for SMW, the request had included Semantic Forms and Widgets to be installed at the same time. Is there any way we could at least get Semantic Forms enabled right now? We need the #arraymap function pretty seriously to handle multi-valued parameters in infoboxes. Without it, we're having to either kludge up a multiple-parameter solution (| parm1= | parm2= | etc.) or leave the infobox with limited functionality. —Dr Ishmael User Dr ishmael Diablo the chicken.png 14:40, 1 November 2012 (UTC)

Hi Dr Ishmael, although this is not a note about the maintenance (;P) this is something that we're going to include in a possible update for this month. Can you please add it to Guild_Wars_2_Wiki:Requests_for_technical_administration, in addition to removing the SMW request? Cheers. (personal note: thanks for your help and interactions around this) --Stephane Lo Presti talk 21:10, 5 November 2012 (UTC)
Thanks :D --Stephane Lo Presti talk 21:13, 5 November 2012 (UTC)
Done, Guild Wars 2 Wiki:Requests for technical administration/Additional semantic extensions. —Dr Ishmael User Dr ishmael Diablo the chicken.png 21:46, 5 November 2012 (UTC)
Looks great, the breakdown by extension will greatly help! Thanks again --Stephane Lo Presti talk 22:54, 5 November 2012 (UTC)

Taking the site notice down?[edit]

Hi, it looks like all the feedback about the maintenance was reported and processed. If not, we'd like to make sure that everyone has an opportunity to do so. It'd be ideal to take down the maintenance part of the site notice fairy quickly. Thanks! --Stephane Lo Presti talk 21:13, 5 November 2012 (UTC)