00:24:12
|
*** mccallumg has quit IRC
|
00:30:56
|
*** lh has joined #openmrs
|
00:30:56
|
*** lh has joined #openmrs
|
00:55:17
|
*** bwolfe has joined #openmrs
|
00:55:20
|
*** ChanServ sets mode: +o bwolfe
|
00:56:29
|
<GDG> hi bwolfe. you asked "are you more on the coding of hl7 side or coding of java side?"
|
00:57:33
|
<GDG> well, I chair several HL7 committees (know several people at regenstrief), and am busy writing standards. But I also lead the eclipse open healthcare framwork project, for instance, and wrote reams of HL7 related code there.
|
00:57:38
|
<bwolfe> hi GDG
|
00:57:56
|
<bwolfe> ah, ok
|
00:57:59
|
<bwolfe> so a little of both?
|
00:58:06
|
<djazayeri> or a lot of both...
|
00:58:16
|
<bwolfe> true(r)
|
00:58:29
|
<bwolfe> GDG, are you on the developer's mailing list? are you following the thread that suranga started about hl7 output?
|
00:58:50
|
<GDG> no, I'm not
|
00:58:54
|
<GDG> guess I'll have to be
|
00:59:59
|
<bwolfe> GDG, you can follow in the google group if you want
|
01:00:21
|
<bwolfe> https://groups.google.com/a/openmrs.org/forum/#!forum/dev
|
01:00:24
|
<OpenMRSBot> <http://ln-s.net/$Kdi> (at groups.google.com)
|
01:00:51
|
<GDG> k ta
|
01:04:07
|
*** elliott_w has quit IRC
|
01:24:42
|
*** suranga has joined #openmrs
|
01:24:42
|
*** ChanServ sets mode: +v suranga
|
01:38:37
|
*** lh has quit IRC
|
01:47:42
|
*** lh has joined #openmrs
|
01:47:42
|
*** lh has joined #openmrs
|
01:50:56
|
*** elliott_w has joined #openmrs
|
01:50:56
|
*** ChanServ sets mode: +o elliott_w
|
02:11:17
|
*** r_friedman_away has quit IRC
|
02:13:09
|
<suranga> djazayeri, thanks for keeping the mail thread running
|
02:13:28
|
<djazayeri> suranga: I'm asking the questions because I'm curious. :-)
|
02:13:29
|
<suranga> djazayeri, im rather stuck with some jembi work right now, but will get back to writing a good response soon :)
|
02:13:43
|
<suranga> djazayeri, either way, it serves my cause :P
|
02:23:34
|
*** elliott_w has quit IRC
|
02:28:14
|
<GDG> how do I get openMRS to generate this restws sample? can you point in the general direction?
|
02:29:00
|
<bwolfe> GDG, https://wiki.openmrs.org/display/docs/Webservices.rest+Module
|
02:29:01
|
<OpenMRSBot> <http://ln-s.net/92SZ> (at wiki.openmrs.org)
|
02:29:47
|
<bwolfe> GDG, install that module. make http rest calls according to this page
|
02:29:50
|
<bwolfe> https://wiki.openmrs.org/display/docs/REST+Web+Services+API
|
02:29:51
|
<OpenMRSBot> <http://ln-s.net/8zTB> (at wiki.openmrs.org)
|
02:30:19
|
<bwolfe> although thats more of the "pull data" use case
|
02:31:08
|
<bwolfe> djazayeri, is there an easy API call to /send/ from restws? if, say, an hl7output module wanted to trigger something to be sent to mirth whenever a new patient was created?
|
02:35:22
|
*** elliott_w has joined #openmrs
|
02:35:22
|
*** ChanServ sets mode: +o elliott_w
|
02:36:17
|
*** lh has quit IRC
|
02:47:52
|
<GDG> thanks. I'll look
|
02:51:04
|
*** elliott_w has quit IRC
|
02:55:52
|
*** elliott_w has joined #openmrs
|
02:55:52
|
*** ChanServ sets mode: +o elliott_w
|
03:06:39
|
*** elliott_w has quit IRC
|
03:43:15
|
*** robbyoconnor has joined #openmrs
|
03:43:15
|
*** ChanServ sets mode: +v robbyoconnor
|
03:53:44
|
<djazayeri> bwolfe: we want to *send* a resource?
|
03:54:32
|
<djazayeri> I guess it's straightforwardâbasically get back the SimpleObject version of the resource, and then manually use jackson to convert it to json (which is what spring is doing anyway)
|
03:54:46
|
<bwolfe> djazayeri, yes, if we wanted to get data to a mirth box for it send somewhere
|
03:55:13
|
<djazayeri> surely one could write a module that requires webservices.rest and eventbus, and does this
|
03:55:21
|
<djazayeri> it should require very little code
|
03:56:54
|
<bwolfe> yes, but not already written in restws per se
|
03:56:56
|
<bwolfe> and its just "event" module (at your suggestion). :-)
|
03:57:14
|
<djazayeri> I realized that after I sent it. :-)
|
03:57:56
|
<djazayeri> anyway, no it's not written yet, but it seems like it'd be less than 100 lines of code
|
03:58:07
|
<bwolfe> GDG, if you want to look into using restws to send json to mirth, there is a pathway
|
04:21:09
|
*** bwolfe has quit IRC
|
04:33:58
|
*** deepaganu has joined #openmrs
|
04:43:43
|
*** lakkarau has joined #openmrs
|
05:00:30
|
*** lakkarau has quit IRC
|
05:06:14
|
*** lakkarau has joined #openmrs
|
05:32:25
|
*** applecool has joined #openmrs
|
05:32:25
|
*** ChanServ sets mode: +v applecool
|
05:55:44
|
*** applecool has quit IRC
|
06:41:32
|
*** dkayiwa has joined #openmrs
|
07:22:23
|
*** goutham has joined #openmrs
|
07:22:33
|
<goutham> dkayiwa: Good morining :)
|
07:22:43
|
<dkayiwa> goutham: good morning :)
|
07:23:31
|
<goutham> dkayiwa: i have committed code against worked on goals. You can test the functioanlity :)
|
07:23:43
|
<dkayiwa> goutham: ok thanks :)
|
07:23:59
|
<goutham> dkayiwa: Had breakfast?
|
07:24:14
|
<dkayiwa> goutham: about to :)
|
07:24:17
|
<dkayiwa> and you?
|
07:25:19
|
<goutham> dkayiwa: Ya had ....its " Upma " our Andhra Pradesh state special break fast :)
|
07:25:30
|
<dkayiwa> goutham: :D
|
07:25:53
|
<goutham> dkayiwa: :)
|
07:26:48
|
<goutham> dkayiwa: whats your place special break fast ?
|
07:30:45
|
<dkayiwa> goutham: mago juice, rice, beans, and eggs :)
|
07:34:08
|
<goutham> dkayiwa: so whats going to be on ur plate today :)
|
07:34:38
|
<dkayiwa> goutham: i have no idea yet :)
|
07:36:47
|
<goutham> dkayiwa: : )
|
07:37:19
|
<goutham> dkayiwa: its lunch time for me..Will have and come :)
|
07:40:07
|
*** applecool has joined #openmrs
|
07:40:07
|
*** ChanServ sets mode: +v applecool
|
07:41:02
|
<dkayiwa> goutham: ok :)
|
07:45:02
|
*** deepaganu has quit IRC
|
08:00:53
|
*** applecool has quit IRC
|
08:08:30
|
*** suranga has quit IRC
|
08:09:03
|
*** dkayiwa has quit IRC
|
08:13:02
|
*** harshadura has joined #openmrs
|
08:16:57
|
*** rafa_ has joined #openmrs
|
08:16:57
|
*** ChanServ sets mode: +v rafa_
|
08:20:54
|
*** lakkarau has quit IRC
|
08:25:25
|
*** lakkarau has joined #openmrs
|
08:42:22
|
*** lakkarau has quit IRC
|
08:48:36
|
*** lakkarau has joined #openmrs
|
08:57:30
|
*** lakkarau has quit IRC
|
09:01:30
|
*** lakkarau has joined #openmrs
|
09:07:30
|
*** lakkarau has quit IRC
|
09:07:57
|
*** lakkarau has joined #openmrs
|
09:12:30
|
*** lakkarau has quit IRC
|
09:13:07
|
*** lakkarau has joined #openmrs
|
09:17:30
|
*** lakkarau has quit IRC
|
09:17:47
|
*** lakkarau has joined #openmrs
|
09:41:02
|
*** upul` has joined #openmrs
|
09:41:02
|
*** ChanServ sets mode: +v upul`
|
10:03:54
|
*** lakkarau has quit IRC
|
10:06:52
|
*** lakkarau has joined #openmrs
|
10:10:43
|
*** snoppy has joined #openmrs
|
10:41:12
|
*** applecool has joined #openmrs
|
10:41:12
|
*** ChanServ sets mode: +v applecool
|
10:41:21
|
<applecool> hi rafa_
|
10:41:31
|
<rafa_> applecool: hi
|
10:41:36
|
*** rafa_ is now known as rafa
|
10:41:59
|
<rafa> applecool: how is the recovery? :)
|
10:42:06
|
<applecool> its done!
|
10:42:11
|
<applecool> it was a great pain
|
10:42:30
|
<applecool> i configured the eclipse and mysql maven and all the other stuff
|
10:42:47
|
<applecool> rafa: my apologies
|
10:42:56
|
*** upul` has quit IRC
|
10:43:01
|
<rafa> applecool: no need to apologize
|
10:43:03
|
<applecool> i have a lot of pending work to do now! rafa
|
10:43:08
|
<rafa> applecool: it happens
|
10:43:40
|
<applecool> rafa: we discussed about it just day before and it happend! damn! :|
|
10:43:56
|
<applecool> rafa: now everything is working like as before :)
|
10:44:03
|
<rafa> applecool: good
|
10:44:28
|
<rafa> applecool: so remind me, are the tasks from the last week completed?
|
10:44:34
|
<applecool> i need to first put an end to the last weeks last tasks
|
10:44:42
|
<rafa> applecool: ohh okay
|
10:44:46
|
<applecool> no rafa not completed
|
10:44:58
|
<applecool> rafa: i have some doubts! can i go on?
|
10:45:03
|
<rafa> applecool: yes
|
10:45:51
|
<applecool> first one is we have the getter and the setter for the User in the MetadataPAckage and i got the foriegn key and everything is good! and i got the instance of the User too
|
10:46:39
|
<applecool> i need to set the Username on the package right? to display it on the mainPage while package listing
|
10:47:17
|
<rafa> applecool: you can call package.getUser().getUsername() right?
|
10:47:24
|
<rafa> applecool: why do you want to set it?
|
10:47:54
|
<applecool> but i am unable to call it rafa i tried it before! no fruitful results
|
10:48:08
|
<rafa> applecool: what do you get?
|
10:48:39
|
<applecool> just a sec! i will do it and tell you! i dont remember to be frank !
|
10:48:50
|
<applecool> i guess i was getting the null pointer exception
|
10:50:41
|
<applecool> rafa: i actually didnt want to miss the scrum today as i missed it yesterday!
|
10:50:59
|
<applecool> i am commuting now! i am in bus :) using my netcard :P
|
10:51:15
|
<rafa> applecool: ohh
|
10:51:28
|
<rafa> applecool: we can talk later...
|
10:51:52
|
<rafa> applecool: I'll be online for the next 4 hours
|
10:51:59
|
<applecool> ah no problem! my home is still 3 miles away :)
|
10:52:07
|
<rafa> applecool: ok
|
10:52:25
|
<applecool> but i wil let you know when i will be getting down :)
|
10:55:10
|
*** venkaban has quit IRC
|
10:55:50
|
<applecool> rafa: i am getting the same error which led to deletion of file!
|
10:56:00
|
<applecool> which folder should be deletd exactly?
|
10:56:29
|
<rafa> applecool: I don't remember
|
10:56:38
|
<applecool> http://pastebin.com/ScBzei82
|
10:56:39
|
<rafa> applecool: it was in the error message
|
10:56:44
|
<applecool> yes!
|
10:57:06
|
*** venksbsn has quit IRC
|
10:57:24
|
<applecool> just it worked fine! i am not understanding y that error mesage is coming!
|
10:57:31
|
<applecool> just now*
|
10:58:13
|
<rafa> applecool: have you simply tried again and it worked?
|
10:58:23
|
<applecool> no
|
10:59:26
|
<applecool> in the morning! it worked fine and now too ! but after i changed the file and ran it! it started giving me this error
|
11:00:11
|
<applecool> rafa: i will get you back in 20 minutes! i need to get out of the bus! :) will be back asap
|
11:00:15
|
<applecool> bye cya rafa
|
11:00:21
|
<rafa> applecool: ok cya
|
11:04:53
|
*** applecool has quit IRC
|
11:05:23
|
*** magoo_ has joined #openmrs
|
11:07:27
|
*** james_regen has joined #openmrs
|
11:07:27
|
*** ChanServ sets mode: +v james_regen
|
11:22:36
|
*** bwolfe has joined #openmrs
|
11:22:36
|
*** ChanServ sets mode: +o bwolfe
|
11:32:31
|
*** magoo_ has quit IRC
|
11:38:36
|
*** applecool has joined #openmrs
|
11:38:36
|
*** ChanServ sets mode: +v applecool
|
11:38:41
|
<applecool> hi rafa
|
11:38:48
|
<rafa> applecool: hi
|
11:38:58
|
<applecool> just came back home!
|
11:38:59
|
<rafa> applecool: any success?
|
11:39:04
|
<applecool> ah trying
|
11:39:08
|
<applecool> just a sec
|
11:40:58
|
<applecool> same error rafa
|
11:41:14
|
<rafa> applecool: have you tried googling it?
|
11:41:16
|
*** goutham has quit IRC
|
11:41:39
|
<applecool> i tried it but no luck
|
11:42:04
|
<rafa> applecool: do you run mvn clean install?
|
11:42:09
|
<applecool> yes
|
11:43:35
|
<rafa> applecool: code committed?
|
11:43:48
|
<applecool> you mean the previous?
|
11:43:54
|
<rafa> applecool: the current
|
11:43:58
|
<rafa> applecool: what you try to run
|
11:44:06
|
<rafa> applecool: I'll try it myself
|
11:44:08
|
<applecool> yes! till the search part everything is committed!
|
11:44:24
|
<applecool> and with your extra comments
|
11:44:46
|
<applecool> okay i will just comit the code again
|
11:48:35
|
<applecool> just now committed the code
|
11:48:37
|
<applecool> again
|
11:48:49
|
<applecool> i dont think there is nothin much in the commit
|
11:49:01
|
<applecool> i mean just the formatting and some small chages
|
11:49:03
|
<applecool> changes
|
11:52:02
|
*** GrahameG has joined #openmrs
|
11:52:40
|
<applecool> i think that .DStore is making some mess i guesS!
|
11:53:01
|
<applecool> do you think that makes any sense?
|
11:53:03
|
<rafa> applecool: what is dstore?
|
11:53:26
|
<applecool> its just a small binary file and it is actually a hidden file
|
11:53:29
|
*** GDG has quit IRC
|
11:53:37
|
<applecool> it generates for every folder and everywhere
|
11:53:48
|
<applecool> i mean it wil be everywhere
|
11:54:16
|
<applecool> actually .m2 and var folders are hidden in mac
|
11:54:37
|
<rafa> applecool: is it something you recently installed?
|
11:54:47
|
<rafa> applecool: why it stopped working right now?
|
11:55:01
|
<applecool> i dont know rafa i dint install anything new
|
11:55:19
|
<applecool> i will remove them and again i will import them and see
|
11:55:36
|
<applecool> is it working for you
|
11:55:39
|
<applecool> i guess it will be
|
11:56:11
|
<rafa> applecool: just pulled
|
11:56:14
|
<rafa> applecool: running right now
|
11:56:19
|
<applecool> day before even darius told me that such type of error was not seen before
|
11:56:46
|
<applecool> he meant our app is entirely seperate one
|
11:57:20
|
<rafa> applecool: testSave fails for me
|
11:57:44
|
<rafa> applecool: ohh I know wahy
|
11:57:51
|
<rafa> applecool: I need to set the upload path
|
11:58:05
|
<applecool> yes! just now i changed that line! just make it as it is! i mean pkg.setUser(uname); for now
|
11:58:09
|
<applecool> to make it pass
|
12:01:57
|
*** wyclif has quit IRC
|
12:01:58
|
<rafa> applecool: it works the container started
|
12:02:11
|
<rafa> applecool: can you run without skipping tests?
|
12:02:21
|
<applecool> no!
|
12:02:26
|
<applecool> i m importing them again
|
12:02:39
|
<rafa> applecool: what are you importing again?
|
12:02:55
|
<applecool> i mean i deleted the project from eclipse
|
12:03:06
|
<applecool> and i am again importing them and making it a fresh build
|
12:03:13
|
<rafa> applecool: ok
|
12:06:27
|
*** magoo_ has joined #openmrs
|
12:07:51
|
*** mvorobey has joined #openmrs
|
12:07:55
|
<mvorobey> hi all
|
12:09:46
|
<mvorobey> hi bwolfe
|
12:09:59
|
<bwolfe> hey
|
12:10:31
|
<mvorobey> bwolfe, ready to answer to couple of my questions ? :)
|
12:11:04
|
<bwolfe> no, let me prepare
|
12:11:05
|
<mvorobey> bwolfe, am testing against 1.9 and now there is only one error left on the way to get it worked
|
12:11:12
|
<mvorobey> bwolfe, :)
|
12:11:17
|
<bwolfe> ok, now I'm ready
|
12:11:27
|
<mvorobey> bwolfe, so, this is an error
|
12:11:33
|
<mvorobey> Caused by: org.h2.jdbc.JdbcSQLException: Column "E.PROVIDER_ID" not found; SQL statement:
|
12:11:33
|
<mvorobey> select e.patient_id from encounter e inner join patient p on e.patient_id = p.patient_id where e.voided = false and e.provider_id in (?) and p.voided = false group by e.patient_id [42122-135]
|
12:12:10
|
<mvorobey> bwolfe, it occurs when org.openmrs.module.reporting.cohort.definition.evaluator.EncounterCohortDefinitionEvaluatorTest.evaluate_shouldReturnCorrectPatientsWhenProviderParametersAreSet() is running
|
12:12:46
|
<mvorobey> bwolfe, and I'm not sure how to get rid of such error basically
|
12:12:48
|
<bwolfe> encounter.provider_id is new in 1.9. are you sure you're running against 1.9 with the right hbm files?
|
12:13:03
|
<bwolfe> mvorobey, the unit tests look for all hbm files and create the in memory h2 db from them
|
12:13:15
|
<bwolfe> so if its finding the wrong hbm files, it would be creating a wrong encounter table
|
12:13:49
|
<mvorobey> bwolfe, huh, am so un-attentive, sorry, for trouble
|
12:14:15
|
<bwolfe> find the issue already?
|
12:14:38
|
<mvorobey> bwolfe, in-progress
|
12:17:06
|
<mvorobey> bwolfe, do not know why it uses older hbm.xmls, may ne it actually do not ...
|
12:17:21
|
<applecool> rafa: not working :( :'(
|
12:17:28
|
<mvorobey> actually do not = actually does not
|
12:17:35
|
<rafa> applecool: delete .m2 directory
|
12:17:45
|
<rafa> applecool: run mvn clean install without skipTests
|
12:17:56
|
<applecool> .m2 !
|
12:18:06
|
<applecool> okay
|
12:18:13
|
<rafa> applecool: maybe the zip got corrupted
|
12:18:35
|
<rafa> applecool: and it's still in the .m2 directory
|
12:18:36
|
<applecool> oh it always gets downloaded right with the clean build
|
12:18:50
|
<rafa> applecool: no, it's only downloaded once
|
12:18:57
|
<applecool> ok
|
12:19:00
|
<rafa> applecool: and it's stored in .m2 dict I think
|
12:20:30
|
<bwolfe> mvorobey, are you doing a clean each time?
|
12:20:40
|
<mvorobey> bwolfe, nope
|
12:20:41
|
<bwolfe> perhaps the hbms are getting copied into some build folder
|
12:21:43
|
<applecool> rafa: the entire .m2 folder?
|
12:21:49
|
<applecool> delete?
|
12:22:15
|
<rafa> applecool: hmmm do you have any download limits?
|
12:22:24
|
<applecool> meaning?
|
12:22:29
|
<applecool> the broadband stuff?
|
12:22:46
|
<rafa> applecool: do you have to pay for MBs?
|
12:22:55
|
<applecool> aah! na
|
12:22:57
|
<applecool> not much
|
12:23:05
|
<applecool> but the speed is not too high
|
12:23:21
|
<rafa> applecool: ok let me try to find which file to delete
|
12:23:21
|
*** magoo_ has quit IRC
|
12:24:00
|
*** kishoreyekkanti has joined #openmrs
|
12:24:41
|
<kishoreyekkanti> bwolfe: Hey ben you there?
|
12:25:10
|
<bwolfe> if you are offering gifts of money, then yes
|
12:25:17
|
<rafa> applecool: try deleting \.m2\repository\org\codehaus\
|
12:25:34
|
<applecool> you mean codehaus folder
|
12:25:37
|
<applecool> right?
|
12:25:37
|
<rafa> yes
|
12:28:10
|
<bwolfe> mvorobey, and then there was one!
|
12:28:41
|
<bwolfe> kishoreyekkanti, you can send monetary gifts to my Grand Caymans offshore account #13247348u34981324
|
12:29:32
|
<kishoreyekkanti> bwolfe: ha haa. Not sure where you are! Are you in Kenya?
|
12:29:40
|
<mvorobey> bwolfe, sorry, did not get
|
12:30:02
|
<bwolfe> back in the US for a long while now
|
12:30:36
|
<applecool> i deleted it!
|
12:30:43
|
<applecool> next
|
12:30:44
|
<bwolfe> mvorobey, Rita signed up on github. now we only have Lu remaining.
|
12:30:50
|
<applecool> building it right? rafa
|
12:31:00
|
<mvorobey> bwolfe, horray, party !!!
|
12:31:42
|
<rafa> applecool: right
|
12:31:44
|
<kishoreyekkanti> bwolfe: oh ok!
|
12:31:54
|
<applecool> ok rafa in process
|
12:32:18
|
<bwolfe> mvorobey, what would happen if you put his username in the authors file with his email, but then his email didn't have a github account on the other side?
|
12:32:27
|
<bwolfe> would github push fail?
|
12:32:43
|
<bwolfe> or would his account just be matched if/when he ever signed up on github?
|
12:32:49
|
<kishoreyekkanti> bwolfe: Fresh installation of OpenMRS was throwing some database update exception. So wanted to know if it is a known problem or if i
|
12:33:04
|
<kishoreyekkanti> bwolfe: I'm missing any thing. Any ways i've sent the mail to dev list
|
12:33:11
|
<bwolfe> I saw your mail
|
12:33:15
|
<bwolfe> trunk?
|
12:33:21
|
<kishoreyekkanti> bwolfe: yep
|
12:33:27
|
<mvorobey> bwolfe, dunno, may be something very sad, but it's not tested yet
|
12:34:10
|
<bwolfe> kishoreyekkanti, rafa just upgraded liquibase yesterday, perhaps this is an issue?
|
12:34:20
|
<bwolfe> in liq 2.0.5 that line is if (changeSet.getFilePath().equalsIgnoreCase(path)
|
12:34:27
|
<bwolfe> so our changesets might not all have a filepath?
|
12:36:04
|
<kishoreyekkanti> bwolfe: you mean we've bumped the liquibase version to 2.0.5 and the way we access the scripts is changed in the latest versions?
|
12:37:00
|
<kishoreyekkanti> bwolfe: Any ways i'm running for a meeting now. Will catch you back in some time
|
12:37:34
|
<bwolfe> kishoreyekkanti, a quick fix is to revert that latest change in the core pom.xml for the liquibase
|
12:37:54
|
<bwolfe> the better fix is for rafa to confirm and fix the NPE. :-)
|
12:38:02
|
<applecool> http://pastebin.com/FFG05X60 rafa
|
12:38:58
|
<rafa> bwolfe: yeah, I'll look into that, I'm about to test installation on postgresql
|
12:39:07
|
*** kishoreyekkanti_ has joined #openmrs
|
12:39:58
|
*** kishoreyekkanti has quit IRC
|
12:40:16
|
*** kishoreyekkanti has joined #openmrs
|
12:41:07
|
*** magoo_ has joined #openmrs
|
12:41:12
|
<applecool> rafa: i guess i should have deleted the apache-tomcat 6.0.29 in the installs folder too i guess
|
12:46:31
|
<rafa> applecool: I temoprarily disabled integration tests
|
12:46:41
|
<rafa> applecool: pull from github
|
12:46:52
|
<rafa> applecool: continue to work on your tasks
|
12:47:03
|
<applecool> oh! just a second rafa trying it once again!
|
12:47:07
|
<applecool> and then i will pull it
|
12:47:13
|
<applecool> okay i will start working
|
12:47:26
|
<applecool> what do you mean by the integration tests rafa?
|
12:47:44
|
<rafa> applecool: it's the integration tests that fail
|
12:47:55
|
<rafa> applecool: tomcat is started for the integration tests
|
12:48:09
|
<applecool> oh! okay :)
|
12:48:37
|
*** wyclif has joined #openmrs
|
12:49:16
|
<magoo_> hi +rafa
|
12:49:26
|
<rafa> hi magoo_
|
12:50:55
|
<magoo_> +rafa i have some few questions to ask let me hope you not busy
|
12:51:14
|
<mvorobey> bwolfe, question
|
12:51:45
|
<mvorobey> bwolfe, I committed code yesterday, when it will be in jars within maven repo ?
|
12:52:12
|
*** harshadura has quit IRC
|
12:52:46
|
<mvorobey> bwolfe, and which versions of jar do I need to use, accordingly to commit, ain't it?
|
12:53:20
|
<magoo_> +rafa ever since i downgraded from jdk 1.7.x to jdk 1.6.x my apache tomcat keeps of failing it does not start
|
12:53:53
|
<bwolfe> mvorobey, let me rephrase for you: "you committed something to trunk yesterday. when will those changes be in the nightly builds of hte openmrs jars on nexus maven repo?"
|
12:54:03
|
<bwolfe> the answer is: after 1am EST
|
12:54:28
|
<bwolfe> so they should be there for trunk already. I don't think the other branches are building automatically yet. (I opened an ITSM ticket for it though)
|
12:56:15
|
<mvorobey> bwolfe, thanks, you are reading my mind
|
12:56:42
|
<mvorobey> of my mind
|
12:56:43
|
<mvorobey> :)
|
12:57:12
|
<bwolfe> mvorobey, I have added lu's name/email to the github authors list on the wiki. I vote you go ahead and push to github when you get a chance. if lu ever creates an account on github, it should merge/match, but I am not sure. I couldn't find anything abou tit
|
12:57:32
|
<bwolfe> mvorobey, but since he hasn't resonded in a week, I assume he either doesn't care or is out of touch for some reason
|
12:58:57
|
<applecool> rafa: http://pastebin.com/zznjGv9V
|
12:59:02
|
<applecool> build success :)
|
12:59:29
|
<applecool> i removed that apache zip too in the Tmp folder
|
12:59:43
|
<applecool> and then deleted the codehaus folder and builded it!
|
12:59:48
|
<applecool> it worked
|
12:59:50
|
<mvorobey> bwolfe, do I need to do conversion to GitHub ASAP or it can wait until I'll done with multiple version testing ?
|
13:01:19
|
*** dkayiwa has joined #openmrs
|
13:02:24
|
<bwolfe> mvorobey, it can wait
|
13:02:38
|
<applecool> rafa: will be back in an hour :)
|
13:02:39
|
<bwolfe> where are you commiting your multiple version stuff?
|
13:03:02
|
<bwolfe> it might be better to push to github then push/pullrequest your changes on github
|
13:03:20
|
*** applecool has quit IRC
|
13:03:44
|
<mvorobey> bwolfe, am in svn yet
|
13:04:10
|
<bwolfe> mvorobey, so you are commiting to a mavenized reporting in svn.openmrs.org?
|
13:05:44
|
<mvorobey> bwolfe, actually, am not committing yet, working locally, on my laptop
|
13:06:09
|
<bwolfe> ok, I'd hate for you to lose that, so make sure its committed somewhere befor eyou start messing with converting
|
13:07:34
|
<mvorobey> bwolfe, I committed it to my flash card, it is safe :)
|
13:08:13
|
<mvorobey> bwolfe, and do commit then, before messy converting
|
13:08:45
|
*** dkayiwa_ has joined #openmrs
|
13:09:47
|
*** dkayiwa has quit IRC
|
13:09:48
|
*** dkayiwa_ is now known as dkayiwa
|
13:10:40
|
<wyclif> hi djazayeri
|
13:16:11
|
<magoo_> +rafa???
|
13:18:11
|
*** downey has joined #openmrs
|
13:18:11
|
*** ChanServ sets mode: +o downey
|
13:18:11
|
*** OpenMRSBot sets mode: +o downey
|
13:22:13
|
<dkayiwa> hi rafa
|
13:22:46
|
<rafa> hi dkayiwa
|
13:23:25
|
<dkayiwa> rafa: do you have any suggestions for the strange behavior am getting on: SXS-1 ?
|
13:23:34
|
<dkayiwa> rafa: the second last comment
|
13:23:39
|
<rafa> !ticket SXS-1
|
13:23:42
|
<OpenMRSBot> rafa: [#SXS-1] Fix unit tests in serialization module - OpenMRS JIRA - https://tickets.openmrs.org/browse/SXS-1
|
13:24:39
|
<rafa> dkayiwa: well what is the error?
|
13:24:57
|
<dkayiwa> rafa: problem is the assertion which fails
|
13:25:02
|
<rafa> dkayiwa: ohh ok
|
13:26:07
|
<downey> kishoreyekkanti: hey, i think your twitter account got compromised
|
13:26:38
|
<kishoreyekkanti> downey: Hey Yep. Just found it and revoked all the apps and changed my password :(
|
13:26:42
|
<kishoreyekkanti> downey: Sorry for the spam
|
13:26:43
|
<rafa> dkayiwa: seems like a problem with XStreamSerializer
|
13:26:48
|
<downey> kishoreyekkanti: it happens :)
|
13:26:50
|
<dkayiwa> rafa: ok
|
13:26:51
|
<rafa> dkayiwa: maybe it looks in a DB
|
13:27:04
|
<kishoreyekkanti> downey: let me know if you get any further messages!
|
13:27:08
|
<rafa> dkayiwa: can you put Context.flushSession() after cohort.getMemberIds().clear();
|
13:27:21
|
<dkayiwa> rafa: ok let me try that. thanks!
|
13:27:38
|
<rafa> dkayiwa: maybe save first
|
13:27:45
|
<dkayiwa> rafa: ok
|
13:27:50
|
*** lakkarau has quit IRC
|
13:29:32
|
<rafa> dkayiwa: Hibernate typically stores original collection and dirty one
|
13:29:42
|
<dkayiwa> rafa: oh i see!!!
|
13:29:51
|
<rafa> dkayiwa: maybe XStreamSerializer serializes the original one
|
13:29:58
|
<dkayiwa> rafa: ok
|
13:31:10
|
*** mvorobey has quit IRC
|
13:32:09
|
<dkayiwa> rafa: thanks man!!! Only adding Context.flushSession(), even without saving, did the trick :)
|
13:32:39
|
<rafa> dkayiwa: yeah, but I think it acutally needs to be fixed in XStramSerializer :P
|
13:32:53
|
<dkayiwa> rafa: ok
|
13:33:28
|
<rafa> dkayiwa: in some CollectionConverter or something
|
13:33:45
|
<dkayiwa> rafa: ok. thanks for the pointer :)
|
13:35:13
|
<rafa> dkayiwa: HibernateCollectionConverter does: col.getStoredSnapshot()
|
13:35:23
|
<rafa> dkayiwa: it's wrong
|
13:35:24
|
<dkayiwa> rafa: oh i see!!!
|
13:36:03
|
*** pushkar has joined #openmrs
|
13:36:16
|
<rafa> dkayiwa: see how it's done in the MDS module :)
|
13:36:17
|
<pushkar> bwolfe, Hi
|
13:36:36
|
<dkayiwa> rafa: ok thanks for making my life easier! :)
|
13:37:04
|
<rafa> dkayiwa: happy to help :)
|
13:37:28
|
<dkayiwa> rafa: you do not know how long i had battled with it! :D
|
13:38:30
|
<rafa> dkayiwa: I had many battles with Hibernate too ;)
|
13:38:51
|
<dkayiwa> rafa: hahahaha . you battles saved mine :D
|
13:48:18
|
<dkayiwa> rafa: do you have an off the head name of the class i should check in the MDS module? :)
|
13:48:54
|
<rafa> dkayiwa: HibernatePersistentCollectionConverter?
|
13:49:02
|
<dkayiwa> rafa: ok thanks
|
13:49:55
|
<bwolfe> rafa++
|
13:50:15
|
<dkayiwa> rafa++ :)
|
13:50:38
|
*** kishoreyekkanti_ has joined #openmrs
|
13:51:14
|
<rafa> ohh I scored today more than ever :D
|
13:51:34
|
*** kishoreyekkanti_ has quit IRC
|
13:51:41
|
*** lakkarau has joined #openmrs
|
13:51:41
|
*** magoo_ has quit IRC
|
13:51:51
|
*** kishoreyekkanti_ has joined #openmrs
|
13:52:27
|
*** kishoreyekkanti has quit IRC
|
13:56:26
|
*** sgithens has joined #openmrs
|
13:57:45
|
*** jriley has joined #openmrs
|
14:04:29
|
<bwolfe> scrum time, but no cpower...
|
14:06:07
|
<bwolfe> lets get started without him.
|
14:06:13
|
<bwolfe> !scrumon
|
14:06:13
|
* OpenMRSBot says the DAILY SCRUM MEETING is STARTING. This meeting should not last longer than 15 minutes. Please hold other comments until the end of the meeting, or message someone privately. Thank you! ScrumMaster - you may begin when ready.
|
14:06:55
|
<bwolfe> today's order: dkayiwa, rafa, lakkarau, wyclif, bwolfe, djazayeri (and mvorobey if he shows up again)
|
14:07:07
|
<bwolfe> (and venkaban last if he shows up again as well)
|
14:08:29
|
<dkayiwa> Wasted the entire day in town working with my bank to correct problem where one was signing cheques on my behalf
|
14:08:30
|
<dkayiwa> Working on: Fix unit tests in serialization module - SXS-1
|
14:08:30
|
<dkayiwa> No Blockers
|
14:08:45
|
<rafa> Today:
|
14:08:45
|
<rafa> Experimented with maven
|
14:08:45
|
<rafa> Reviewed TRUNK-3245 Limit concept mappings on concept drugs to concepts with Drug class
|
14:08:45
|
<rafa> In Progress TRUNK-3469 Incompatibility with postgresql -- data type double should be "double precision" for postgresql
|
14:08:53
|
<lakkarau> bwolfe:Thursday:
|
14:08:54
|
<lakkarau> -worked on the ticket-#trunk-3328.
|
14:08:54
|
<lakkarau> -refactored the code and find that still geting NPE.
|
14:08:54
|
<lakkarau> Today :
|
14:08:54
|
<lakkarau> -work on the ticket #trunk-3328.
|
14:08:54
|
<lakkarau> Monday:
|
14:08:59
|
<lakkarau> will check-in the code if the code works fine..
|
14:09:54
|
<wyclif> Thursday:
|
14:09:54
|
<wyclif> dev call
|
14:09:54
|
<wyclif> REPORT-383 - On run / schedule report page, allow for expressions to be specified in addition to fixed values
|
14:09:54
|
<wyclif> Spent some time on REPORT-374, then mike commented he had covered it in another task - UI pages for creating and managing report processor configurations
|
14:09:56
|
<wyclif> GSoc project code review
|
14:09:58
|
<wyclif> Friday:
|
14:10:00
|
<wyclif> REPORT-399 - Support "groups of things" in UI that are not patient cohorts
|
14:10:02
|
<wyclif> Weekly gsoc chat with Dmitry
|
14:10:04
|
<wyclif> pick up a bug ticket
|
14:10:06
|
<wyclif>
|
14:10:08
|
<wyclif> Blockers: None
|
14:11:18
|
<bwolfe> Thursday
|
14:11:18
|
<bwolfe> reviewed rafal's role ticket
|
14:11:18
|
<bwolfe> get htmlformentry testing in ci
|
14:11:18
|
<bwolfe> dev call
|
14:11:18
|
<bwolfe> leaderhsip call
|
14:11:19
|
<bwolfe> review daniel's sprint tickets
|
14:11:21
|
<bwolfe> fixed HTML-362 for relationships
|
14:11:23
|
<bwolfe> fixed TRUNK-3473 concurrent trunk testing
|
14:11:25
|
<bwolfe> Friday
|
14:11:27
|
<bwolfe> emails
|
14:11:29
|
<bwolfe> review daniel's sxs code
|
14:11:31
|
<bwolfe> fix github authors
|
14:11:33
|
<bwolfe> fix htmlformentry compiling
|
14:11:35
|
<bwolfe> no blockers
|
14:11:37
|
<bwolfe> (I'm leaving after lunch today, so get your questions in early)
|
14:11:47
|
<bwolfe> djazayeri, you awake yet?
|
14:13:05
|
<bwolfe> guess not, lets scrum off
|
14:13:13
|
<bwolfe> !scrumoff
|
14:13:13
|
* OpenMRSBot says the DAILY SCRUM MEETING has ENDED. This channel is now returned to normal hacking operations. Post-scrum meeting follow-up conversations may now begin.
|
14:13:19
|
<bwolfe> discussion points:
|
14:13:20
|
<bwolfe> dkayiwa: get all your money back? :-(
|
14:13:20
|
<bwolfe> rafa: experimented with what in maven?
|
14:13:20
|
<bwolfe> lakkarau: perhaps you're over thinking your ticket, lets discuss the approach
|
14:13:35
|
<rafa> bwolfe: dependencies
|
14:13:36
|
<dkayiwa> bwolfe: fortunately the money was not mine :)
|
14:13:36
|
<bwolfe> lakkarau, are you home now? or are you working already?
|
14:13:38
|
<lakkarau> bwolfe:ok
|
14:13:48
|
<dkayiwa> bwolfe: i was just a signatory
|
14:13:48
|
<downey> bwolfe: http://mavenrepo.openmrs.org/nexus/content/repositories/snapshots/org/openmrs/openmrs/1.9.0-SNAPSHOT/
|
14:13:50
|
<lakkarau> bwolfe:yes
|
14:13:56
|
<rafa> bwolfe: for the question you asked on the dev list
|
14:13:56
|
<bwolfe> downey++
|
14:14:31
|
<lakkarau> bwolfe i am home and will be going to office .
|
14:14:44
|
<dkayiwa> bwolfe: and good enough i responded quickly enough when the first cheque had just been cashed :)
|
14:15:14
|
<bwolfe> lakkarau, ok, ping me as soon as you get in? 1 hour?
|
14:15:36
|
<lakkarau> bwolfe:sure
|
14:15:38
|
<bwolfe> rafa, ok, I'm also looking into it, so no need to duplicate. you fix your NPE problem for now. ;-)
|
14:15:49
|
<jriley> bwolfe and djazayeri, Neal Lesh sends his regards. He told me about the Neal Reports.
|
14:16:08
|
<bwolfe> jriley, :-)
|
14:16:14
|
<rafa> bwolfe: yeah that was in the morning, now I'm working on NPE :)
|
14:16:23
|
<jriley> It was a cautionary tale about "temporary" code
|
14:16:28
|
<dkayiwa> rafa: your HibernatePersistentCollectionConverter fixed the unit test. am tempted to overwrite the entire HibernateCollectionConverter.marshal with yours. do you advise me to do that?
|
14:16:37
|
<bwolfe> nearlreports was intentionally named hackily so that we were sure to pull it out someday. its just that it took 4 years to do it. :-(
|
14:18:27
|
<rafa> dkayiwa: If it works then I don't see a reason why not to do it
|
14:18:37
|
*** pushkar has quit IRC
|
14:18:39
|
<bwolfe> downey, both 1.8.x and 1.9.x building nightly. :-) are you pushing the war to a server somewhere?
|
14:18:42
|
* bwolfe crosses fingers
|
14:18:45
|
<downey> bwolfe: not yet
|
14:19:01
|
<bwolfe> downey, you can probably skip 1.7.x, it might be a pain to push to nexus because its not mavenized
|
14:19:15
|
<downey> bwolfe: that's what i did
|
14:19:42
|
*** sgithens has quit IRC
|
14:19:59
|
<downey> bwolfe: i need to work out a new file structure for sourceforge first
|
14:20:32
|
<dkayiwa> rafa: ok
|
14:30:55
|
*** lakkarau_ has joined #openmrs
|
14:30:56
|
*** lakkarau has quit IRC
|
14:31:40
|
<wyclif> hi djazayeri
|
14:34:59
|
*** pushkar has joined #openmrs
|
14:35:23
|
*** sgithens has joined #openmrs
|
14:35:48
|
*** lakkarau_ has quit IRC
|
14:36:46
|
*** lh has joined #openmrs
|
14:40:06
|
<pushkar> bwolfe, Hi
|
14:41:00
|
<pushkar> other than the js variable are there any other java variable which provides the contextpath ?
|
14:43:23
|
<bwolfe> wyclif, fork.
|
14:43:34
|
<bwolfe> wyclif, the r is important.
|
14:43:38
|
<bwolfe> :-)
|
14:43:57
|
<bwolfe> pushkar, a java variable? sure, theres a WebConstants one
|
14:44:11
|
<bwolfe> WebConstants.WEBAPP_URL or something
|
14:47:24
|
<wyclif> bwolfe, i dont know why i type folk, i need to get the l off my keyboard and out of my head
|
14:48:14
|
*** harshadura has joined #openmrs
|
14:49:00
|
*** dkayiwa has quit IRC
|
14:50:22
|
*** venkaban has joined #openmrs
|
14:50:34
|
<venkaban> bwolfe, Hi
|
14:50:44
|
<venkaban> bwolfe, Sorry I missed today's Scrum
|
14:50:54
|
<venkaban> bwolfe, Here are my updates
|
14:51:08
|
<venkaban> bwolfe, completed HTML 357 and currently workig on 357
|
14:51:12
|
<venkaban> No Blockers
|
14:51:21
|
*** dkayiwa has joined #openmrs
|
14:53:23
|
<bwolfe> !ticket TRUNK-357
|
14:53:24
|
<OpenMRSBot> bwolfe: [#TRUNK-357] ajax search widget for adding a drug to regimen - OpenMRS JIRA - https://tickets.openmrs.org/browse/TRUNK-357
|
14:53:33
|
<bwolfe> venkaban, aren't those the same number you put in?
|
14:53:41
|
<bwolfe> oops
|
14:53:45
|
<bwolfe> !ticket HTML-357
|
14:53:46
|
<OpenMRSBot> bwolfe: [#HTML-357] Verify unit tests are working in HTML Form Entry Extensions module after mavenization - OpenMRS JIRA - https://tickets.openmrs.org/browse/HTML-357
|
14:54:20
|
<pushkar> bwolfe, theres a WEBAPP_NAME but it just provides the name of the webapp
|
14:55:04
|
<bwolfe> pushkar, right, so you do "/" + WEBAPP_NAME + "/moduleResources/drawing/..."
|
14:55:54
|
<pushkar> Oh yes Didnt think of that ! Thank you ben :-)
|
14:56:56
|
<dkayiwa> bwolfe: do you have an idea why serialization.stream bamboo fails yet all tests pass locally on my computer?
|
14:58:53
|
<bwolfe> dkayiwa, here are the full build logs
|
14:58:53
|
<bwolfe> https://ci.openmrs.org/download/BUNDLED-SXS-JOB1/build_logs/BUNDLED-SXS-JOB1-4.log
|
14:58:58
|
*** mvorobey has joined #openmrs
|
14:59:06
|
<bwolfe> build 22-Jun-2012 10:53:47 Running org.openmrs.module.xstream.TestUtil
|
14:59:06
|
<bwolfe> build 22-Jun-2012 10:53:47 Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.004 sec <<< FAILURE!
|
14:59:24
|
<bwolfe> dkayiwa, its because the new TestUtil you put in there needs an @Ignore on it.
|
14:59:33
|
<dkayiwa> bwolfe: ok thanks :)
|
14:59:35
|
<bwolfe> junit sees its named "Test*" and tries to execute it as a junit test
|
14:59:47
|
<bwolfe> dkayiwa, if you run "mvn clean test" at command line you would probably get this same error
|
14:59:56
|
<dkayiwa> bwolfe: ok
|
15:00:06
|
<bwolfe> thats what ci runs, and thats probably what you should run on every project before committing
|
15:00:14
|
<dkayiwa> bwolfe: ok
|
15:01:33
|
<mvorobey> bwolfe, how about my prev question ? do you have any ideas on this ? (p.s.: I've cleaned project, tried from scratch but it did not help ... )
|
15:02:22
|
<dkayiwa> bwolfe: surprisingly mvn clean test passes!!!
|
15:02:44
|
<mvorobey> dkayiwa, it is not possible :)
|
15:02:55
|
<dkayiwa> mvorobey: but it does :)
|
15:03:02
|
<mvorobey> :D
|
15:03:50
|
<mvorobey> dkayiwa, I've faced in the same issue in reporting, so I tried to do clean, ran test again, but nothing helped until I ignored the testutil class
|
15:04:05
|
<mvorobey> faced in = faced to
|
15:04:19
|
<dkayiwa> mvorobey: ok :)
|
15:05:00
|
<pushkar> The enter part of htmlformentry is done bwolfe !
|
15:09:11
|
<bwolfe> pushkar, wow, awesome! so you can draw on an image and have it persisted when you save a form?
|
15:09:55
|
<pushkar> yes
|
15:10:26
|
<pushkar> I've not yet finished the view and edit part
|
15:10:37
|
*** kishoreyekkanti_ has quit IRC
|
15:10:49
|
<bwolfe> pushkar, but the hard part is done. :-)
|
15:11:04
|
<bwolfe> pushkar, you really need to talk to jibesh asap to make sure there is no duplicate there
|
15:11:12
|
<bwolfe> you might have something to help him with
|
15:11:15
|
<bwolfe> or he can help you
|
15:11:40
|
<pushkar> okay
|
15:13:20
|
<mvorobey> bwolfe, seems like provider_id colunmn is missing for encounter in 1.9.x but in used in org.openmrs.module.reporting.cohort.query.db.hibernate.HibernateCohortQueryDAO.getPatientsHavingEncounters(Date, Date, TimeQualifier, List<Location>, List<Person>, List<EncounterType>, List<Form>, Integer, Integer, User, Date, Date)
|
15:13:29
|
*** jriley has quit IRC
|
15:14:39
|
<bwolfe> mvorobey, oooo
|
15:15:05
|
<bwolfe> mvorobey, in 1.9 encounter.provider_id was removed in place of a many-to-1 table I think. sorry for leading you astray. :-(
|
15:15:17
|
<bwolfe> mvorobey, in 1.8 and below encounter.provider_id pointed to users.user_id
|
15:16:00
|
<mvorobey> bwolfe, so, what is a workaround to get that test working ?
|
15:16:45
|
<bwolfe> mvorobey, its just in the dbunit xml file, right?
|
15:17:00
|
<mvorobey> bwolfe, nope, javacode
|
15:17:07
|
<bwolfe> mvorobey, oh, hmm,
|
15:17:34
|
<mvorobey> bwolfe, here is that failing method - org.openmrs.module.reporting.cohort.definition.evaluator.EncounterCohortDefinitionEvaluatorTest.evaluate_shouldReturnAllPatientsWithEncountersIfAllArgumentsToCohortDefinitionAreEmpty()
|
15:19:09
|
*** harshadura has quit IRC
|
15:20:10
|
*** dkayiwa has left #openmrs
|
15:20:24
|
*** harshadura has joined #openmrs
|
15:20:42
|
<mvorobey> bwolfe, will be AFK for couple of hours will ping you later
|
15:21:05
|
<bwolfe> mvorobey, ok
|
15:21:37
|
<bwolfe> mvorobey, I'm not sure how to handle java compilation problems.
|
15:21:55
|
<bwolfe> mvorobey, you might have to get fancy with how the evaluators are defined (are they in xml files?)
|
15:24:41
|
*** mccallumg has joined #openmrs
|
15:24:41
|
*** ChanServ sets mode: +v mccallumg
|
15:24:53
|
*** mvorobey has quit IRC
|
15:27:37
|
*** mvorobey has joined #openmrs
|
15:27:56
|
<mvorobey> bwolfe, am back, will be available for 15 minute - 20 minutes while am far from the base station
|
15:28:26
|
<bwolfe> mvorobey, mmk, don't really get you
|
15:28:33
|
<bwolfe> mvorobey, ok
|
15:28:33
|
<bwolfe> mvorobey, I'm not sure how to handle java compilation problems.
|
15:28:33
|
<bwolfe> mvorobey, you might have to get fancy with how the evaluators are defined (are they in xml files?)
|
15:29:26
|
<mvorobey> bwolfe, am on way to home, using 3g modem, connectivity will be lost in 15 minutes
|
15:29:31
|
*** magoo_ has joined #openmrs
|
15:29:41
|
<mvorobey> bwolfe, bwolfe it's not a compilation problem
|
15:29:57
|
<mvorobey> bwolfe, it's sql query evaluation problem
|
15:30:10
|
<bwolfe> ok, so you need to get a diff sql query in there for 1.9+
|
15:30:26
|
*** venkaban has quit IRC
|
15:31:09
|
<downey> bwolfe: https://sourceforge.net/projects/openmrs/files/nightly/2012-06-22/
|
15:31:14
|
<OpenMRSBot> <http://ln-s.net/$Kw6> (at sourceforge.net)
|
15:32:05
|
<mvorobey> bwolfe, yeah, agreed, but how to do it if sql query is created dynamically using hibernate utils ?
|
15:32:41
|
<bwolfe> dynamically off of the db? or what?
|
15:32:59
|
<mvorobey> bwolfe, wait a minute, doing a paste
|
15:34:05
|
*** djazayeri has quit IRC
|
15:35:27
|
<mvorobey> bwolfe, here is a paste - http://pastebin.com/nZU3CiGU, see the line with comment // I THINK THAT THIS IS A SOURCE OF THE PAIN :)
|
15:35:51
|
*** magoo_ has quit IRC
|
15:37:15
|
<bwolfe> mvorobey, is that in code, or in a test?
|
15:38:00
|
<mvorobey> bwolfe, in code - org.openmrs.module.reporting.cohort.query.db.hibernate.HibernateCohortQueryDAO
|
15:42:29
|
<mvorobey> bwolfe, I'd change it not asking you if it would be in test
|
15:43:10
|
*** Jibesh has joined #openmrs
|
15:44:49
|
*** magoo_ has joined #openmrs
|
15:46:41
|
<pushkar> bwolfe,
|
15:47:29
|
<pushkar> rather than having a seprate drawing tag I think we should create a drawing widget which can be set to the existing obs tag
|
15:47:49
|
<pushkar> like <obs .... style='drawing'/>
|
15:47:57
|
<pushkar> what do you think ?
|
15:49:08
|
<wyclif> hi dj
|
15:50:46
|
<bwolfe> pushkar, if that is possible, yes. then you don't have to duplicate some of the attributes
|
15:51:00
|
<bwolfe> pushkar, only investigate that if there are some clear advantages
|
15:51:13
|
<bwolfe> otherwise its just cleanup for the sake of cleanup
|
15:52:06
|
<pushkar> obs tag provides all the ways for creating an obs
|
15:54:49
|
<pushkar> providing different tag for creating an observation (complexobservation) dosent look userfriendly to me
|
15:55:45
|
<pushkar> But I have to further investigate on the complexity of the above idea
|
15:57:02
|
<Jibesh> pushkar I second you on your idea
|
15:57:29
|
<Jibesh> I have implemented something similar to that
|
15:57:56
|
<pushkar> Jibesh, :-)
|
16:04:08
|
*** wyclif has quit IRC
|
16:04:08
|
*** wluyima has joined #openmrs
|
16:06:04
|
*** sthaiya has joined #openmrs
|
16:06:16
|
<downey> Hi sthaiya and welcome to the #openmrs IRC channel.
|
16:06:32
|
<sthaiya> :) Hi Downey
|
16:07:10
|
*** mvorobey has quit IRC
|
16:07:45
|
*** magoo_ has quit IRC
|
16:09:45
|
<pushkar> bwolfe, can I change the dependency of htmlformentry to 1.9 ?
|
16:10:09
|
<pushkar> *openmrs dependency
|
16:12:12
|
<sthaiya> is there script to mavenize a mercurial hosted module?
|
16:20:32
|
*** lakkarau has joined #openmrs
|
16:26:14
|
<sthaiya> rafa
|
16:27:36
|
*** applecool has joined #openmrs
|
16:27:36
|
*** ChanServ sets mode: +v applecool
|
16:32:33
|
*** elliott_w has joined #openmrs
|
16:32:33
|
*** ChanServ sets mode: +o elliott_w
|
16:33:58
|
*** mccallumg has quit IRC
|
16:34:19
|
*** mccallumg has joined #openmrs
|
16:34:19
|
*** ChanServ sets mode: +v mccallumg
|
16:36:33
|
*** kreddy has joined #openmrs
|
16:37:07
|
<bwolfe> pushkar, no, I don't think you should. that is up to mark goodrich to decide (he's the module owner)
|
16:37:34
|
*** wyclif has joined #openmrs
|
16:38:07
|
<bwolfe> sthaiya, if the layout is like our old modules, the mavenize script should work fine on it
|
16:38:27
|
<bwolfe> sthaiya, can you paste your stack trace and problem here? rafa might be away for now
|
16:39:19
|
<bwolfe> Jibesh, pushkar: but can a module modify an existing tag? how would the drawing module do its thing if it was an attribute?
|
16:40:09
|
*** sthaiya has quit IRC
|
16:40:38
|
*** sthaiya has joined #openmrs
|
16:40:43
|
*** venkaban has joined #openmrs
|
16:41:30
|
<pushkar> bwolfe, oops I just commited it !
|
16:41:49
|
<bwolfe> committed to your fork of htmlformentry?
|
16:42:03
|
<Jibesh> bwolfe What I have implemented is <obs conceptId="6100">.....Now this concept id is of type complex ....On wrting this in Html form a custom widget will get rendered
|
16:42:36
|
<bwolfe> Jibesh, yeah, you are coding directly against htmlformentry though. pushkar's drawing element is in a module and won't be in htmlformentry core module
|
16:42:54
|
<Jibesh> Ok...got it
|
16:45:30
|
<sthaiya> bwolfe, just sorted out the issue with metadata module
|
16:45:39
|
<bwolfe> sthaiya, what was the problem?
|
16:45:48
|
<sthaiya> problem was due to form export import module
|
16:46:07
|
*** djazayeri has joined #openmrs
|
16:46:07
|
*** ChanServ sets mode: +o djazayeri
|
16:46:10
|
<sthaiya> there is a ticket for it
|
16:47:25
|
<bwolfe> sthaiya, problem with MDS or importexport? which one did you file the ticket in?
|
16:47:56
|
<pushkar> bwolfe, yes
|
16:48:06
|
<sthaiya> i did not file the ticket
|
16:48:22
|
<sthaiya> the ticket was already filed by Judy
|
16:48:48
|
*** jordank has joined #openmrs
|
16:49:09
|
<sthaiya> on MDS but rafa closed it and opened one under FIE module
|
16:50:25
|
<bwolfe> oh, sorry, misread you the first time sthaiya
|
16:50:37
|
<bwolfe> I would guess FIE won't be updated. its ooold
|
16:50:49
|
<bwolfe> so if you see the problem and/or can supply a patch, all hte better
|
16:50:51
|
<sthaiya> :)
|
16:50:52
|
<applecool> rafa: there? busy?
|
16:52:37
|
*** suranga has joined #openmrs
|
16:52:37
|
*** ChanServ sets mode: +v suranga
|
16:53:40
|
<bwolfe> lakkarau, at work yet?
|
16:53:59
|
<lakkarau> bwolfe:yes i am at work
|
16:54:21
|
<bwolfe> ok, ready to talk? I leave very soon. I was expecting you to ping me as soon as you got there...
|
16:54:40
|
<lakkarau> bwolfe:i am ready
|
16:57:49
|
<lakkarau> bwolfe: i added tht statement "new ConceptServiceImpl().getConcepts("trust", Collections.singletonList(Locale.ENGLISH), false, null, null, null,null, null, null, null);" for testing and it throws NPE after running.
|
16:58:11
|
<bwolfe> lakkarau, good, so now you have created the failing unit test. now you make that test NOT throw an error
|
16:58:32
|
<lakkarau> bwolfe: yes i am trying to do that
|
16:58:33
|
<bwolfe> lakkarau, just change ConcpetServiceImpl to assign an empty list to variable if a null one is passed in
|
16:58:41
|
*** suranga_ has joined #openmrs
|
17:00:52
|
<lakkarau> bwolfe:i can see that already that check is done for the datatypes and classees like----excludeDatatypes = new Vector<ConceptDatatype>();
|
17:01:27
|
*** suranga has quit IRC
|
17:02:09
|
*** kishoreyekkanti has joined #openmrs
|
17:02:28
|
<bwolfe> lakkarau, ok...so copy that for the parameter mentioned in the ticket
|
17:03:18
|
*** pushkar has quit IRC
|
17:06:08
|
<lakkarau> bwolfe: means for datatypes and classes
|
17:06:35
|
*** kishoreyekkanti has quit IRC
|
17:07:31
|
<lakkarau> bwolfe: i did not understand the above statement
|
17:07:37
|
*** kishoreyekkanti has joined #openmrs
|
17:07:51
|
<bwolfe> lakkarau, which ticket?
|
17:08:33
|
<lakkarau> bwolfe:#Trunk-3328
|
17:09:49
|
<suranga_> good morning james_regen
|
17:10:01
|
<suranga_> james_regen, any chance for a quick question or two today ?:)
|
17:11:54
|
<james_regen> suranga_: hi
|
17:12:07
|
<suranga_> james_regen, howdy !
|
17:12:22
|
<bwolfe> lakkarau, hmm, yes, it seems to be doing what he wants already. perhaps he was using an earlier version that didn't have htat already?
|
17:12:31
|
<suranga_> james_regen, I was trying to test if the estimated number of pairs which pulasthi is calculating are correct or not
|
17:13:13
|
<bwolfe> 1) check the svn history of the files ConceptServiceImpl and/or HibernateConceptDAO to see if changes have been committed
|
17:13:24
|
<bwolfe> 2) check out the 1.9.x branch and the 1.8.x branch to see if you can get the test to fail
|
17:13:31
|
<bwolfe> 3) what line is the NPE on ?
|
17:13:35
|
<bwolfe> lakkarau ^^
|
17:14:18
|
<lakkarau> bwolfe:bwolfe i think the NPE now i am getting is because of the last three parameters i have passed as nulls that is the Concept ,size and the Start.
|
17:14:29
|
<james_regen> suranga_: do you know how to do that?
|
17:14:38
|
<suranga_> james_regen, I have an idea
|
17:14:44
|
*** applecool has quit IRC
|
17:14:46
|
<bwolfe> lakkarau, if the javadoc says those are nullable, then thats a different problem to fix. (and needs it)
|
17:14:55
|
<suranga_> james_regen, its too detailed for IRC, im putting it into a email
|
17:15:01
|
<suranga_> james_regen, :)
|
17:15:02
|
<bwolfe> if the javadoc says they're required, then leave them as they are
|
17:15:38
|
<james_regen> suranga_: ok. i also have an idea that should fit on IRC, though i'm still curious what your idea is
|
17:16:58
|
<lakkarau> bwolfe: okay i will do the steps you gave.
|
17:20:08
|
*** mvorobey has joined #openmrs
|
17:20:36
|
*** applecool has joined #openmrs
|
17:20:36
|
*** ChanServ sets mode: +v applecool
|
17:21:58
|
*** djazayeri has quit IRC
|
17:22:29
|
*** bwolfe has quit IRC
|
17:24:31
|
*** kreddy has quit IRC
|
17:24:59
|
*** kreddy has joined #openmrs
|
17:25:12
|
<suranga_> james_regen, hi, just emailed you
|
17:25:34
|
<suranga_> james_regen, im also very open to your suggestion on how to test this :)
|
17:27:33
|
<james_regen> suranga_: when we run the process to create a strategy, there's a step where we analyze the data using EM to calculate some values for the fields. this analysis should be printed in the log file. one of the lines printed, for each block or strategy, si the number of pairs in the block
|
17:28:51
|
<suranga_> james_regen, um... you mean when we create a strategy, or a report :)
|
17:29:29
|
<james_regen> suranga_: oops, when we create a report
|
17:29:59
|
<suranga_> james_regen, um.. you mean that somewhere in the log output, this is being calculated already ?
|
17:30:06
|
<suranga_> james_regen, neat
|
17:30:32
|
*** rafa has quit IRC
|
17:30:34
|
<james_regen> suranga_: yeah, in catalina.out you shoould be able to search for "pairs processed" or something like that. i'd have to check to see what it is exactly
|
17:31:35
|
<james_regen> suranga_: looks like the line is "Total pairs processed:"
|
17:32:00
|
<suranga_> james_regen, I just found it :-)
|
17:32:25
|
<james_regen> suranga_: and some logging output of what the blocking columns were should be 10 or so lines above it, so you can match up the strategy to pair count
|
17:32:57
|
<james_regen> suranga_: how is the HQL performance? does the query take a long time to run?
|
17:33:16
|
<suranga_> james_regen, its on link.regenstrifef, you can check it out
|
17:33:21
|
<suranga_> james_regen, the timing is not tooo bad
|
17:33:40
|
<suranga_> but we may need to do something to tell the user to 'wait' until the calculation is done
|
17:33:54
|
<suranga_> james_regen, it takes about 10-15 seconds max, as I remeber
|
17:34:20
|
<james_regen> suranga_: that's not too bad considering creating the scratch database and creating the report could take hours
|
17:34:44
|
*** applecool has quit IRC
|
17:34:47
|
<suranga_> james_regen, I tried the report for a column that has only two matching values
|
17:35:02
|
<suranga_> james_regen, pretend out data is a,b,x,c,1,1
|
17:35:22
|
<suranga_> james_regen, this (as expected) returns the number of estimated pairs as '1'
|
17:35:48
|
*** sthaiya has quit IRC
|
17:35:50
|
<suranga_> james_regen, BUT
|
17:36:05
|
<suranga_> james_regen, consider the column values A,A,A,B,B,B
|
17:36:12
|
<suranga_> james_regen, this seems to return 6
|
17:36:28
|
*** djazayeri has joined #openmrs
|
17:36:28
|
*** ChanServ sets mode: +o djazayeri
|
17:36:39
|
<suranga_> james_regen, oh, thats correct, I see
|
17:36:43
|
<suranga_> james_regen, :)
|
17:39:03
|
*** venkaban_ has joined #openmrs
|
17:40:03
|
*** venkaban has quit IRC
|
17:40:37
|
*** applecool has joined #openmrs
|
17:40:37
|
*** ChanServ sets mode: +v applecool
|
17:41:10
|
<suranga_> james_regen, so the good news is that the values returnd from my test == your results
|
17:41:25
|
<suranga_> james_regen, the bad news is that they dont match pulasthi's
|
17:41:53
|
<suranga_> james_regen, pulasthi needs to divide his result by two to match ours :-)
|
17:42:19
|
<james_regen> suranga_: is there a clause in the HQL to add "where a.ID > b.ID" ?
|
17:42:43
|
*** harshadura has quit IRC
|
17:42:44
|
<james_regen> suranga_: that might be the problem. it might be counting pair {1,2} and {2,1}
|
17:43:13
|
<suranga_> james_regen, I think you are right (as usual :-)
|
17:43:56
|
<james_regen> suranga_: i remember talking about that a few weeks ago when we started this. compared to the work done, this is a pretty minor detail. i'm really impressed and glad that we're this close
|
17:45:16
|
<suranga_> james_regen, he he.. thanks. needless to say that if not for you, pulasthi and myself would have probably commited joint suicide by now :P
|
17:46:42
|
<suranga_> james_regen, and also, something I just noticed...
|
17:46:58
|
<suranga_> for patientmatching, identifiers can also be added as fields for matching
|
17:47:14
|
*** elliott_w has quit IRC
|
17:47:20
|
<suranga_> james_regen, but, since identifieirs are unique, then can never be duplicates, right ?
|
17:47:49
|
<suranga_> james_regen, therfore, every identifier should result in zero estimated pairs
|
17:49:22
|
<james_regen> suranga_: this probably gets into usage details of OpenMRS i'm not familiar with, since i don't really use it. identifiers are like attributes, right? extensible and user defined? i'm not sure what types of data is normally placed in there. i'm also not sure if the system enforcese uniquness for identifiers. if not, then it is likely that the quality of the data will degrade over time, and there will be duplicates
|
17:51:21
|
<suranga_> james_regen, not really, im afraid, OpenMRS identifiers dont allow duplicates :)
|
17:51:51
|
<suranga_> james_regen, if we try to create a duplicate identifier, it will give you an error :)
|
17:52:44
|
<james_regen> suranga_: that's good to know. it might be worth it to remove it from the module, then, since it won't be useful for matching in either type of field, and can be ignored when importing and when displaying the fields to the user. it'd make the page where the user selects fields smaller, at least
|
17:53:05
|
<suranga_> james_regen, ooops, I seee
|
17:54:04
|
<james_regen> suranga_: i'd email people asking if anyone sees a problem in doing this.
|
17:54:20
|
<suranga_> james_regen, aha.. mm... u mean to the dev list ?
|
17:54:49
|
<james_regen> i'd email the module people, judy, shaun, pulasthi
|
17:55:00
|
<james_regen> and ada
|
17:55:05
|
<suranga_> james_regen, cool, Iwill do that...
|
18:02:23
|
<snoppy> wyclif:hi, wyclif!
|
18:02:25
|
<wyclif> hi djazayeri
|
18:02:28
|
<wyclif> hi snoppy
|
18:02:40
|
<wyclif> snoppy, how old
|
18:02:49
|
<wyclif> snoppy, can you wait for about 20min
|
18:03:03
|
<snoppy> wyclif: yes, i can
|
18:03:30
|
<wyclif> snoppy, cool thanks
|
18:03:54
|
<snoppy> no problem)
|
18:08:41
|
*** lh has quit IRC
|
18:11:45
|
<djazayeri> wyclif: in a meeting
|
18:11:56
|
<wyclif> djazayeri, ok
|
18:19:28
|
*** maurya has joined #openmrs
|
18:19:53
|
*** ChanServ sets mode: +v maurya
|
18:33:26
|
*** Jibesh has quit IRC
|
18:36:49
|
*** applecool has quit IRC
|
18:38:20
|
*** applecool has joined #openmrs
|
18:38:20
|
*** ChanServ sets mode: +v applecool
|
18:41:08
|
<wyclif> hi snoppy
|
18:41:28
|
<snoppy> wyclif: hi
|
18:41:51
|
<wyclif> i think am ready snoppy
|
18:42:30
|
<snoppy> wyclif: ok, i'm waiting for you in skype
|
18:43:06
|
<snoppy> wyclif: or you want to speak in irc chat?
|
18:52:08
|
*** rafa has joined #openmrs
|
18:52:08
|
*** ChanServ sets mode: +v rafa
|
18:55:58
|
*** rafa has quit IRC
|
19:00:08
|
*** applecool has quit IRC
|
19:04:21
|
*** james_regen has quit IRC
|
19:05:19
|
*** djazayeri has quit IRC
|
19:08:40
|
*** mvorobey has quit IRC
|
19:15:16
|
*** djazayeri has joined #openmrs
|
19:15:16
|
*** ChanServ sets mode: +o djazayeri
|
19:16:45
|
<djazayeri> wyclif: hey, what's up?
|
19:21:52
|
<wyclif> djazayeri, hold on
|
19:21:57
|
<djazayeri> np
|
19:32:47
|
<wyclif> djazayeri, reporting has so many blockers
|
19:33:16
|
<wyclif> djazayeri, probably not so many a couple
|
19:33:17
|
<djazayeri> wyclif: what do you mean? that there are lots of things blocking you about the reporting tickets?
|
19:34:28
|
*** sgithens has quit IRC
|
19:34:38
|
<wyclif> djazayeri, cant run single unit tests, the module versions it seems to depend on seem to be none existent, i can fetch report design
|
19:35:06
|
<wyclif> djazayeri, i think there is something wrong in the mapping file
|
19:35:41
|
<djazayeri> wyclif: the module versions of htmlwidgets and serialization.xstream?
|
19:36:07
|
<djazayeri> wyclif: it was working before mavenization, so it's possible that mavenization has broken things
|
19:36:20
|
<djazayeri> what error do unit test failures report?
|
19:39:18
|
*** djazayeri has quit IRC
|
19:42:23
|
<wyclif> i think the slf4j lib id old
|
19:42:28
|
<wyclif> is old
|
20:20:15
|
*** lh has joined #openmrs
|
20:22:41
|
*** jordank has quit IRC
|
20:30:05
|
*** snoppy has left #openmrs
|
20:31:33
|
*** diptanuc has joined #openmrs
|
20:39:15
|
*** rafa has joined #openmrs
|
20:39:15
|
*** ChanServ sets mode: +v rafa
|
20:39:25
|
*** djazayeri has joined #openmrs
|
20:39:25
|
*** ChanServ sets mode: +o djazayeri
|
20:43:50
|
*** elliott_w has joined #openmrs
|
20:43:50
|
*** ChanServ sets mode: +o elliott_w
|
20:44:22
|
*** lakkarau has quit IRC
|
20:46:29
|
*** elliott_w has quit IRC
|
20:47:01
|
<venkaban_> djazayeri, Hi
|
20:47:25
|
<venkaban_> djazayeri, Can you please send me the link for the list of openmrs versions?
|
20:47:56
|
*** rafa has quit IRC
|
20:52:19
|
*** rafa has joined #openmrs
|
20:52:19
|
*** ChanServ sets mode: +v rafa
|
20:54:17
|
*** maurya has quit IRC
|
20:58:01
|
<djazayeri> venkaban_: it should be pretty a pretty obvious wiki search, or look at the irc logs from the other day when I pasted it
|
20:58:20
|
*** lakkarau has joined #openmrs
|
20:58:31
|
*** rafa has quit IRC
|
21:04:05
|
*** rafa has joined #openmrs
|
21:04:05
|
*** ChanServ sets mode: +v rafa
|
21:05:57
|
<OpenMRSBot> Recent updates in the world of openmrs: On Twitter: OpenMRS: Our #OpenMRS #GSoC students are hard at work on their projects. Some advice from student-turned-mentor @suranga_kas: http://t.co/6ZJzFNaZ <http://twitter.com/OpenMRS/statuses/216269142578302978>
|
21:07:29
|
*** rafa has quit IRC
|
21:09:31
|
*** venkaban_ has quit IRC
|
21:19:09
|
*** diptanuc has quit IRC
|
21:19:21
|
*** diptanuc has joined #openmrs
|
21:21:10
|
*** venkaban has joined #openmrs
|
21:36:42
|
*** wyclif has quit IRC
|
21:36:54
|
*** goutham has joined #openmrs
|
21:41:16
|
*** goutham has quit IRC
|
21:48:44
|
*** jordank has joined #openmrs
|
22:02:29
|
*** kreddy has quit IRC
|
22:03:58
|
*** suranga_ has quit IRC
|
22:15:23
|
*** mccallumg has quit IRC
|
22:15:36
|
*** mccallumg has joined #openmrs
|
22:15:36
|
*** ChanServ sets mode: +v mccallumg
|
22:25:43
|
*** r0bby has joined #openmrs
|
22:25:43
|
*** ChanServ sets mode: +v r0bby
|
22:29:50
|
*** robbyoconnor has quit IRC
|
22:54:01
|
*** downey has quit IRC
|
22:58:45
|
*** lh has quit IRC
|
23:01:27
|
*** wyclif has joined #openmrs
|
23:11:21
|
*** diptanuc has quit IRC
|
23:29:04
|
*** jordank_ has joined #openmrs
|
23:29:08
|
*** jordank has quit IRC
|
23:29:08
|
*** jordank_ is now known as jordank
|
23:40:27
|
*** lakkarau has quit IRC
|
23:53:21
|
*** venkaban has quit IRC
|