[10:03:25] New review: John Erling Blad; "env 'MW_INSTALL_PATH=/var/www/repo' php phpunit.php --verbose --group Wikibase --conf /var/www/repo/..." [mediawiki/extensions/Wikibase] (master); V: -1 C: 0; - https://gerrit.wikimedia.org/r/39229 [10:23:44] anybody has an idea why phpunit doesn't work: http://pastebin.com/mujxsxYm ? [10:27:29] AnjaJ_WMDE: i'm trying all the tests [10:28:10] looks like maybe namespace settings are not right? [10:31:42] Change merged: John Erling Blad; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42254 [10:44:15] New patchset: Aude; "Remove shared table setting from example settings" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42528 [10:44:20] Nikerabbit: hi! could you give a quick feedback on https://gerrit.wikimedia.org/r/#/c/42248/ ? [10:44:36] Chris wants your input on that, it seems. Or, if you feel like it, just merge it. [10:45:40] any admins about? https://www.wikidata.org/wiki/Wikidata:Requests_for_deletions#User:AlbertMiller02 [10:45:41] AnjaJ_WMDE: "unsupported operand types"? huh. Maybe Nikerabbit also has an idea abotu that one... [10:46:14] Franco|Andro: looks like the page is already gone [10:47:15] oh. huh. looks like Stryn got it right after I wrote that [10:47:34] AnjaJ_WMDE: i don't see any issues with the example settings [10:47:38] DanielK_WMDE__: If you got some spare time could you maybe have a look at https://gerrit.wikimedia.org/r/34843 It's blocking a Wikibase change since November [10:48:40] hoo: looked at it briefly before, but didn't quite understand the issue. will look after lunch, probably. what is it blocking? [10:49:22] DanielK_WMDE__: https://gerrit.wikimedia.org/r/34662 MediaWiki will look weird with that patch applied but not the core one [10:51:44] hoo: is there a bugzilla ticket for either issue? [10:51:59] I don't think so [10:52:08] by the time I wrote the patch there wasn't [11:35:46] In my test setup changes in the repo is picked up but changes isn't injected in RC on client .. o_O [11:35:57] 'So what the heck is wrong? [11:36:25] Seems like it is according to the pages that describes this.. :( [11:36:44] aaaaaaaaaaaaaaaaaaaaaaude! [11:41:05] Jeblad_WMDE: do the links appear in the sidebar on the client? [11:44:08] aude but in the rc they should be listed anyhow? [11:48:15] New patchset: Jeroen De Dauw; "Added compatibility with Diff < 0.4 in Diff::unserialize" [mediawiki/extensions/Diff] (master) - https://gerrit.wikimedia.org/r/42532 [11:48:42] Jeblad_WMDE: they should be but the links int he sidebar do not depend on the pollforchanges now [11:48:54] they directly access the repo database [11:49:08] that would tell us if the database connection, at least works [11:49:17] Jeblad_WMDE: ^ should fix the issue you saw [11:52:50] http://pastebin.com/JfR92jXz [11:53:00] No database changes in the client, pollForChanges reports as it should [11:53:05] test failures and errors [11:53:06] No failure reported [11:53:44] Jeblad_WMDE: just because pollforchanges says a change is handled, sometimes it lies :o [11:54:09] Well, in my case it lies more that a politician [11:54:19] Jeblad_WMDE: is there stuff in the recentchanges table in the client with rc_type = 5 ? [11:54:31] it should insert stuff there [11:55:03] I have checked the two last entries, none of them comes from the repo [11:55:07] if not you may have to delete the .continue file that is in /tmp or /var/run [11:55:26] does it say starting from change #0 or something? [12:01:51] Change merged: John Erling Blad; [mediawiki/extensions/Diff] (master) - https://gerrit.wikimedia.org/r/42532 [12:02:00] New patchset: Aude; "move content model constants to lib, client needs to know about them" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42534 [12:05:14] aude, it says "99" [12:05:49] and it is called WBpollForChanges_client.continue [12:06:40] On start it says.. [12:06:43] 13:06:07 Loaded saved state: continue after change #99 [12:07:03] Jeblad_WMDE: then it will only show new stuff [12:07:49] * aude often removes the rc_type = 5 entries from my recent changes table, removes the .continue and lets pollforchanges re-populate stuff [12:07:58] It updates, but new stuff doesn't reach the client [12:08:04] just to see it works with my old entries inw b_changes [12:08:16] Jeblad_WMDE: nothing with rc_type = 5 in recentchanges table? [12:08:17] New patchset: Jeroen De Dauw; "Added test to check if serialization handling remains compatible with v0.3" [mediawiki/extensions/Diff] (master) - https://gerrit.wikimedia.org/r/42536 [12:08:35] * aude confused  [12:17:22] Change merged: John Erling Blad; [mediawiki/extensions/Diff] (master) - https://gerrit.wikimedia.org/r/42536 [12:17:59] New review: Jeroen De Dauw; "How is this related to namespaces?" [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/42534 [12:56:52] Change merged: John Erling Blad; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42528 [13:07:14] hoo: hm, how does the list of restriction types become empty? shouldn't it always at least contain "edit", at least in the wikidata case? [13:07:45] DanielK_WMDE__: A non-existing page can't be edited ;) [13:08:07] ah, I see [13:08:10] So in the item NS where we don't have create and upload it can become empty for non-existing items [13:12:09] wow, phpstorm takes like 5 minutes to reindex my working install [13:12:25] i guess java's gc got it's panties in a knot or something [13:12:37] I don't think it takes mine that long [13:12:38] * Reedy tries [13:12:49] no, it was the first time it took *that* long [13:13:02] Yeah [13:13:06] Just invalidated all caches [13:13:14] lot of disk io. maybe it decided to swap out thunderbird or something :P [13:13:29] heh [13:13:32] Yay for ssds [13:13:39] hmhm [13:14:14] my laptop is making a lot of hot air now [13:15:00] is that all extensions in our repo too? [13:15:18] no, just a hand full [13:15:36] 2 minutes and it's over halfway there... [13:15:42] hehe [13:16:30] done [13:16:38] 3 and a half minutes [13:17:00] for all extensions + core, tools, [13:17:07] PHP loaded with numerous pecl modules [13:17:10] oh, on an EAP.. [13:17:23] you got SSD and more RAM. sounds about right. [13:18:07] i5-2520M @ 2.50GHz and 8GB ram [13:18:12] on Windows too! ;) [13:19:04] http://pastebin.com/Vi3kH0MH when creating the HTML reports [13:19:25] only using ~ 0.7GB ram [13:19:49] AnjaJ_WMDE: you broke it [13:29:22] New review: Daniel Kinzler; "Works as expected with I0e5d3cfb, can be merged once that is in." [mediawiki/extensions/Wikibase] (master); V: 2 C: 1; - https://gerrit.wikimedia.org/r/34662 [13:30:57] hm, Message::__toString() should probably catch any exceptions to avoid fatal errors. [13:57:52] New patchset: Reedy; "Followup to 36780, add new columns FIRST" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/37485 [13:58:21] ^ Someone want to approve that :p [14:21:59] New review: John Erling Blad; "On my own test repo, when I first create sitelinks for Berlin in en, de and no as anonymous, and the..." [mediawiki/extensions/Wikibase] (master); V: -1 C: 0; - https://gerrit.wikimedia.org/r/39229 [14:23:12] New review: John Erling Blad; "Note that the undo operation was accessed from the links at the top of the page after a diff in the ..." [mediawiki/extensions/Wikibase] (master); V: -1 C: 0; - https://gerrit.wikimedia.org/r/39229 [14:23:40] New review: John Erling Blad; "Rollback works when accessed at the history page." [mediawiki/extensions/Wikibase] (master); V: -1 C: 0; - https://gerrit.wikimedia.org/r/39229 [14:34:00] New review: Hoo man; "Re to Jeroen" [mediawiki/extensions/Wikibase] (master) C: 0; - https://gerrit.wikimedia.org/r/34662 [14:35:55] New review: John Erling Blad; "Double checked and the undo function works as expected when this changeset isn't included." [mediawiki/extensions/Wikibase] (master); V: -1 C: 0; - https://gerrit.wikimedia.org/r/39229 [14:52:31] Change merged: Daniel Werner; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42091 [14:53:47] DanielK_WMDE: So... shall I remove that one line reindexation thingy? [14:54:25] yea, nicer that way [14:56:11] ok, doing [14:57:45] New patchset: Hoo man; "Add TitleGetRestrictionTypes hook" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/34662 [14:58:03] New review: Hoo man; "Removed useless clutter" [mediawiki/extensions/Wikibase] (master) C: 0; - https://gerrit.wikimedia.org/r/34662 [15:05:11] fancy new code coverage (repo only jeroen ;) http://wikidata-docs.wikimedia.de/testcoverage/phpcoverage/20130107/ [15:05:31] you can hover over the lines and see the tests executing these lines [15:06:43] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [15:07:44] AnjaJ_WMDE: oh, that *is* very pretty, thanks! [15:08:36] hola [15:08:50] poor mediawiki core :( [15:08:53] looking at test2. [15:09:03] New review: Tobias Gritschacher; "removing the do-not-submit as the required core-patch was merged" [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/34662 [15:09:08] EditPage.php [15:09:09] 0.00% [15:09:10] :o [15:09:17] hi aharoni [15:09:35] Wikidata seems to work, but the "edit links" link still leads to the repo... are there any plans to do something more clever with the links adding workflow? [15:09:57] http://wikidata-docs.wikimedia.de/testcoverage/phpcoverage/20130107/includes_specials.html looks sad [15:10:17] aharoni: yes [15:10:46] unfortunately not for HU wikipedia but maybe after that [15:10:51] OK. [15:12:04] AnjaJ_WMDE: on http://wikidata-docs.wikimedia.de/testcoverage/phpcoverage/20130107/includes_specials.html [15:12:12] why is SpecialRecentchanges and SpecialWatchlist missing [15:12:38] oh, setup of suite.xml on dev-9 [15:12:52] there needs to be a whitelist in there for the reports [15:13:23] we will upgrade the local test server and create test reports again from here instead of dev-9 [15:13:32] to much resources needed [15:13:44] ok, it's just runnign wikibase tests [15:13:45] ? [15:14:04] grrr, but ok [15:18:54] New review: Hoo man; "Tested again: Still works fine (against latest Wikibase and core master)" [mediawiki/extensions/Wikibase] (master) C: 0; - https://gerrit.wikimedia.org/r/34662 [15:23:44] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [15:34:59] Change merged: Daniel Kinzler; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/34662 [16:04:29] New patchset: Daniel Werner; "(minor) Introduced new base widget for template based widgets" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42349 [16:06:20] New patchset: Daniel Werner; "(minor) Introduced new base widget for template based widgets" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42349 [16:06:33] New review: Tobias Gritschacher; "When editing a statement and changing the value, the old value is sent to the API instead of the new..." [mediawiki/extensions/Wikibase] (master); V: 0 C: -1; - https://gerrit.wikimedia.org/r/42100 [16:17:21] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [16:26:01] Change merged: Henning Snater; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42349 [16:36:50] New patchset: Henning Snater; "Simplified xpath declarations in alias Selenium module" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42569 [16:40:15] Silke_WMDE_: [16:43:33] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [16:45:14] here o/ [16:45:54] Silke_WMDE_: are you a developer? [16:46:02] or do you know one? [16:46:48] no and yes [16:47:29] so. i want to help developing the wikibase extension but I need someone who can help me a bit [16:49:38] You ought to talk to Lydia_WMDE. She has collected information on how/where to start. [16:49:59] I already did [16:50:07] but I think she isn't here [16:50:14] Lydia_WMDE: here? [16:50:15] She's on the phone right now [16:50:43] benestar: need another 5 or 10 mins [16:50:53] no problem ;) [16:51:24] benestar: what sort of stuff are you interested in? [16:51:36] well [16:51:49] * aude thinks some of the gadgets might be nice to have integrated in the extension [16:51:50] Everything I can help [16:51:55] aude: ;) [16:52:06] * aude not sure how simple it is [16:52:26] I think the extension should be able to be edited without javascript, too [16:52:32] like checking existing site links when you make a new item [16:52:41] benestar: yes! [16:53:06] * benestar howeber thinks, that everyone, who has javascript disabled, is a bit mad [16:53:10] *v [16:53:13] :o [16:53:27] well, maybe not mad [16:53:33] but you lose many features [16:54:05] aude: I also need some help with the technic [16:54:11] there is a git, right? [16:55:30] yes [16:55:36] gerrit :) [16:56:03] hi rschen7754 :) [16:56:06] hi [16:56:10] i'm in class [16:57:03] benestar: i see you have account on wikimedia labs and gerrit [16:57:32] yes [16:57:41] I run my bot on labs, so I got it, too [16:57:45] good [16:58:06] aude: I will have to create a clone of the git, right? [16:58:07] to use gerrit, you should be able login [16:58:11] and add your ssh keys [16:58:12] https://gerrit.wikimedia.org/r/#/settings/ssh-keys [16:58:18] benestar: yes [16:58:43] wikibase is here: [16:58:44] https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/Wikibase.git;a=summary [16:58:47] aude: should I create a new one? [16:58:58] or can I use my existing from labs [16:59:06] existing one should be ok [16:59:18] * aude uses git clone ssh://aude@gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase.git [16:59:21] so done .) [16:59:28] not the https link for cloning [16:59:38] ok [16:59:42] Change merged: Tobias Gritschacher; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42569 [16:59:52] the ssh one allows you to submit patches [17:00:03] I understand [17:00:13] aude: which program are you using? [17:00:35] for what? [17:00:43] for git clone [17:00:48] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [17:00:54] benestar: ok here now :) [17:00:59] Thieol_away: I tried wbeditentities. It created an empty item. [17:01:00] benestar: you use linux, mac, windows? [17:01:07] windows [17:01:10] ok [17:01:39] * aude not very familiar with windows but think TortoiseGit would work [17:01:48] TobiasG_WMDE might have better advice [17:02:24] I found one on my pc [17:02:26] New patchset: John Erling Blad; "Work on cleaning up EntityDiff" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/39229 [17:02:31] lol [17:02:42] benestar: have you already looked at http://www.mediawiki.org/wiki/Git ? [17:03:23] benestar: look at http://code.google.com/p/tortoisegit/ [17:03:23] yes [17:03:27] http://www.mediawiki.org/wiki/Hackathon/Laptop_setup/Windows_git [17:03:32] http://www.mediawiki.org/wiki/Hackathon/Laptop_setup/Windows_git-review [17:03:36] those links might help [17:03:53] http://www.mediawiki.org/wiki/Hackathon/Laptop_setup [17:04:00] more stuff ^ [17:04:11] benestar: if you're fine with the git console you just need http://code.google.com/p/msysgit [17:04:34] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [17:05:09] getting git-review to work might be a bit tricky [17:12:24] TobiasG_WMDE: it is installed [17:12:45] New review: Tobias Gritschacher; "pressing ESCAPE key while adding statements does not work anymore." [mediawiki/extensions/Wikibase] (master); V: -1 C: 0; - https://gerrit.wikimedia.org/r/42100 [17:13:09] benestar: what else do you need? [17:13:22] I need to add the ssh key [17:15:25] is ssh-agent running? [17:15:42] hu? [17:16:13] it should be installed with msysgit [17:16:19] ssh-agent allows you to store your passphrase [17:16:36] so try to run ssh-add path/to/key in your git bash [17:17:24] https://help.github.com/articles/working-with-ssh-key-passphrases#platform-windows [17:17:25] New patchset: John Erling Blad; "Work on cleaning up EntityDiff" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/39229 [17:19:24] aude: Could not open a connection to your authentication agent. [17:20:01] benestar: run ssh-agent.exe in your git bash [17:20:23] New review: John Erling Blad; "There are still 2 failures" [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/39229 [17:20:34] k [17:20:35] if it is not installed, reinstall msysgit and check to install ssh-agent [17:20:49] i think it is installed [17:21:03] but ssh-add does not work [17:21:04] everything you need is written on http://www.mediawiki.org/wiki/Git/Workflow#Install_.26_configure_Git [17:21:20] and on http://www.mediawiki.org/wiki/Git/Workflow#Windows [17:21:41] I installed it the same way as it is described there and it worked [17:23:11] ssh-add -l ? [17:23:41] if key is not there, then ssh-add "pathtothepublickey" [17:27:39] New review: Aude; "I don't see any problems in the client with this patch. Deleting / restoring items, updating site l..." [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/39229 [17:28:37] aude: I think I will create a new key [17:28:49] what is the command? [17:29:19] ssh-keygen ? [17:29:33] thx [17:30:02] disclaimer that i haven't used windows in a while but that should work i think [17:30:32] worked [17:30:34] good [17:32:48] aude: :D [17:33:15] can you clone the git repo? [17:34:20] just working [17:34:29] ok [17:35:04] 250 KiB/s [17:35:07] ah [17:35:49] done [17:35:56] when it's done, you will also need to clone the Diff and DataValues extension for things to work [17:36:03] and optionally UniversalLanguageSelector [17:36:32] I will have to install mediawiki? [17:36:42] yes [17:36:51] puh [17:37:10] well, to be able to see how things work [17:37:35] ok, I will do this later [17:37:55] can you first tell me how to add a change to the git? [17:37:55] we, for example, have some "experimental" features that are disabled on wikidata.org as they are not ready yet [17:38:42] how about editing our README [17:38:42] https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/extensions/Wikibase.git;a=tree [17:38:42] where do you have them? [17:39:02] it could use mention that the extension requires Diff and DataValues, for example [17:39:19] ok, lets do this :) [17:39:33] we have stuff in repo/Wikibase.experimental.php [17:39:37] for example [17:39:48] that's not included by default. only with a setting [17:39:54] add a new section? [17:39:58] sure [17:40:03] -> == Requirements == [17:40:07] sure [17:41:13] aude: which extensions are required? [17:41:21] Diff and DataValues [17:42:03] == Requirements == [17:42:05] To use this extension, it is required to install the Diff and DataValues extension and optionally the UniversalLanguageSelector. [17:42:11] sure [17:42:24] the language selector though, is only for the repo [17:42:35] i will remove it [17:42:38] ok [17:42:52] how to upload teh changes? [17:42:54] http://www.mediawiki.org/wiki/Hackathon/Laptop_setup/Windows_git-review [17:42:55] *the [17:43:00] you need git-review [17:43:14] so I need python? [17:43:18] yes [17:43:21] hope I have installed it [17:43:26] * aude hopes it just works :) [17:43:36] aude: how can I quickly check this? [17:43:57] found [17:44:02] good [17:44:08] python 2.6 [17:44:12] is this ok? [17:44:14] should be ok [17:44:47] http://pypi.python.org/pypi/git-review [17:45:06] * aude looks for a new version of git-review than the one on the wiki [17:45:46] probably the git-review-1.20.tar.gz link on http://pypi.python.org/pypi/git-review is ok [17:46:02] aude: will come back [17:46:06] ok [17:46:07] now eating ;) [17:46:19] * aude going home soon and eating too [17:46:26] will be around in ~1 hour [18:16:50] aude: back :) [18:27:59] aude: still here? [18:43:18] benestar: she went home already [18:43:24] may be back later [18:43:29] oh [18:43:39] I am trying on my own :p [18:43:51] good :) [18:45:06] !log [18:45:06] Publicly logged channel — logs at http://bots.wmflabs.org/~wm-bot/logs/%23wikimedia-wikidata/ [18:46:23] Lydia_WMDE: Hooray! It is properly installed. [18:46:29] wohooooo! [18:46:29] by http://www.mediawiki.org/wiki/Hackathon/Laptop_setup/Windows_git-review [18:46:34] awesome [18:47:03] do you know how to go on now? [18:47:33] do you have a local wikidata set up? [18:48:46] no [18:48:51] I want to do this later [18:49:03] I already have created a change with aude [18:49:08] ah [18:49:12] we changed some lines in the README file [18:49:16] ok [18:49:16] secd [18:49:18] and now I want to upload it [18:49:26] (just to see how it works) [18:49:53] benestar: then you want to follow https://www.mediawiki.org/wiki/Git/Workflow#How_to_submit_a_patch [18:49:55] Lydia_WMDE: and now I don't know how to go on :( [18:50:08] Lydia_WMDE: <3 [18:50:12] ;-) [18:50:14] no problem [18:56:28] Lydia_WMDE: I don't understand this [18:56:31] :/ [18:57:28] benestar: i need to go to my last meeting for today now but can you maybe tell where you are stuck? then JeroenDeDauw or Jeblad_WMDE can maybe help you [18:57:46] ok [18:57:48] huh? [18:57:55] or I will wait for aude [18:58:00] hi Jeblad_WMDE :) [18:58:14] I am just trying to create my first edit on gerrit [18:58:18] You called, queen of the dungeons ... [18:58:32] hehe, and then she left.. [19:00:24] Okey, you are an registered user? [19:00:30] yes [19:00:38] Bene [19:00:49] have you paid your tribute to the God of Gerrit? [19:00:59] hu? [19:01:32] (You can send us chocolate, but at least 50kg) [19:01:40] lol [19:02:02] aude told me everything until installing gerrit review for python [19:02:20] You must have an account, and I have no clue on how to set up any additional stuff [19:02:44] :/ [19:02:46] After you get things up and working you usually use shell commands to do the rest [19:03:11] Jeblad_WMDE: I am now here: https://www.mediawiki.org/wiki/Git/Workflow#How_to_submit_a_patch [19:03:28] Yes, that is basically everything [19:03:43] First you can try is to check out core [19:03:58] Then you need Wikibase [19:04:18] Jeblad_WMDE: here my error [19:04:23] https://meta.wikimedia.org/wiki/Wikidata/Development/Setup [19:04:37] I try to execute "git pull --rebase gerrit master" [19:05:04] ugh.. [19:05:08] and it gives me "fatal: Not a git repository (or any of the parent direcotries): .git [19:06:18] moment [19:07:23] https://meta.wikimedia.org/w/index.php?title=Wikidata/Development/Setup&oldid=4192670 [19:07:49] Do a git clone into a directory [19:08:13] then when you want to pull, do a "git pull" [19:08:31] If the repo sint set up correctly you will get an error message [19:09:00] A git pull with rebase isn't something I do.. ;) [19:09:16] chocolate? [19:09:17] +2 [19:09:26] I'll even merge it :) [19:09:37] If you need to do strange things you must either briebe one of the Gods or read the man pages [19:10:37] hi benestar [19:10:49] I guess you try to do something like "git pull --rebase origin/master" [19:10:49] aude: :DD [19:11:11] aude: gerrit review is installed [19:11:28] but I cannot upload our changes [19:11:57] ok, one second [19:12:22] can you do git remote -v [19:12:44] let me try [19:13:17] aude: fatal: Not a git repository (or any of the parent directories): .git [19:13:28] hmmm [19:13:46] sure you are in the correct directory? [19:14:19] yes [19:15:06] and you did git clone ssh://bene@gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase.git [19:15:25] * aude assumes your name is bene [19:16:00] true [19:16:00] it also shows https://gerrit.wikimedia.org/r/p/mediawiki/extensions/Wikibase.git on the gerrit page, but i choose the ssh one [19:16:12] I did it with the ssh [19:16:23] so there is a folder called Wikibase [19:17:09] did you cd into the Wikibase folder? [19:17:10] Where's Jeroen when I want to glare at him? [19:17:36] aude: doing [19:17:40] ah, ok [19:17:47] that would be the reason i think it didn't work [19:17:52] lol [19:18:07] now try git status [19:18:44] Jeroen is... [19:18:47] :D [19:18:56] onoez [19:18:59] aude: modified: README [19:19:01] Reedy: you called? [19:19:05] benestar: ok [19:19:11] now git add README [19:19:17] I think that's my edit [19:19:21] ok [19:19:21] yes it is [19:19:31] k [19:19:38] jeroendedauw_: https://gerrit.wikimedia.org/r/42590 [19:19:49] * Reedy glares at jeroendedauw_ [19:19:57] aude: what next? [19:20:04] then do either "git commit" and it will bring up a text editor where you enter your commit summary [19:20:11] or, if you commit summary is short [19:20:22] Reedy: lolol :D [19:20:29] Reedy: I do think of them [19:20:29] then "git commit -m mycommitsummary" [19:20:33] with lots of pity [19:20:37] * jeroendedauw_ hides [19:20:52] -m attaches the summary, but sometimes you want to provide more detailed summary than one line [19:20:52] 04Error: Command “m” not recognized. Please review and correct what you’ve written. [19:20:57] hah [19:21:07] * aude stabs AsimovBot  [19:22:37] aude: hmm [19:22:48] can there be a space in the summary? [19:23:16] no [19:23:16] only one! [19:23:21] you can put quotes around it [19:23:24] * Jeblad_WMDE hides [19:23:40] or if you use the text editor, then yes [19:28:26] * jeroendedauw_ waves at Tpt [19:29:26] jeroendedauw: Hi! [19:37:03] aude: did it work? [19:37:50] benestar: can you do "git log" [19:38:08] if the commit works, you should see it in the list [19:38:11] aude: yeah [19:38:23] but it didn't appear in this chat [19:38:29] one more step [19:38:32] "git review" [19:39:50] New patchset: Bene; "Added requirements to README" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42592 [19:39:57] :D [19:40:02] yay! :) [19:40:10] aude: thanks a lot!!! [19:40:23] this is fine for now, but in the future you want to create a "topic branch" locally [19:40:39] ok, so how to do this? [19:42:03] my notes are rough, but https://www.mediawiki.org/wiki/User:Aude/Git#Branching [19:42:10] aude: and when will my change be added to the extension? [19:42:17] benestar: shortly [19:42:36] if i'm on master (check it by doing git status) [19:42:47] then i do git checkout -b mycoolfeature [19:42:56] make my changes there and commit them and all [19:43:02] git review (from the branch) [19:43:34] https://gerrit.wikimedia.org/r/#/q/project:mediawiki/extensions/Wikibase,n,z [19:43:44] and why to tdo this? [19:43:46] that's how you see "entitydiff" for jeroen's change, for example [19:44:27] it allows you to work on multiple things at once, makes it easier when you want to modify a patch [19:44:36] so he did git checkout -b entitydiff ? [19:44:48] probably [19:44:54] it's not the only way to do it [19:45:01] ah, I think I understand [19:45:40] New review: Aude; "looks good :) Thanks Bene!" [mediawiki/extensions/Wikibase] (master); V: 2 C: 2; - https://gerrit.wikimedia.org/r/42592 [19:45:41] Change merged: Aude; [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42592 [19:46:07] now when you want someone to review your patch, it helps to add some reviewers on gerrit [19:47:27] ok [19:47:41] can I also review other's code? [19:47:49] yes [19:48:52] and how? [19:52:02] aude: how? [19:52:10] like here, https://gerrit.wikimedia.org/r/#/c/41926/ [19:52:17] just click the review button [19:52:54] where? [19:53:04] below the patchset [19:53:08] found [19:53:18] aude: how can I see the diff? [19:53:27] Side-by-Side [19:54:13] if you actually want to try the patch, then.... [19:54:17] git review -d 41926 [19:54:33] aude: can you explain me the change? [19:54:35] and the git checkout master to get back to master [19:55:05] umm... we're inserting entries in the recent changes table in the client [19:55:17] k [19:55:23] https://test2.wikipedia.org/w/index.php?title=Special:RecentChanges&hidewikidata=0 [19:56:04] i'm just assigning the page id from the client in the table, instead of the page id from the repo [19:56:28] the watchlist does queries which take in to account page id [19:56:38] uff [19:57:06] I think I will need some time still I have understand Wikibase [19:57:11] if you are more interested in the javascript stuff, then henning and Danwe_WMDE 's stuff would be more interesting [19:57:45] the client is a bit trickier to get working, since there's a synchronisation script, and multiple databases to access [19:58:10] you can install the wikibase repo on a wiki and have just that, and it's like wikidata.org [19:59:36] aude: do you know any not too hard bug/feature I can add for the beginning? [20:00:13] one sec [20:01:59] http://bit.ly/13bXb0v is a list of bugs tagged as "need-volunteer" but not necessarily all easy [20:02:50] widget for adding links to wikidata from wikipedia pages [20:02:53] aude: is there one where you could answer me a question? [20:03:08] Jeblad_WMDE: please explain [20:03:59] aude: just in case I don't find a solution [20:04:16] benestar: or maybe your site link thing on the create item page [20:04:27] maybe that could be incorporated into the special page itself, rather than a gadget [20:04:28] oh, that's a good idea [20:04:54] there is a special page already [20:05:17] but no field at [[Special:CreateItem]] [20:05:17] http://www.wikidata.org/wiki/Special:CreateItem [20:05:54] right [20:06:01] We need a widget to add a new wikipedia-page to either the same item as another page or to a new one [20:06:39] Don't send wikipedians to CreateItem, that page is weird for them [20:06:58] aude: but I think it is possible to add a sitelink field: [20:07:15] [[Special:CreateItem?title=Abc]] [20:07:15] http://www.wikidata.org/wiki/Special:CreateItem?title=Abc [20:07:18] Let them search for a similar page from wikipedia [20:07:20] actually, when you start at special item by title [20:08:30] k [20:08:40] one sec [20:08:45] ( tI mean this: http://www.wikidata.org/wiki/Special:CreateItem?site=enwiki&page=Abc ) [20:08:46] * aude struggles to find items that don't exist [20:09:00] ah, yes [20:09:55] same search as ItemByTitle initially, but if that doesn't work use ItemDisambiguation, or something similar [20:14:21] aude: did you find one? [20:19:44] aude: still there? [20:20:20] DanielK_WMDE: https://bugzilla.wikimedia.org/show_bug.cgi?id=40157 [20:20:40] here [20:21:07] aude: is thie still actual? https://bugzilla.wikimedia.org/show_bug.cgi?id=40157 [20:21:56] hello kondi [20:22:02] glad that it worked [20:22:11] the empty property is probably not too hard to do [20:22:26] i tried to see what you have done but i do not know you're wiki user ? [20:22:34] kondi does not exists [20:22:45] aude: so where is this class? [20:23:02] Thieol_away: oh, it's User:Kondicherry [20:23:20] benestar: there is an EditEntity class in the repo [20:23:31] ah, editentity [20:23:38] ok i'm having a look [20:24:27] Jeblad_WMDE knows more about than i do [20:25:01] there's also a bunch of Content and ContentHandler classes in the repo [20:25:19] ItemContent for example (and it's a subclass of EntityContent) [20:25:27] Thieol_away: it basically created a blank item, id:Q2672515. I've updated it now. [20:25:48] * aude sees a save() method in EntityContent [20:26:04] or hooks might be used to accomplish this [20:26:07] aude: but what is EntiyObject::isEmpty() ? [20:26:20] * aude thinks it's in the lib [20:26:24] WikibaseLib [20:26:55] looks like it got renamed to just Entity.php and that's in the Lib extension [20:27:14] kondi : is it Q2672515 ? [20:27:36] Thieol_away: yes [20:27:44] aude: so there should be two new methods? [20:28:16] isEmpty already exits :/ [20:28:52] isEmpty exists but not yet used to save the page prop [20:29:15] we could also use an isStub method for items i think [20:29:37] I am creating it just now [20:29:48] aude: what is wfProfileIn ? [20:31:07] aude: when is an entity a stub? [20:32:32] benestar: we have profiling points to help see which parts of the code are slow [20:32:41] http://wikidata-test-repo.wikimedia.de/w/profileinfo.php (not fully enabled or used, it appears) [20:32:46] ah, ok [20:32:47] but can be [20:33:02] about the stub, i would ask DanielK_WMDE what he thinks [20:33:03] or Jeblad_WMDE [20:33:14] but maybe something that has no sitelinks [20:33:20] DanielK_WMDE: poke [20:33:24] Jeblad_WMDE: poke [20:33:33] that way we could have a special page that tells us which items have no site links [20:33:36] There are no well-defined stubb [20:33:49] :/ [20:34:19] An idea was to say that an item with no statements is a stub [20:35:43] Jeblad_WMDE: statement == clame ? [20:35:47] *claime [20:35:52] yes [20:36:08] statements are a subclass of claims [20:36:53] Jeblad_WMDE: so I just have to check if there are claims [20:37:00] and if so, I will return false [20:37:01] kondi : just seen in the example how to do it ... [20:37:03] otherwise true [20:37:10] in fact, that is really simple ! [20:37:22] haven't looked at it yet ! [20:37:23] Yes, but you can't because it is to heavy [20:37:36] hu? [20:37:46] there is a function called hasClaims [20:38:16] Thieol_away: it is easy but doesn't work :\ [20:38:23] Jeblad_WMDE: doesn't this function work? [20:38:32] what's the matter ? [20:38:37] Yes, but that is a function that imply parsing the item [20:38:53] what does it mean? [20:39:07] You can't get 5000 items and parse them, it will kill the server [20:39:17] ouch [20:39:18] Thieol_away: The nasty hack works but it only creates an empty item. [20:39:36] I think i'll give it another try [20:39:40] that is not possible ! [20:39:48] Jeblad_WMDE: :( [20:39:59] It could be that there are some smart ways around this, I'm not sure [20:40:19] I would be quite glad [20:40:21] if you create the item and fill labels for instance that should work because it works fine for me with the save function ? [20:40:33] Its possible to do a lot of smart stuff, ask Aude or jeroendedauw_ [20:40:38] did the wikidata staff already registerd: https://www.wikidata.org/wiki/Wikidata:Contact_the_development_team#Impossible_to_set_a_label [20:40:41] Or DanielK_WMDE [20:41:23] DanielK_WMDE: poke [20:41:51] Thieol_away: you created the item first right? [20:41:55] Daniel will probably be around tomorrow but not tonight :/ [20:41:56] Sk1d: i saw it. But better file a report on bugzilla. I thought this had been taken care of a couple of weeks ago by another change, but apparently not. [20:42:01] ah [20:42:03] :) [20:42:08] * DanielK_WMDE wibbles [20:42:19] but i'm only half here. time for a movie soon. [20:42:34] New patchset: Daniel Werner; "(bug 43304) Mechanism for supporting different Snak types in the user interface" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42100 [20:43:10] this error occure only since the last update [20:43:12] no...as i was not sure it worked i did not try [20:43:56] benestar: well, if we want a page_prop for items with no sitelinks, then [20:43:59] DanielK_WMDE: did you follow the discussion? [20:44:03] but if you do what is mentionned in example.py, should work ????!!! [20:44:10] we have the wb_items_per_site table [20:44:16] we are talking about https://bugzilla.wikimedia.org/show_bug.cgi?id=40157 [20:45:01] Thieol_away: I tried that only but it didn't work [20:45:09] ah ok ! [20:45:29] DanielK_WMDE: label-description starts to generate conflicts now as the number of items grows [20:45:57] Its not dublets as in our previous failures [20:46:04] Jeblad_WMDE: yes, but i thought we removed the uniqueness constraint all together. [20:46:15] kondi : are you sure that present bots do not fill items with only one link ? [20:46:20] No, its added as soft constraints [20:46:33] Sk1d: so, the acutal problem isn't the conflict, but that there is no error shown, right? that's definitly a bug. [20:46:40] Or I think it is.. [20:46:54] Thieol_away: pretty sure, yes [20:47:41] Jeblad_WMDE: it sucks as a soft constraint, because you can have two conflicting items in the DB then. These can then no longer be changed, because trying to save them will fail on the uniqueness constraint. [20:47:59] agree [20:48:27] benestar: i read what's on that page, yes. seems like the conflict causes the save to abort, but doesn't report an error. that sucks. please put it on bugzilla [20:48:28] https://bugzilla.wikimedia.org/show_bug.cgi?id=43720 [20:48:44] kondi : just seen that 3 forks exists on github but i cannot see them ( don't know why ). Have you investigate them ? [20:49:04] * kondi looks [20:49:25] Jeblad_WMDE: we can have the soft constraint for creation only. this will catch the usual case (creating somethign that already exists), but doesn't cause problems if two conflicting items are in the DB. [20:51:00] Thieol_away: I saw those forked repos. They haven't been updated. [20:51:14] since ? [20:51:21] A warning like on the image upload pages? [20:51:37] Intercepting the save, but let the user continue? [20:51:44] kondi : what is the latest update ? [20:52:04] Thieol_away: the last update was about 3 months ago [20:52:25] kondi : ok. so it is not in line with the api [20:52:55] kondi: you would like to use the pywikidatabot? [20:53:02] kondi : i think i am going to try this because i am interested in your idea for french wikipedias [20:53:15] Thieol_away: no. but i tried a quick hack which, as expected, didn't work well [20:53:25] Sk1d: yes! [20:53:39] was your hack taken from the api ? [20:53:49] i develeoped my bot from this [20:53:57] what do you want to do? [20:54:01] i mean the new release ? [20:54:32] i would have upated the git but till end of february i dont have enoght time to check the code... [20:54:44] Thieol_away: yes, I used wbeditentities [20:54:46] Jeblad_WMDE: could, but we could also just refuse. But only on item creation. So it's always possibel to modify existign items. We don't run into a deadlock. [20:55:06] Sk1d: simple. bot addition of items that are unique to certain wikipedias [20:55:33] hm ok my bot currently adds all pages [20:55:36] in ns0 [20:56:12] ns0 = mainspace? [20:56:43] yes [20:57:20] yes [20:57:29] kundi : have you still got the link to the wikidata api....i lost it ! [20:58:05] i will reconfigure my bot the next week to start to update from the fr-wikipedia as my bot "completed" the german wikipedia [20:58:13] Thieol_away: yopu mean https://www.wikidata.org/w/api.php ? [20:59:28] Sk1d: would be interesting to have a list of pages that have the same title on de and fr, but the de page has no fr link, and the fr page has no de link. [20:59:30] Sk1d: thanks ! [20:59:43] these should be good candidates for missing interwiki links... [20:59:46] Sk1d: cool. guwp is relatively small [21:00:06] finding those before importing "unique" pages would avoid duplucates in wikidata [21:01:53] DanielK_WMDE: i am not running a interwiki bot so i dont know [21:02:26] could be done on the toolserver, without too much pain... [21:03:04] The special page I was toying arund with could do that.. ;) [21:03:06] DanielK_WMDE: talk to Merlissimo as he has the possebiltity to accsess the database over the toolserver he might create this list [21:03:47] kondi: i could start with guwp if you like. [21:04:25] Sk1d : does your bot test for disambiguation page and fill description if so ? [21:04:42] Sk1d: yes of course! [21:04:49] ok ! great ! [21:05:06] how many articles has guwp [21:05:21] 22k+ [21:05:35] k [21:06:50] Sk1d : sorry thought kundi answer was yours to my question ! [21:08:42] Sk1d : * sorry, but i thought that kundi's answer was yours to my question ! [21:09:37] Thieol_away: yep but at creation only if a link exists in the languages listede here: https://www.wikidata.org/wiki/Wikidata_talk:Labels_and_descriptions_task_force#Bot_setting_automatic_disambiguation_description [21:10:39] i have also a secund instance running which starts in on language (currently en) checks all pages in the disambuigation category sets the correct description and if no interwikilink exists also sets the descriptions [21:10:59] for other languages [21:11:57] this might be improved in the future to list up conflicts becaus i think if one item is a disambuigationpage all items should be, but this is currently not implementet [21:13:37] do you know the description for french ? [21:14:31] Sk1d : for disambiguation page ? [21:16:18] https://www.wikidata.org/wiki/Wikidata_talk:Labels_and_descriptions_task_force#Bot_setting_automatic_disambiguation_description [21:16:21] https://www.wikidata.org/wiki/Wikidata_talk:Labels_and_descriptions_task_force#Bot_setting_automatic_disambiguation_description [21:17:48] Sk1d : just seen that Eric-92 told it to you. All's fine then/ [21:18:10] page d'homonymie [21:18:21] ? [21:18:21] yes , that's it. [21:19:43] Sk1d : hope kundi will give the right description for gu .... [21:21:31] Sk1d : wanted to do it in my bot but leave it to you then ? [21:25:46] ok [21:25:52] nice [21:27:39] Thieol|afk: do you know his wikidata username? [21:28:31] Sk1d: mine? [21:29:57] kondi|afk: yep [21:30:06] it's Kondicherry [21:30:46] ok could should i add a descpription for disambuigatinpages in gu [21:31:46] btw see https://www.wikidata.org/wiki/Q2720542 (dont worry labels should be added my bot only removes labels with "/" as this isnt correct most of the time in der german wikipedia) [21:32:29] Sk1d: સ્પષ્ટતા પાનું [21:32:32] * kondi|afk looks [21:33:41] Sk1d: looks fine :) [21:34:41] -.- i only got encoding bullshit could you add it to https://www.wikidata.org/wiki/Wikidata_talk:Labels_and_descriptions_task_force#Bot_setting_automatic_disambiguation_description [21:36:05] Sk1d: done [21:36:19] ok good [22:10:11] New patchset: Jeroen De Dauw; "Revert "Added requirements to README"" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42681 [22:38:24] New review: Aude; "no harm to have requirements info here." [mediawiki/extensions/Wikibase] (master); V: 0 C: -1; - https://gerrit.wikimedia.org/r/42681 [23:23:37] New patchset: Daniel Werner; "(bug 43304) Refactoring of Snakview variations registration" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42693 [23:27:33] New patchset: Daniel Werner; "(bug 43304) Added snakview variation definition for handling of NoValueSnaks" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42695 [23:33:35] New review: Jeroen De Dauw; "Disagree, having inferior (less complete, out of date) and possibly conflicting copies of the instru..." [mediawiki/extensions/Wikibase] (master); V: 0 C: 0; - https://gerrit.wikimedia.org/r/42681 [23:34:48] New patchset: Daniel Werner; "(bug 43304) Added snakview variation definition for handling of SomeValueSnaks" [mediawiki/extensions/Wikibase] (master) - https://gerrit.wikimedia.org/r/42697 [23:46:40] guerillero: I see you've joined the cup [23:46:51] I did [23:47:08] Good luck. [23:47:14] thanks [23:47:28] It is a reason to write a string of articles [23:47:44] Well, if you review my DYKs, I'll review yours. [23:48:03] I mean, I might review yours anyways, but... [23:48:19] I have one to write tonight [23:48:54] en.wikipedia.org/wiki/User_talk:Guerillero#Future_Projects [23:49:43] I so want to make straight edge a GA [23:49:50] but it is so much work [23:50:43] I see Toil is a DYK [23:50:44] yes [23:50:50] 2445 B (412 words) "readable prose size", so 5 points [23:50:56] that will be a GA soon [23:51:17] I would need to double the length to get 10 [23:51:30] The way it's looking, I won't feel safe about making the second round with anything less than 35 points [23:53:54] I wonder if I could get straight edge to a GA by the end of break [23:54:02] a GA is how many points [23:54:04] ? [23:54:59] 30 [23:55:13] and the article is on 35 pedias [23:56:28] it would be 72 points [23:56:47] but I have been working on it since 2009 [23:56:51] so it wouldn't count [23:56:57] pff