Last modified: 2014-09-26 07:08:03 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T73048, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71048 - Make the HHVM BetaFeature up to snuff (BetaFeature-wise)
Make the HHVM BetaFeature up to snuff (BetaFeature-wise)
Status: RESOLVED FIXED
Product: MediaWiki extensions
Classification: Unclassified
WikimediaEvents (Other open bugs)
master
All All
: Normal normal (vote)
: ---
Assigned To: James Forrester
: hhvm
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2014-09-19 18:03 UTC by Greg Grossmeier
Modified: 2014-09-26 07:08 UTC (History)
11 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
beta features image (6.43 KB, image/svg+xml)
2014-09-19 18:46 UTC, Jared Zimmerman (WMF)
Details

Description Greg Grossmeier 2014-09-19 18:03:38 UTC
Sorry Jared :(
Comment 1 Jared Zimmerman (WMF) 2014-09-19 18:46:11 UTC
Updated test string…


"This experiment improve browsing and editing speed and performance. As more users enable the experiment the system will actually improve over time."
Comment 2 Jared Zimmerman (WMF) 2014-09-19 18:46:30 UTC
Created attachment 16520 [details]
beta features image
Comment 3 Kunal Mehta (Legoktm) 2014-09-19 19:02:13 UTC
(In reply to Jared Zimmerman (WMF) from comment #1)
> Updated test string…
> 
> 
> "This experiment improve browsing and editing speed and performance. As more
> users enable the experiment the system will actually improve over time."

That wording really isn't consistent with what is already on [[Special:BetaFeatures]] (there isn't much consistency to begin with but lets not make it less consistent?).

Also, it's misleading since more people enabling HHVM isn't going to magically start a snowball effect that makes the site faster...

What's wrong with the current wording?
Comment 4 Jared Zimmerman (WMF) 2014-09-19 20:21:29 UTC
Too technical, too many acronyms, its not in the voice and tone that we should be writing in. Someone should not have to understand what any of those concepts are to want to enable the beta feature, and it may actually cause a lower uptake of users. 

My understanding of talking to Ori about HHVM is that as more pages were accessed by users via HHVM is that subsequent access to those pages would be sped up. Is that not the case?
Comment 5 Matthew Flaschen 2014-09-20 04:02:47 UTC
(In reply to Jared Zimmerman (WMF) from comment #4)
> My understanding of talking to Ori about HHVM is that as more pages were
> accessed by users via HHVM is that subsequent access to those pages would be
> sped up. Is that not the case?

Ori may want to comment, but I don't think it's on a page-specific basis.  There's a couple things:

* Bug reports are essential (this is the main reason it's a beta feature, I believe) to minimize problems with a wider rollout.  This could be expressed as "user feedback".
* The Wikipedia article says, "HHVM has the ability to use live type information to produce more efficient native code" which I think may mean that code paths get re-compiled to native code when frequent executions reveal the most common data types.

Also, we could avoid acronyms by expanding it.  It stands for "HipHop Virtual Machine", which is a pretty cool name even if you don't know what it does.
Comment 6 Jared Zimmerman (WMF) 2014-09-20 21:19:34 UTC
I talked to Ori friday, he's fine with the proposed text. We also need to convert the talk page to a flow enabled one. Ori will work with the Flow team on this.
Comment 7 Jared Zimmerman (WMF) 2014-09-20 21:21:15 UTC
Danny/S we need to Flow enable https://www.mediawiki.org/wiki/Talk:HHVM/About can you please work with Ori if you have any questions or clarifications.
Comment 8 Greg Grossmeier 2014-09-22 17:47:48 UTC
To be explicit on the changes needed:
1) James(?): Description update
2) James(?): Add image
3) S: Flow-ify the talk page: https://www.mediawiki.org/wiki/Talk:HHVM/About
Comment 9 Gerrit Notification Bot 2014-09-22 18:07:39 UTC
Change 162002 had a related patch set uploaded by Jforrester:
Enable Flow on [[mw:Talk:HHVM/About]]

https://gerrit.wikimedia.org/r/162002
Comment 10 Gerrit Notification Bot 2014-09-22 18:26:48 UTC
Change 162008 had a related patch set uploaded by Jforrester:
HHVM BetaFeature: Add screenshot; improve text

https://gerrit.wikimedia.org/r/162008
Comment 11 James Forrester 2014-09-22 18:38:04 UTC
See https://gerrit.wikimedia.org/r/#/c/162008/2/i18n/en.json for an attempt to do a description (and title) that makes more sense, and is inline with our standards for copy in BetaFeatures' descriptions. Thoughts?
Comment 12 Gerrit Notification Bot 2014-09-23 18:24:34 UTC
Change 162002 merged by jenkins-bot:
Enable Flow on [[mw:Talk:HHVM/About]]

https://gerrit.wikimedia.org/r/162002
Comment 13 Gerrit Notification Bot 2014-09-26 06:35:58 UTC
Change 162008 merged by jenkins-bot:
HHVM BetaFeature: Add screenshot; improve text

https://gerrit.wikimedia.org/r/162008
Comment 14 Gerrit Notification Bot 2014-09-26 06:37:12 UTC
Change 163104 had a related patch set uploaded by Ori.livneh:
HHVM BetaFeature: Add screenshot; improve text

https://gerrit.wikimedia.org/r/163104
Comment 15 Gerrit Notification Bot 2014-09-26 06:39:50 UTC
Change 163105 had a related patch set uploaded by Ori.livneh:
HHVM BetaFeature: Add screenshot; improve text

https://gerrit.wikimedia.org/r/163105
Comment 16 Gerrit Notification Bot 2014-09-26 06:40:17 UTC
Change 163104 merged by jenkins-bot:
HHVM BetaFeature: Add screenshot; improve text

https://gerrit.wikimedia.org/r/163104
Comment 17 Gerrit Notification Bot 2014-09-26 06:41:00 UTC
Change 163105 merged by jenkins-bot:
HHVM BetaFeature: Add screenshot; improve text

https://gerrit.wikimedia.org/r/163105
Comment 18 Ori Livneh 2014-09-26 07:08:03 UTC
Done! Thanks very much for this, James and Jared.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links