00:03:37
|
*** burke has joined #openmrs
|
00:03:37
|
*** ChanServ sets mode: +o burke
|
00:24:03
|
*** lh has quit IRC
|
00:31:15
|
*** burke has quit IRC
|
00:31:22
|
*** burke has joined #openmrs
|
00:31:22
|
*** ChanServ sets mode: +o burke
|
01:54:34
|
<OpenMRSBot> Recent updates in the world of openmrs: New Changeset: OpenMRS (concept_mappings): concept_mappings: Adding concept source select option to the concept reference term form <http://feedproxy.google.com/~r/OMRStrunk/~3/isqHlOnCG_8/OpenMRS>
|
02:22:20
|
*** sunbiz1 has joined #openmrs
|
02:23:05
|
*** sunbiz1 has left #openmrs
|
02:25:05
|
*** sunbiz has quit IRC
|
02:46:07
|
*** burke_ has joined #openmrs
|
02:50:03
|
*** burke has quit IRC
|
02:50:33
|
*** upul` has joined #openmrs
|
02:50:33
|
*** ChanServ sets mode: +v upul`
|
03:16:34
|
*** djazayeri has joined #openmrs
|
03:16:34
|
*** ChanServ sets mode: +o djazayeri
|
03:16:34
|
*** djazayeri has quit IRC
|
03:18:06
|
*** gbastien has joined #openmrs
|
03:18:44
|
*** djazayeri has joined #openmrs
|
03:18:44
|
*** ChanServ sets mode: +o djazayeri
|
03:34:57
|
*** gbastien has quit IRC
|
04:00:23
|
*** surangak has joined #openmrs
|
04:03:56
|
*** sunbiz has joined #openmrs
|
04:03:56
|
*** ChanServ sets mode: +v sunbiz
|
04:18:17
|
*** surangak_ has joined #openmrs
|
04:42:46
|
*** burke_ has quit IRC
|
05:06:31
|
*** djazayeri has quit IRC
|
06:00:27
|
*** bryq has joined #openmrs
|
06:00:28
|
*** ChanServ sets mode: +v bryq
|
06:14:13
|
<surangak_> bryq, heloo
|
06:14:35
|
<bryq> surangak_: Hi :)
|
06:14:39
|
*** dkayiwa has joined #openmrs
|
06:15:03
|
<surangak_> bryq, I was wondering, did u submit that code to gsoc peope ?
|
06:15:34
|
<bryq> surangak_: Yes, I've done it
|
06:15:57
|
<surangak_> bryq, its basicaly a compressed tar file that they need, is it ?
|
06:16:18
|
<surangak_> bryq, can you please tell me the steps u used ?
|
06:16:43
|
<bryq> surangak_: Yes, with *.diff and README files in it
|
06:16:44
|
<surangak_> bryq, Im guessing that I need to do a diff, then create a patch for the diff and compres that ?
|
06:17:00
|
<surangak_> bryq, did u do using windows ?
|
06:17:16
|
<bryq> surangak_: No, I'm on Linux
|
06:17:32
|
<surangak_> oh, i seee..
|
06:17:33
|
*** mwogi has joined #openmrs
|
06:17:54
|
<surangak_> whats created by the diff ? is it like a patch ? :)
|
06:18:00
|
<bryq> surangak_: Was you working on a separate branch during this GSoC?
|
06:18:08
|
<surangak_> yep
|
06:18:34
|
<bryq> surangak_: Yea, diff is the same as a patch you might have attached sometimes to JIRA
|
06:18:52
|
<bryq> surangak_: Did you merge your branch into trunk?
|
06:19:05
|
<surangak_> bryq, yes, its up to date...
|
06:19:14
|
<bryq> surangak_: Good
|
06:19:36
|
<bryq> surangak_: After merging did you commit some more code?
|
06:19:42
|
<surangak_> so u basically compress and tar a patch file, is it :-)
|
06:19:53
|
<bryq> surangak_: Yes :)
|
06:20:10
|
<surangak_> bryq, great, thanks, let me try it out :P
|
06:20:36
|
<bryq> surangak_: You can get the diff from source.openmrs.org site
|
06:21:01
|
<bryq> surangak_: By selecting the revision where you merge the branch into trunk
|
06:21:07
|
<OpenMRSBot> Recent updates in the world of openmrs: New Changeset: OpenMRS (concept_mappings): Renaming ConceptReferenceMap back to ConceptMap <http://feedproxy.google.com/~r/OMRStrunk/~3/CauuW3dG9qY/OpenMRS>
|
06:24:22
|
*** mwogi_ has joined #openmrs
|
06:25:19
|
*** mwogi has quit IRC
|
06:27:56
|
<surangak_> bryq, im in source.openmrs, and looking at my branch.... how exactly do I get the diff, please :-)
|
06:28:37
|
<bryq> surangak_: Click "View as patch" and save the output
|
06:30:32
|
<surangak_> bryq, im not seeing the 'view as patch' :-(
|
06:30:35
|
<surangak_> im here https://source.openmrs.org/changelog/~br=obs-codes-expanded/OpenMRS
|
06:30:52
|
<OpenMRSBot> <http://ln-s.net/97k-> (at source.openmrs.org)
|
06:31:42
|
<bryq> surangak_: Hmm... But did you merge BRANCH INTO TRUNK or TRUNK INTO BRANCH ?
|
06:31:56
|
*** pascal` has joined #openmrs
|
06:32:06
|
<surangak_> i took latest changes from the tunk and put them into my branch...
|
06:32:43
|
<surangak_> oh i see, thats trunk into branch, is it ?
|
06:33:11
|
<bryq> surangak_: Yes, that's trunk into branch
|
06:33:30
|
<bryq> surangak_: This complicates things a little bit
|
06:33:38
|
<surangak_> bryq, ooops, how do i do the opposite ? i hadint done that :(
|
06:34:01
|
<surangak_> bryq, did u merge to trunk gradually, or all at once ?
|
06:34:25
|
<bryq> surangak_:You must talk with your mentor if the code is mature to be merged into trunk
|
06:34:39
|
<surangak_> bryq, oh i see
|
06:34:39
|
<bryq> surangak_:I've merged all at once
|
06:34:51
|
<surangak_> bryq, do we hav permission to merge into trunk ?
|
06:34:57
|
<surangak_> or do they have to do it ?
|
06:35:38
|
<bwolfe> we have to do it
|
06:35:47
|
<bryq> surangak_:I was commiting to modules and I have permission to all my project "branches, trunk"
|
06:35:48
|
<surangak_> oh, hi Ben
|
06:36:10
|
<surangak_> seems i have not merged branch to tunk, only the other way around :-(
|
06:36:26
|
<bryq> surangak_: But don't worry, you can create the *.diff without merging
|
06:36:41
|
<surangak_> bryq, thats a great relief :)
|
06:36:57
|
<bryq> surangak_: And you can give it to someone who has trunk commit privileges and he will do the merge
|
06:36:57
|
<surangak_> bwolfe, by "we" do u mean u guys, or me :-)
|
06:37:04
|
<bwolfe> hi suranga
|
06:37:38
|
<surangak_> bryq, ah, I get it, u r on modules, let me try that...
|
06:37:56
|
<bwolfe> surangak_, I mean any "full committer". see https://wiki.openmrs.org/display/docs/Subversion+Code+of+Conduct
|
06:38:02
|
<OpenMRSBot> <http://ln-s.net/97kC> (at wiki.openmrs.org)
|
06:38:27
|
<bryq> surangak_:I can give you a short guide how to create the *.diff. Want this?
|
06:38:27
|
<bwolfe> surangak_, but yes, if you have a diff/patch file that applies cleanly to trunk, whomever does the merging over will be very happy with you. :-)
|
06:38:57
|
<surangak_> bryq, unfortunately im on windows :-(
|
06:39:07
|
<bryq> surangak_:That's no problem
|
06:39:19
|
<bryq> surangak_:You will use only Eclipse
|
06:39:42
|
<surangak_> bryq, thats great, can u give me the steps please ?
|
06:39:49
|
<surangak_> thanks awfully
|
06:39:57
|
<bryq> surangak_: Okay
|
06:40:20
|
<bryq> surangak_: Do you have eclipse with "Subversive" plugin?
|
06:40:29
|
<surangak_> bryq, yeps
|
06:41:17
|
<surangak_> is it a compare with branch thing in eclipse ?
|
06:41:35
|
<bryq> surangak_: Good, then switch Eclipse to some fresh workspace and import project "openmrs trunk"
|
06:41:54
|
*** rafa has joined #openmrs
|
06:41:54
|
*** ChanServ sets mode: +v rafa
|
06:42:01
|
<surangak_> ok, and then I would ...
|
06:42:29
|
<bryq> surangak_: I do not know if "compare with branch" is good thing. Follow my steps, which are surely good :)
|
06:42:40
|
<surangak_> bryq, tell me the steps, and ill try them out
|
06:42:44
|
<surangak_> bryq, sure, will do
|
06:43:29
|
<bryq> surangak_: Then after you check out the trunk, right-click on the project and select "Team > Merge"
|
06:44:21
|
<surangak_> bryq, aha, im getting a dialog box with some options :)
|
06:44:49
|
*** muthah has joined #openmrs
|
06:44:50
|
<bryq> surangak_: Good, now wait a moment. I have to look on this dialog
|
06:45:03
|
<surangak_> bryq, sure
|
06:46:38
|
<bryq> surangak_: On the "URL" tab you select browse and select an url to your branch
|
06:47:18
|
<bryq> surangak_: In my example it is: https://svn.openmrs.org/openmrs-modules/metadatasharing/branches/publish-subscribe
|
06:47:24
|
<OpenMRSBot> <http://ln-s.net/97kG> (at svn.openmrs.org)
|
06:47:45
|
<bwolfe> dkayiwa, how goes that visit infinite loop?
|
06:48:15
|
<surangak_> bryq, hmm.. im not getting a "url" tab.. im getting something called a "collabnet merge"
|
06:48:19
|
<dkayiwa> bwolfe: on one of my attributes i had a maxOccurs value of 9999999
|
06:48:36
|
<bwolfe> dkayiwa, ahahah, so what was it doing?
|
06:48:36
|
<surangak_> bryq, with a list of merge input options
|
06:48:37
|
<dkayiwa> bwolfe: so it was not an infinite loop. it just had not finished :)
|
06:48:54
|
<bwolfe> was it adding 10 million elements to the page?
|
06:49:00
|
<dkayiwa> yes
|
06:49:00
|
<bryq> surangak_: It seems that you have different SVN plugin
|
06:49:13
|
<bryq> surangak_: Can you send me a screenshot of this dialog
|
06:49:17
|
<surangak_> bryq, hmm...I get it
|
06:49:28
|
<surangak_> bryq, I will have to update my stuff a bit I see
|
06:49:33
|
<dkayiwa> bwolfe: so i changed it to a lower value like 5 and all was well :)
|
06:49:42
|
<surangak_> shall I update everything and get back to you later on ?
|
06:50:10
|
<bwolfe> dkayiwa, nice. why'd you set it to 9999999 in the first place??
|
06:50:24
|
<bryq> surangak_: Wait moment
|
06:50:52
|
<dkayiwa> bwolfe: did not know what maxOccurs is for so i just entered any value. and never knew that it would get back to me :)
|
06:52:52
|
<bryq> surangak_: Yes, update your things :)
|
06:53:13
|
<bryq> surangak_: In particular, get "Subversive" as SVN plugin
|
06:53:22
|
<surangak_> bryq, thanks for your help, Ill have a go and get back to you :-)
|
06:53:48
|
<bryq> surangak_: You can get everything from "Eclipse marketplace" if you have latest Eclipse version
|
06:54:21
|
<bryq> surangak_: Bye now. I'll be available in few hours
|
06:54:30
|
<surangak_> bryq, i seem to be using subclipse...
|
06:54:35
|
<surangak_> bryq, sure, no problem
|
07:03:53
|
<dkayiwa> bwolfe: is today also a public holiday in Kenya?
|
07:04:01
|
<bwolfe> dkayiwa, no, tomorrow is though
|
07:04:08
|
<dkayiwa> oh
|
07:04:20
|
<dkayiwa> bwolfe: IDI?
|
07:04:39
|
<bwolfe> no idea
|
07:04:45
|
<dkayiwa> :)
|
07:04:49
|
<bwolfe> its a muslim holiday though, thats all I know
|
07:05:09
|
<bwolfe> but I follow the US schedule. So I'm taking Monday off for Labor Day
|
07:05:09
|
<dkayiwa> bwolfe: today it is in Uganda :)
|
07:05:31
|
<dkayiwa> bwolfe: oh i see
|
07:06:04
|
<bwolfe> dkayiwa, so are you taking the day off?
|
07:06:16
|
<dkayiwa> bwolfe: yes :)
|
07:06:25
|
<bwolfe> sawa sawa
|
07:06:48
|
*** muthah has quit IRC
|
07:07:33
|
*** muthah has joined #openmrs
|
07:15:47
|
<mwogi_> Is there a way to read connections to the database in openmrs?
|
07:17:30
|
*** muthah has quit IRC
|
07:17:42
|
*** muthah has joined #openmrs
|
07:24:21
|
*** muthah has quit IRC
|
07:24:59
|
*** muthah has joined #openmrs
|
07:25:08
|
*** bryq has quit IRC
|
07:28:26
|
*** muthah_ has joined #openmrs
|
07:28:26
|
*** muthah has quit IRC
|
07:28:28
|
*** muthah_ is now known as muthah
|
07:32:45
|
*** muthah has quit IRC
|
07:36:52
|
*** muthah has joined #openmrs
|
07:39:02
|
*** rafa has quit IRC
|
07:39:29
|
*** bryq has joined #openmrs
|
07:39:29
|
*** ChanServ sets mode: +v bryq
|
07:41:07
|
*** muthah_ has joined #openmrs
|
07:41:08
|
*** muthah has quit IRC
|
07:41:09
|
*** muthah_ is now known as muthah
|
07:58:07
|
<bwolfe> mwogi_, you have to get it from hibernate
|
07:58:13
|
<bwolfe> from the current session
|
08:03:37
|
<mwogi_> so it possible? i needed to implement it through the amrscustomization module
|
08:04:05
|
<bwolfe> mwogi_, yes. its sessionFactory.getCurrentSession().getConnection() or something
|
08:04:14
|
<bwolfe> you might need to create an inner method to be able to use the connection
|
08:04:27
|
<bwolfe> hibernate did something funky with the latest release where they didn't want to just give out the connection
|
08:13:08
|
*** dkayiwa has quit IRC
|
08:16:55
|
*** dkayiwa has joined #openmrs
|
08:22:11
|
<mwogi_> bwolfe, Maybe i will need sometime to study that. :P
|
08:29:57
|
*** pascal` has quit IRC
|
08:44:31
|
<surangak_> bwolfe, hi, i have a small clarification.....
|
09:03:42
|
*** pascal` has joined #openmrs
|
09:07:14
|
*** muthah has quit IRC
|
09:11:04
|
*** pascal` has quit IRC
|
09:15:16
|
<bryq> surangak_: Hi, I'm available for a while.
|
09:15:32
|
<bryq> surangak_: And I've got an idea how to quickly solve your problem
|
09:15:51
|
<surangak_> bryq, what have u got in mind ? :-)
|
09:15:59
|
<bryq> surangak_: I see that you merged trunk into your branch many times
|
09:16:13
|
<bryq> surangak_: So this means that you know how to do it
|
09:16:14
|
<surangak_> yepsssss
|
09:16:26
|
<surangak_> yaaa, thats one thing im good at :P
|
09:16:41
|
<bryq> surangak_: Now you must merge your branch into trunk
|
09:16:53
|
<bryq> surangak_: But you will not commit changes
|
09:17:15
|
<bryq> surangak_: But "Team > Create patch"
|
09:17:32
|
<bryq> surangak_: And this will be your GSoC diff
|
09:18:06
|
<surangak_> bryq, hmmm.... I see what u mean
|
09:18:17
|
<surangak_> bryq, actually that sounds like a cool idea :-)
|
09:18:29
|
<sunbiz> hi bwolfe
|
09:18:54
|
<bryq> surangak_: Yes, it sounds good to me too :)
|
09:19:41
|
<surangak_> bryq, actually i didnt get a chance to try out anything yet
|
09:19:51
|
<surangak_> bryq, bit tied up at office
|
09:20:08
|
<surangak_> bryq, have to wait a few hours to try this out :-)
|
09:20:21
|
<bryq> surangak_: After merging you should resolve all conflicts (I think you know that), run tests and check if everything works. If yes, create the patch and send it to Google and someone who will be able to commit it to the trunk
|
09:21:04
|
<bryq> surangak_: It's no problem for me :) I'm working now too, but have a short break
|
09:21:35
|
<surangak_> bryq, I think your solution should definitely work, I will try that out this evneing...
|
09:21:58
|
<surangak_> bryq, thanks a lot for the help, svn izing has always been my weak point :-)
|
09:22:19
|
<bryq> surangak_: You're welcome
|
09:28:27
|
*** pascal` has joined #openmrs
|
09:55:39
|
*** rafa has joined #openmrs
|
09:55:39
|
*** ChanServ sets mode: +v rafa
|
10:54:43
|
*** bwolfe has quit IRC
|
11:12:47
|
*** james_regen has joined #openmrs
|
11:12:47
|
*** ChanServ sets mode: +v james_regen
|
11:15:51
|
*** dkayiwa has left #openmrs
|
11:35:51
|
*** upul` has quit IRC
|
11:43:53
|
*** mwogi_ has quit IRC
|
11:51:08
|
*** bwolfe has joined #openmrs
|
11:51:08
|
*** ChanServ sets mode: +o bwolfe
|
11:57:59
|
<sunbiz> bwolfe: Hi...
|
11:58:09
|
<bwolfe> hey
|
11:58:19
|
<sunbiz> is there a reason why the OpenMRS formatter doesnt format xml??
|
11:58:50
|
<sunbiz> because its quiet haphazard at the moment... to look at the xmls
|
12:01:49
|
<bwolfe> the xml should be left along
|
12:01:51
|
<bwolfe> *alone
|
12:02:07
|
<bwolfe> if it formats the xml, then things like the openmrs-servlet.xml gets messed up and spring cna't read it
|
12:02:28
|
<sunbiz> ahh... why is that so??
|
12:02:41
|
<sunbiz> I can do a Ctrl+Shift+F and do format in my IDE
|
12:02:44
|
<sunbiz> and spring reads fine
|
12:04:07
|
<bwolfe> it didn't use to
|
12:08:02
|
<sunbiz> ohh... so when eclipse formats the xmls... spring fails to read it??
|
12:08:20
|
<sunbiz> eclipse--
|
12:08:37
|
<sunbiz> !karma eclipse
|
12:08:37
|
<OpenMRSBot> sunbiz: Karma for "eclipse" has been increased 2 times and decreased 2 times for a total karma of 0.
|
12:08:42
|
<sunbiz> netbeans++
|
12:08:46
|
<sunbiz> !karma netbeans
|
12:08:46
|
<OpenMRSBot> sunbiz: Karma for "netbeans" has been increased 4 times and decreased 0 times for a total karma of 4.
|
12:08:53
|
<sunbiz> wooohhhhh!!!
|
12:09:04
|
<bwolfe> netbeans--
|
12:09:08
|
<sunbiz> :(
|
12:09:49
|
<sunbiz> why bwolfe??
|
12:10:01
|
<bwolfe> because I don't like it
|
12:10:46
|
<sunbiz> bwolfe: u said u'll use the hot deploy... but u didnt
|
12:10:54
|
<sunbiz> its nice... and u'd say ++ as well
|
12:12:53
|
<bwolfe> haha
|
12:13:03
|
<bwolfe> I tried to use it. I hated the interface. and nothing was intuitive
|
12:13:14
|
<bwolfe> I wasted a day trying to get something basic done...
|
12:14:26
|
<sunbiz> u need to give it more than a day...
|
12:15:39
|
<sunbiz> its called the "inertia"... newton's first law
|
12:16:08
|
<bwolfe> the coefficient of static friction / activation energy was too high (depending on if you like physics or chemistry)
|
12:21:32
|
<sunbiz> bwolfe: but seriously?? eclipse has problems formatting xml or the OpenMRS-Formatter??
|
12:21:57
|
<bwolfe> sunbiz, dunno which one it is
|
12:36:42
|
<surangak_> bwolfe, hi.. mm small question.... even though the pencils down date went by, we're still supposed to update our branches to trunk until it is merged back, right ?
|
12:37:11
|
<bwolfe> surangak_, you can do all the work you want. it just doesn't count towards whether you pass or fail
|
12:37:13
|
<surangak_> bwolfe, otherwise itll be so outdated by the time someones tries to merge it, they'll go crazy :-)
|
12:37:20
|
<bwolfe> (which you already know anyway, since you passed) :-)
|
12:37:27
|
<surangak_> bwolfe, aaaaaaaaah, i get it. Will do
|
12:37:34
|
<surangak_> bwolfe, :D
|
12:45:09
|
*** pascal` has quit IRC
|
12:56:22
|
*** sunbiz1 has joined #openmrs
|
12:58:39
|
*** sunbiz has quit IRC
|
13:04:19
|
*** surangak_ has quit IRC
|
13:11:18
|
*** james_regen has quit IRC
|
13:17:35
|
*** surangak has quit IRC
|
13:24:26
|
*** Gretchie has joined #openmrs
|
13:37:18
|
<OpenMRSBot> Recent updates in the world of openmrs: New Changeset: OpenMRS (trunk): TRUNK-2602: Move the "active visit" on the patient dashboard next to the Last Encounter and highlight it a bit less ... <http://feedproxy.google.com/~r/OMRStrunk/~3/iyJ211XzJiw/OpenMRS>
|
13:39:52
|
*** upul` has joined #openmrs
|
13:39:53
|
*** ChanServ sets mode: +v upul`
|
13:44:52
|
*** aja has joined #openmrs
|
13:50:07
|
*** sunbiz has joined #openmrs
|
13:50:07
|
*** ChanServ sets mode: +v sunbiz
|
13:50:59
|
*** sunbiz1 has quit IRC
|
13:56:29
|
*** upul` has left #openmrs
|
13:59:37
|
*** gbastien has joined #openmrs
|
14:02:00
|
<bwolfe> rafa, I'll be back on in 10 mins for the scrum
|
14:02:09
|
<rafa> bryq: cool
|
14:02:17
|
<rafa> bwolfe: cool :)
|
14:04:43
|
*** djazayeri has joined #openmrs
|
14:04:49
|
*** ChanServ sets mode: +o djazayeri
|
14:04:53
|
<djazayeri> hi
|
14:05:45
|
<djazayeri> bwolfe, rafa, dkayiwa, wyclif
|
14:06:04
|
<rafa> hi djazayeri, Ben will be back in 10 mins
|
14:06:13
|
<djazayeri> ok
|
14:06:16
|
<djazayeri> we can wait for him then
|
14:06:20
|
<rafa> sure
|
14:07:15
|
<wyclif> hi
|
14:07:25
|
*** bwolfe has quit IRC
|
14:08:13
|
<rafa> djazayeri: meanwhile can you tell me what calls should interest devs (me) in the new schedule?
|
14:08:46
|
<rafa> djazayeri: there's a buch of them now and I'm not sure what to pick ;)
|
14:10:17
|
<djazayeri> You should definitely be at the Developer Community Forum. I believe the Implementer Forum will be interesting.
|
14:11:10
|
<djazayeri> I would try to sometimes join the Technical Design Forum once in a while, though that's obviously at an inconvenient time.
|
14:13:04
|
<rafa> djazayeri: yes, it's in the evening here
|
14:13:52
|
<rafa> djazayeri: the other two sound good
|
14:19:52
|
*** bwolfe has joined #openmrs
|
14:19:52
|
*** ChanServ sets mode: +o bwolfe
|
14:20:54
|
<djazayeri> hi bwolfe, you back?
|
14:21:44
|
<djazayeri> okay, my turn to step away for 5 mins
|
14:21:52
|
<bwolfe> heh, yes, am back
|
14:22:02
|
<bwolfe> did you guys start without me? or were you delaying>?
|
14:22:17
|
<rafa> we were waiting for you
|
14:22:56
|
<bwolfe> how sweet of you guys
|
14:23:08
|
<rafa> and guess what the waiting game continues ;)
|
14:23:23
|
<rafa> we're waiting for Darius
|
14:25:12
|
*** aja has quit IRC
|
14:25:39
|
<bwolfe> screw darius, lets go ahead without him. :-p
|
14:26:19
|
<rafa> haha how sweet of you :D
|
14:28:57
|
<djazayeri> okay
|
14:29:03
|
<djazayeri> let's go
|
14:29:45
|
<bwolfe> I'll go first
|
14:29:52
|
<bwolfe> (for a change)
|
14:29:55
|
<bwolfe> today:
|
14:29:56
|
<bwolfe> fix the maven setup in the sync module
|
14:29:56
|
<bwolfe> get ci working correctly
|
14:29:56
|
<bwolfe> fixed the 1.6.3 release in the mavenrepo
|
14:29:56
|
<bwolfe> sync overview chat with the ampath devs in prep for next week
|
14:29:56
|
<bwolfe> reviewed SMART-25 for wyclif
|
14:29:58
|
<bwolfe> triaged some tickets for saptarshi
|
14:30:00
|
<bwolfe> release process discussion with vijay
|
14:30:02
|
<bwolfe> tomorrow:
|
14:30:04
|
<bwolfe> actually do a sprint tickets?!
|
14:30:06
|
<bwolfe> (and because I just finished typing it up)
|
14:30:43
|
<rafa> bwolfe: did you find out why ci jobs got killed?
|
14:30:54
|
<bwolfe> no, wasn't looking into that
|
14:31:02
|
<bwolfe> mdowney is on that case
|
14:31:13
|
<bwolfe> I was just on the "why doesn't sync build when pointing at 1.6.2 case"
|
14:31:23
|
<rafa> bwolfe: ohh okay
|
14:31:24
|
<bwolfe> (and then I realized it only supported 1.6.3)
|
14:31:38
|
<bwolfe> so then I was on the "why doesn't sync build with 1.6.3 case"
|
14:31:38
|
<bwolfe> :-)
|
14:32:03
|
<djazayeri> any blockers?
|
14:33:03
|
<bwolfe> no, cleared it up about an hour and a half ago...just enough time to give the overview talk with the guys and then come back here
|
14:33:34
|
<djazayeri> okay
|
14:33:36
|
<djazayeri> I'll go next
|
14:33:47
|
<djazayeri> Monday
|
14:33:47
|
<djazayeri> * sprint-related things (code review, ticket design)
|
14:33:48
|
<djazayeri> * discuss sql indicators for reporting with Mike
|
14:33:48
|
<djazayeri> * discuss sync-related of a PIH pharmacy module with Mike and Justin
|
14:33:48
|
<djazayeri> * a few code reviews
|
14:33:48
|
<djazayeri> Today
|
14:33:49
|
<djazayeri> * discuss row-per-patient (& row-per-xyz) reporting with Mike
|
14:33:49
|
<djazayeri> * record screencast for OpenMRS University
|
14:33:50
|
<djazayeri> * get more tickets ready-for-work
|
14:33:50
|
<djazayeri> * fly back to Seattle
|
14:33:51
|
<djazayeri> * code reviews
|
14:33:51
|
<djazayeri> no blockers
|
14:35:05
|
<djazayeri> rafa?
|
14:35:17
|
<rafa> *** Rafal ***
|
14:35:18
|
<rafa> Today:
|
14:35:18
|
<rafa> * Minor fixes in yesterday's commits
|
14:35:18
|
<rafa> * Worked on TRUNK-2596: Add an Edit Person page
|
14:35:18
|
<rafa> https://tickets.openmrs.org/browse/TRUNK-2596
|
14:35:18
|
<rafa> Tomorrow:
|
14:35:18
|
<rafa> * Commit TRUNK-2596
|
14:35:19
|
<rafa> * Pick up a new ticket
|
14:35:19
|
<rafa> No blockers.
|
14:35:39
|
<djazayeri> you went with the Long-form page to copy?
|
14:35:56
|
<rafa> djazayeri: yes
|
14:36:10
|
<rafa> djazayeri: I saw your message
|
14:36:29
|
<rafa> djazayeri: I guess I forgot to respond?
|
14:36:38
|
<djazayeri> you probably responded, I was away from the computer a lot
|
14:36:48
|
<djazayeri> okay
|
14:36:52
|
<djazayeri> wyclif?
|
14:37:46
|
<bwolfe> why do you fly djazayeri?
|
14:38:35
|
<djazayeri> Why fly? As opposed to what?
|
14:38:46
|
<bwolfe> err, when
|
14:38:48
|
<djazayeri> Boston to Seattle is pretty far for any other form of transport. :-)
|
14:38:52
|
<djazayeri> 9pm flight
|
14:39:05
|
<djazayeri> I will really enjoy landing at midnight, and having to lead a 6am call.
|
14:39:12
|
<bwolfe> ahaha
|
14:39:19
|
<bwolfe> poor planning on that one :-(
|
14:39:31
|
<wyclif> yesterday:
|
14:39:31
|
<wyclif> - Added support for users to select a concept source where to search terms to map to when mapping terms
|
14:39:32
|
<wyclif> - renamed concep reference map class back to concept map(required alot of code changes)
|
14:39:32
|
<wyclif> - Fixed a couple of other small bugs
|
14:39:32
|
<wyclif> - cleaned up reviews/tickets from SMART sprint
|
14:39:32
|
<wyclif> - exchanged some emails about some features of the concept mappings
|
14:39:34
|
<wyclif> today:
|
14:39:36
|
<wyclif> - my TODOs depend on 2 decisions
|
14:39:38
|
<wyclif> - i.e if we decide to allow user create terms when mapping concepts
|
14:39:40
|
<wyclif> - if we want to make term to term mappings bidirectional
|
14:40:23
|
<djazayeri> did you do a mockup of create-terms-while-mapping, or any visual aid?
|
14:40:57
|
<wyclif> nope
|
14:41:52
|
<bwolfe> wyclif, yes ,they should be able to create terms that way. I think that was decided long ago. BUT they should not be able to edit them that way I think. (am I remembering that right?)
|
14:42:01
|
<wyclif> but i honestly don't think it worth allowing this\
|
14:42:35
|
<wyclif> according to burke, implemenations usually import dictionaries where these mapping really exist
|
14:43:09
|
<djazayeri> Well, I think that depends
|
14:43:12
|
<bwolfe> right, but its 100x more annoying to have to FIRST go to another page to make a term, then to map it
|
14:43:24
|
<djazayeri> maybe implementations will do that if we provide a tool for importing dictionaries
|
14:43:41
|
<wyclif> so spending extra time to add something that in most cases is already done for them might not be worth the effort
|
14:43:53
|
<wyclif> there a page for creating terms
|
14:44:06
|
<djazayeri> right now, though, PIH only maps to terminologies opportunistically (i.e. without importing the whole terminology)
|
14:44:09
|
<djazayeri> just as an example
|
14:44:13
|
<wyclif> it is like saying we want users to create concepts on the add obs form
|
14:44:57
|
<wyclif> well, then i will add it back
|
14:45:06
|
<bwolfe> no, not really. its more like the Manage Form / schema editing where we wanted Fields to be created at the same time as FormFields
|
14:45:08
|
<djazayeri> Can you please put a screenshot somewhere that we can look at?
|
14:45:37
|
<wyclif> for the concept form?
|
14:45:51
|
<djazayeri> yes, the place where this is happening
|
14:46:18
|
<djazayeri> The page we're talking about, I mean.
|
14:46:35
|
<wyclif> ok
|
14:48:21
|
<djazayeri> One option that springs to mind is how labeling emails works in gmail: you can apply an existing label without confirmation, but if you want to create a new label, you have to explicitly click on "add new label XYZ" button
|
14:49:10
|
*** lh_ has joined #openmrs
|
14:49:30
|
<wyclif> i have attached a screenshot on https://tickets.openmrs.org/browse/TRUNK-412
|
14:50:54
|
*** lh_ is now known as lh
|
14:51:25
|
<wyclif> djazayeri, bwolfe i have attached the file to https://tickets.openmrs.org/browse/TRUNK-412
|
14:52:14
|
<djazayeri> So, the scenario is that you have *already* chosen a source out of the dropdown, right?
|
14:52:56
|
<wyclif> correct!
|
14:53:22
|
<djazayeri> (As a side-note, I'd change "All Sources" to "(Search all sources)"
|
14:53:29
|
<wyclif> ok
|
14:53:44
|
<djazayeri> So, in that case I think it's okay to automatically add it
|
14:54:07
|
<djazayeri> You should also display in the flash message that you have done so.
|
14:54:37
|
<wyclif> i would have to break the code(name) cells and make them separate sothat the user enters both the name and code
|
14:54:43
|
<djazayeri> e.g. in addition to "Concept Saved" also say "Added term Xyz to Abc terminology"
|
14:54:58
|
<djazayeri> there's already a dropdown for the soure
|
14:55:00
|
<djazayeri> source
|
14:55:12
|
<wyclif> yes
|
14:55:23
|
<djazayeri> Oh, sorry, you're saying that the term table has both name and code columns?
|
14:55:39
|
<wyclif> i would have to change it to have both
|
14:55:45
|
<djazayeri> The database table
|
14:55:48
|
<wyclif> yes
|
14:55:57
|
<wyclif> and technically name and code are required
|
14:56:15
|
<wyclif> but in the old concept mappings name wasn't present\
|
14:56:20
|
<djazayeri> Oh, in that case I would lean towards not allowing this automatically on this page.
|
14:56:40
|
<wyclif> so the existing mappings during upgrades will have to have name the same as the code
|
14:56:44
|
<djazayeri> Personally I think it's annoying that name will now be required
|
14:56:55
|
<djazayeri> ah...
|
14:56:58
|
<wyclif> i think i removed the constraint
|
14:57:51
|
<wyclif> but the suggestion was to make the code the name too
|
14:59:00
|
<djazayeri> For example, if I don't know much about mapping, and someone tells me that the snomed code for weight in kg is abc123, I'd just want to type that in
|
14:59:09
|
<djazayeri> I would have no idea what the name of that snomed term is
|
14:59:36
|
<djazayeri> I'm not sure what benefit we really get from duplicating the code as the name instead of just leaving the name blank.
|
15:00:19
|
<djazayeri> So, my personal preference would be to leave name non-required, and if you type in a new code (after having chosen a source) on the edit concept page, that creates a new term with code, but name=null
|
15:00:37
|
<djazayeri> (I'm not sure if there's a reason that's bad, though.)
|
15:03:11
|
<djazayeri> bwolfe: how does that sound to you?
|
15:03:31
|
<djazayeri> wyclif: do you remember who suggested that we duplicate code as name?
|
15:04:07
|
<wyclif> ben and i think burke
|
15:04:43
|
<wyclif> i think i would be fine with leaving the name blank
|
15:06:11
|
<djazayeri> bwolfe: you gone?
|
15:06:33
|
<djazayeri> wyclif: okay, anyway, you know what I'm proposing. Hopefully Ben gets back soon and we can get him to agree or disagree.
|
15:06:41
|
<djazayeri> about the bidirectional mappings...
|
15:07:07
|
<djazayeri> can you remind me what some of the map types are, for use between terms?
|
15:07:26
|
<wyclif> i think Burke had a good suggestion
|
15:07:43
|
<wyclif> there are 2 forms of mappings
|
15:08:20
|
<wyclif> concept to terms(mappings on the concept form) and then terms to terms(mappings on the reference term form)
|
15:08:23
|
*** rafa has quit IRC
|
15:08:39
|
<djazayeri> Right.
|
15:08:55
|
<wyclif> so my question is mainly about the latter
|
15:08:58
|
<djazayeri> So, I don't think the the first one of those is bidirectional
|
15:09:03
|
<wyclif> correct!
|
15:09:22
|
<djazayeri> And given that, it means we're going to have to have a-is-to-b types for both narrower-than and wider-than (for example)
|
15:09:37
|
<djazayeri> so I think it's better to have them be unidirectional, and force people to create both, for now.
|
15:10:06
|
<wyclif> Burke's suggestion is to add an service methods that returns 'incoming' mappings to a term
|
15:10:54
|
<wyclif> i.e it would be saying 'give me all mappings where am am not a but b'
|
15:11:13
|
<djazayeri> only for the term-to-term mappings though?
|
15:11:29
|
<djazayeri> concepts are always A, right?
|
15:11:45
|
<wyclif> yes, and then list out these on the reference term form as links to the actual terms that have own these mappings
|
15:12:03
|
<wyclif> it is terms
|
15:12:17
|
<wyclif> so both A and b are terms
|
15:12:31
|
<djazayeri> I mean that for concept mappings, the concept is A and the term is B
|
15:12:39
|
<wyclif> correct!
|
15:12:40
|
<djazayeri> for term-to-term mappings, there are A and B terms
|
15:12:50
|
<wyclif> yes
|
15:13:20
|
<djazayeri> So, I agree with burke that we should leave the mappings unidirectional, and have API methods for getting terms that are mapped to a term as A or B.
|
15:14:14
|
<wyclif> just to correct you, the service method shoudl be specific, it gets terms where 'this' term is in the b term
|
15:14:45
|
<wyclif> since already the terms as a collection property containings mappings where it is the A term
|
15:15:03
|
<djazayeri> fine
|
15:15:07
|
<wyclif> sorry! since already the term has
|
15:15:24
|
<wyclif> it sounds fine to me
|
15:15:40
|
<wyclif> so if ben agrees with these, then i can get these done today
|
15:16:04
|
<wyclif> and then i can put the branch into review
|
15:16:54
|
<djazayeri> So, I don't think you need to wait for Ben for the bidirectional thing
|
15:17:04
|
<djazayeri> I'll email him and you my proposal
|
15:17:30
|
<wyclif> ok
|
15:17:42
|
<wyclif> any suggestions for the service methods
|
15:17:56
|
<wyclif> am struggling with getting a proper name for it
|
15:18:04
|
<djazayeri> getTermsMappingToTerm(Term term)?
|
15:18:10
|
<wyclif> ok
|
15:18:13
|
*** gauravpaliwal has joined #openmrs
|
15:18:13
|
*** ChanServ sets mode: +v gauravpaliwal
|
15:18:14
|
<djazayeri> getTermsThatMapTo(Term term)
|
15:18:27
|
<wyclif> can you please copy roger, burke and andy to the emails
|
15:19:56
|
<wyclif> this methods returns the maps and not the terms, right
|
15:20:02
|
<wyclif> methos
|
15:20:06
|
<wyclif> method
|
15:20:29
|
<djazayeri> Yes, because you'd want to know the type, etc
|
15:20:37
|
<djazayeri> so maybe getMappingsTo(Term)
|
15:21:05
|
<wyclif> ok
|
15:22:25
|
<wyclif> thanks
|
15:39:30
|
<djazayeri> email sent
|
15:51:49
|
*** mario__ has joined #openmrs
|
15:52:19
|
*** lh has quit IRC
|
15:52:37
|
<mario__> Hi Darius
|
15:53:05
|
<gauravpaliwal> mario__: say Hi djazayeri
|
15:54:21
|
<mario__> Hi djazayeri
|
16:02:06
|
*** gauravpaliwal has left #openmrs
|
16:15:53
|
*** rafa has joined #openmrs
|
16:15:53
|
*** ChanServ sets mode: +v rafa
|
16:19:13
|
*** lh has joined #openmrs
|
16:29:53
|
*** sunbiz has quit IRC
|
16:33:19
|
<mario__> Hola djazayeri
|
16:53:49
|
*** rafa has quit IRC
|
17:04:43
|
*** wyclif has quit IRC
|
17:17:07
|
*** mario__ has quit IRC
|
17:55:49
|
<OpenMRSBot> Recent updates in the world of openmrs: On Twitter: OpenMRS: Less than 2 weeks to register for the next #OpenMRS conference in #Rwanda at early bird rate! http://t.co/QjX4vLL #ICT4D #HealthIT <http://twitter.com/OpenMRS/statuses/108558774482108416> || OpenMRS Modules: Form Entry 4.5.9.3 uploaded to OpenMRS Module Repository <https://dev.openmrs.org/modules/view.jsp?module=formentry&version=&4.5.9.3>
|
17:58:02
|
*** wyclif has joined #openmrs
|
18:01:27
|
*** wyclif has joined #openmrs
|
18:14:59
|
*** Suranga has joined #openmrs
|
18:28:41
|
*** sunbiz has joined #openmrs
|
18:28:41
|
*** ChanServ sets mode: +v sunbiz
|
18:43:42
|
*** bryq1 has joined #openmrs
|
18:45:40
|
*** bryq has quit IRC
|
18:57:02
|
*** jriley has joined #openmrs
|
19:01:54
|
<OpenMRSBot> Recent updates in the world of openmrs: On Twitter: OpenMRS: RT @askanter: I am presenting Millennium Villages Global Network and #OpenMRS at VLDB conference in Seattle today at 4PM: http://t.co/jS ... <http://twitter.com/OpenMRS/statuses/108600148548001792>
|
19:09:23
|
*** sunbiz has quit IRC
|
19:10:11
|
*** Gretchie has quit IRC
|
19:14:10
|
*** rafa has joined #openmrs
|
19:14:10
|
*** ChanServ sets mode: +v rafa
|
19:20:17
|
*** bryq1 has quit IRC
|
19:22:28
|
<Suranga> djazayeri, helloo
|
19:25:24
|
*** gbastien has quit IRC
|
19:36:34
|
<wyclif> hi djazayeri
|
19:37:00
|
<wyclif> djazayeri, did you get my response to your email?
|
19:46:31
|
*** gbastien has joined #openmrs
|
19:46:41
|
*** Suranga has quit IRC
|
20:08:16
|
*** Gretchie has joined #openmrs
|
20:10:41
|
*** rafa has quit IRC
|
20:37:59
|
<wyclif> hi djazayeri
|
20:38:06
|
<wyclif> are you getting my emails
|
20:38:20
|
<djazayeri> not really
|
20:38:23
|
<wyclif> in response to the mappings discussion
|
20:38:23
|
<djazayeri> doesn't seem like it
|
20:38:26
|
<djazayeri> in a meeting now
|
20:38:27
|
<wyclif> i wonder why
|
20:38:36
|
<djazayeri> got one from you at some point, but not recently
|
20:40:18
|
<wyclif> djazayeri, did you get the one where is suggested we use a global property
|
20:41:48
|
<wyclif> djazayeri, ?
|
20:43:29
|
<wyclif> djazayeri, ?
|
20:44:51
|
*** jprice has joined #openmrs
|
20:45:46
|
<jprice> does anyone know if openMRS sends an HL7 ack back when it receives and HL7 msg ?
|
20:47:49
|
<wyclif> jprice, no
|
20:48:06
|
<djazayeri> wyclif: yes, i got that. i agree with it
|
20:48:10
|
<djazayeri> did you get my reply email?
|
20:48:12
|
<wyclif> ok
|
20:48:20
|
<djazayeri> from my phone, where I said I agree with using a gp
|
20:48:28
|
<wyclif> djazayeri, am getting delivery failure emails
|
20:49:26
|
<jprice> thanks
|
20:51:57
|
<wyclif> djazayeri, i assumed that all concept mappings are technically pointing to codes in a given source and the source is the one that knows which concept is represented by that code, meaning a map can not exist without a code
|
20:52:05
|
<wyclif> wonder what andy meant by no code
|
20:52:41
|
<wyclif> since he said a code should not be required\
|
21:10:12
|
*** djazayeri has quit IRC
|
21:19:03
|
*** Gretchie has quit IRC
|
21:56:05
|
*** jriley has left #openmrs
|
22:43:33
|
*** bwolfe has quit IRC
|
22:44:09
|
*** gbastien has quit IRC
|
22:45:04
|
*** wyclif has quit IRC
|
22:54:13
|
<OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Announcements List: Reminder: New OpenMRS Weekly Call Schedule Starts this Week <https://listserv.iupui.edu/cgi-bin/wa-iupui.exe?A2=ind1108&L=openmrs-announce-l&P=2257>
|
22:54:19
|
*** gbastien has joined #openmrs
|
23:03:19
|
*** gbastien has quit IRC
|
23:05:20
|
*** gbastien has joined #openmrs
|
23:24:26
|
*** lh has quit IRC
|
23:34:49
|
*** wyclif has joined #openmrs
|
23:54:28
|
*** Gretchie has joined #openmrs
|