| 01:41:08 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-RESTWEBSERV-86 has FAILED : Scheduled build <https://ci-stg.openmrs.org/browse/SON-RESTWEBSERV-86> |
| 01:56:00 | *** wluyima has joined #openmrs |
| 02:19:16 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-OPENMRSCOREMASTER-88 has FAILED (4 tests failed) : Scheduled build <https://ci-stg.openmrs.org/browse/SON-OPENMRSCOREMASTER-88> |
| 02:54:03 | *** wluyima has quit IRC |
| 02:59:49 | *** djazayeri has quit IRC |
| 03:04:47 | *** robbyoconnor has quit IRC |
| 03:05:18 | *** robbyoconnor has joined #openmrs |
| 03:05:19 | *** ChanServ sets mode: +v robbyoconnor |
| 03:06:10 | *** yanosz has quit IRC |
| 03:11:25 | *** Shangeetha has joined #openmrs |
| 03:18:59 | *** yanosz has joined #openmrs |
| 03:23:15 | *** yanosz has quit IRC |
| 03:23:25 | *** wluyima has joined #openmrs |
| 03:28:17 | *** GitHub162 has joined #openmrs |
| 03:28:17 | <GitHub162> [openmrs-core] wluyima reopened pull request #475: [#TRUNK-4159] Add OrderFrequency to the data model (1.10.x...TRUNK-4159) http://git.io/MH58BA |
| 03:28:17 | *** GitHub162 has left #openmrs |
| 03:31:49 | *** yanosz has joined #openmrs |
| 03:33:31 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-UICOMMONS-90 has FAILED : Scheduled build <https://ci-stg.openmrs.org/browse/SON-UICOMMONS-90> |
| 03:46:28 | <Shangeetha> Hi we have basic opd, ipd flow compatible with 1.8 and radiology flow compatible with 1.9. Can anyone pls help us to link two versions of OMRS |
| 03:50:23 | *** Sumith has joined #openmrs |
| 03:51:03 | *** mshaikh has joined #openmrs |
| 03:58:40 | *** djazayeri has joined #openmrs |
| 03:58:41 | *** ChanServ sets mode: +o djazayeri |
| 04:00:01 | *** Bob__ has joined #openmrs |
| 04:00:09 | <Bob__> Teehee |
| 04:05:03 | *** Bob__ has quit IRC |
| 04:08:05 | <Shangeetha> djazayeri: Hi we have basic opd, ipd flow compatible with 1.8 and radiology flow compatible with 1.9. Can u pls suggest us how to link two versions of OMRS? Thanks :) |
| 04:13:54 | *** vinayv has joined #openmrs |
| 04:17:34 | *** Shangeetha has quit IRC |
| 04:18:06 | *** Sumith has quit IRC |
| 04:28:00 | *** wluyima has quit IRC |
| 04:28:23 | *** wluyima has joined #openmrs |
| 04:33:44 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-UILIBRARY-87 has FAILED : Scheduled build <https://ci-stg.openmrs.org/browse/SON-UILIBRARY-87> |
| 05:03:19 | *** djazayeri has quit IRC |
| 05:07:03 | *** vinayv has quit IRC |
| 05:10:32 | *** wluyima has quit IRC |
| 05:15:57 | *** vinayv has joined #openmrs |
| 05:40:07 | *** dkayiwa has joined #openmrs |
| 06:10:37 | *** Shangeetha has joined #openmrs |
| 06:10:44 | <Shangeetha> dkayiwa: Hi |
| 06:11:43 | <Shangeetha> dkayiwa: We need a help. We have deployed OMRS 1.8.3 and we have to open OMRS 1.9.3 in separate tab when radiology record tab is clicked in patient dashboard view. |
| 06:12:07 | <Shangeetha> dkayiwa: How can we acheive this task? Can you suggest any ideas? Thanks :) |
| 06:24:58 | *** Shangeetha has quit IRC |
| 06:38:03 | *** GitHub156 has joined #openmrs |
| 06:38:04 | <GitHub156> [openmrs-core] dkayiwa pushed 1 new commit to 1.10.x: http://git.io/jCYu0g |
| 06:38:04 | <GitHub156> openmrs-core/1.10.x b426919 dkayiwa: Throwing exceptions for invalid next order number global property -... |
| 06:38:04 | *** GitHub156 has left #openmrs |
| 07:00:32 | *** mshaikh has quit IRC |
| 07:18:42 | *** djazayeri has joined #openmrs |
| 07:18:43 | *** ChanServ sets mode: +o djazayeri |
| 07:31:33 | *** Shangeetha has joined #openmrs |
| 07:31:43 | <Shangeetha> dkayiwa: Hi |
| 07:34:01 | *** Shangeetha has quit IRC |
| 07:50:01 | *** Sakith has joined #openmrs |
| 07:55:16 | *** Shangeetha has joined #openmrs |
| 07:58:11 | *** Saki has joined #openmrs |
| 08:00:02 | <Saki> 1 |
| 08:02:28 | *** Saki has quit IRC |
| 08:03:18 | *** Saki has joined #openmrs |
| 08:03:47 | *** Sakith has quit IRC |
| 08:18:38 | <OpenMRSBot> Recent updates in the world of openmrsci: EMRAPI-EMRAPI-354 has FAILED (59 tests failed) : Dependant of REP-REP-141 <https://ci-stg.openmrs.org/browse/EMRAPI-EMRAPI-354> |
| 08:23:35 | <dkayiwa> Shangeetha: hi |
| 08:24:50 | <Shangeetha> dkayiwa: Hi |
| 08:25:33 | <Shangeetha> dkayiwa: We have opd, ipd flow running in OMRS 1.8.3 and we have radiology flow running in 1.9.3 |
| 08:26:02 | <Shangeetha> dkayiwa: We want to get it linked both flows. Can you suggest any idea? |
| 08:26:49 | <Shangeetha> dkayiwa: We thought of getting OMRS 1.9.3 webapp opening in a separate tab, when radiology tab is clicked in patient dashboard view of OMRS 1.8.3 |
| 08:27:56 | <dkayiwa> Shangeetha: why are you running different versions? |
| 08:29:35 | <Shangeetha> dkayiwa: Because we have opd, ipd flow working fine in 1.8.3 |
| 08:29:52 | <Shangeetha> dkayiwa: But our radiology module is only running in 1.9.3 |
| 08:30:15 | <Shangeetha> dkayiwa: One option is to downgrade radiology to 1.8.3 |
| 08:31:14 | <dkayiwa> Shangeetha: dont they both work on 1.9.3? |
| 08:31:31 | *** GitHub44 has joined #openmrs |
| 08:31:31 | <GitHub44> [openmrs-core] vinayvenu opened pull request #483: TRUNK-4156 - revised (master...TRUNK-4156) http://git.io/lnXbeA |
| 08:31:31 | *** GitHub44 has left #openmrs |
| 08:32:06 | <Shangeetha> dkayiwa: No because we have set up opd, ipd flow based on hispindia, it is compatible with 1.8.* |
| 08:32:52 | *** GitHub47 has joined #openmrs |
| 08:32:53 | <GitHub47> [openmrs-core] vinayvenu opened pull request #484: TRUNK-4156 - Add care_setting to orders table. (1.10.x...TRUNK-4156) http://git.io/0m3mDg |
| 08:32:53 | *** GitHub47 has left #openmrs |
| 08:32:55 | <dkayiwa> Shangeetha: have you already added the tab? |
| 08:33:48 | *** k-joseph has joined #openmrs |
| 08:34:44 | <Shangeetha> dkayiwa: tab of radiology is already there. We want to link the radiology module of OMRS 1.9.3 |
| 08:35:21 | <dkayiwa> Shangeetha: just add the link there |
| 08:36:09 | <Shangeetha> dkayiwa: Yeah we want to know how to add? Is it at the backend? |
| 08:36:30 | <dkayiwa> Shangeetha: link is front end |
| 08:36:36 | *** GitHub74 has joined #openmrs |
| 08:36:36 | <GitHub74> [openmrs-core] shruthidipali closed pull request #480: 1.10.x (1.10.x...1.10.x) http://git.io/eI8E7A |
| 08:36:36 | *** GitHub74 has left #openmrs |
| 08:37:05 | *** GitHub158 has joined #openmrs |
| 08:37:06 | <GitHub158> [openmrs-core] shruthidipali closed pull request #483: TRUNK-4156 - revised (master...TRUNK-4156) http://git.io/lnXbeA |
| 08:37:06 | *** GitHub158 has left #openmrs |
| 08:37:10 | <Shangeetha> dkayiwa: sorry i cannot understand how to add. |
| 08:38:29 | <dkayiwa> Shangeetha: show me a screenshot of how it should look like |
| 08:39:11 | *** GitHub36 has joined #openmrs |
| 08:39:12 | <GitHub36> [openmrs-core] shruthidipali closed pull request #484: TRUNK-4156 - Add care_setting to orders table. (1.10.x...TRUNK-4156) http://git.io/0m3mDg |
| 08:39:12 | *** GitHub36 has left #openmrs |
| 08:39:12 | *** GitHub109 has joined #openmrs |
| 08:39:12 | <GitHub109> [openmrs-core] shruthidipali pushed 2 new commits to 1.10.x: http://git.io/oW-zhg |
| 08:39:12 | <GitHub109> openmrs-core/1.10.x 2bc1af0 Vinay Venu: TRUNK-4156 - Add care_setting to orders table.... |
| 08:39:12 | <GitHub109> openmrs-core/1.10.x f70fbfa shruthidipali: Merge pull request #484 from vinayvenu/TRUNK-4156... |
| 08:39:12 | *** GitHub109 has left #openmrs |
| 08:39:12 | <Shangeetha> dkayiwa: Ok sure |
| 08:42:53 | <Shangeetha> dkayiwa: http://snag.gy/SeN9z.jpg Here u can see radiology record tab |
| 08:44:15 | <Shangeetha> dkayiwa: Once it is clicked it should direct to localhost:8080/slhmsradiology web app, which is OMRS 1.9.3 |
| 08:45:03 | <Shangeetha> dkayiwa: what time you will be back? We have lectures now :) |
| 08:45:22 | <dkayiwa> Shangeetha: do yuo have the module code |
| 08:46:10 | <Shangeetha> dkayiwa: yeah we have. Shall i get back to you after one or two hours? |
| 08:46:21 | <Shangeetha> dkayiwa: We have to go now because of lectures. Really sorry :) |
| 08:46:30 | <dkayiwa> Shangeetha: ok |
| 08:46:39 | *** Shangeetha has quit IRC |
| 08:50:26 | *** robbyoconnor has quit IRC |
| 08:54:14 | *** k-joseph has quit IRC |
| 08:54:45 | *** robbyoconnor has joined #openmrs |
| 08:54:53 | *** ChanServ sets mode: +v robbyoconnor |
| 08:55:39 | *** djazayeri has quit IRC |
| 08:56:05 | *** k-joseph has joined #openmrs |
| 09:08:49 | *** Saki has quit IRC |
| 09:48:52 | *** vinayv has quit IRC |
| 09:50:22 | *** vinayv has joined #openmrs |
| 10:07:52 | <k-joseph> dkayiwa: Good morning!!! |
| 10:08:40 | <dkayiwa> k-joseph: morning |
| 10:09:26 | <k-joseph> dkayiwa: i was wondering whether a changeset can have more than one <preConditions onFail="MARK_RAN">s |
| 10:09:29 | <k-joseph> !!!! |
| 10:09:30 | <OpenMRSBot> k-joseph: Error: "!!!" is not a valid command. |
| 10:14:17 | <dkayiwa> k-joseph: its can have a list of them |
| 10:18:10 | <dkayiwa> k-joseph: here is an example: https://github.com/openmrs/openmrs-core/commit/617152d551d91d30e813c01dd085ad47642bbca5 |
| 10:18:19 | <k-joseph> dkayiwa: i have tried it and it stopped openmrs from starting with an erro until i had to add the <not /> tags for each of <preconditions /> tags i wanted to add into one !!!! |
| 10:18:24 | *** harsz89 has joined #openmrs |
| 10:18:25 | *** ChanServ sets mode: +v harsz89 |
| 10:20:26 | <dkayiwa> k-joseph: how did you do it |
| 10:21:52 | <k-joseph> dkayiwa: i had at first included each <not /> tag forexample in its own <preconditions /> which failed but as as that in the page you have just given to me worked well |
| 10:22:11 | <dkayiwa> k-joseph: ok |
| 10:42:39 | *** Shangeetha has joined #openmrs |
| 10:42:47 | <Shangeetha> dkayiwa: HI |
| 10:43:07 | <dkayiwa> Shangeetha: hi |
| 10:43:43 | <Shangeetha> dkayiwa: Hope u can remember our requirement and hope u saw the screen shot i sent u earlier |
| 10:44:12 | <dkayiwa> Shangeetha: can you change the source code for that module? |
| 10:45:39 | <Shangeetha> dkayiwa: yeah we can. But we do not know how to change. We tried with editing config.xml and pom.xml but they did not work |
| 10:46:56 | <dkayiwa> Shangeetha: i think it makes more sense for you to make all modules work on one version such that you do not use two |
| 10:48:50 | <Shangeetha> dkayiwa: Okay . That means it is better to downgrade radiology to make it compatible with 1.8.* |
| 10:49:00 | *** GitHub136 has joined #openmrs |
| 10:49:01 | <GitHub136> [openmrs-core] k-joseph closed pull request #475: [#TRUNK-4159] Add OrderFrequency to the data model (1.10.x...TRUNK-4159) http://git.io/MH58BA |
| 10:49:01 | *** GitHub136 has left #openmrs |
| 10:49:01 | <dkayiwa> Shangeetha: ok |
| 10:49:47 | <Shangeetha> dkayiwa: We are currently using akhil's radiology module. Can you pls help us to downgrade it? |
| 10:51:21 | <dkayiwa> Shangeetha: what error do u get? |
| 10:53:42 | <Shangeetha> dkayiwa: Yeah currently i removed openmrs from our server. I will deploy it again and tell you. The error was related jsp portlet exception. Give me some time, once i set up server. |
| 10:56:20 | *** GitHub149 has joined #openmrs |
| 10:56:20 | <GitHub149> [openmrs-core] k-joseph opened pull request #485: [#TRUNK-4159] Add OrderFrequency to the data model (1.10.x...TRUNK-4159) http://git.io/o7U1Mg |
| 10:56:20 | *** GitHub149 has left #openmrs |
| 11:01:50 | *** GitHub90 has joined #openmrs |
| 11:01:51 | <GitHub90> [openmrs-core] dkayiwa closed pull request #471: Trunk 3646 (master...TRUNK-3646) http://git.io/OMiVQg |
| 11:01:51 | *** GitHub90 has left #openmrs |
| 11:01:51 | *** GitHub38 has joined #openmrs |
| 11:01:51 | <GitHub38> [openmrs-core] dkayiwa pushed 3 new commits to master: http://git.io/tPpiig |
| 11:01:51 | <GitHub38> openmrs-core/master 161bbb1 channab: TRUNK-3646:Add ability to select/unselect all privileges on Role Management page:kua |
| 11:01:51 | <GitHub38> openmrs-core/master 6060489 channab: TRUNK-3646:Add ability to select/unselect all privileges on Role Management page:kua |
| 11:01:51 | <GitHub38> openmrs-core/master 5a194f2 dkayiwa: Merge pull request #471 from channab/TRUNK-3646... |
| 11:01:51 | *** GitHub38 has left #openmrs |
| 11:12:05 | *** dkayiwa has quit IRC |
| 11:13:55 | *** Shangeetha has quit IRC |
| 11:14:12 | *** travis-ci has joined #openmrs |
| 11:14:12 | <travis-ci> [travis-ci] [openmrs/openmrs-core] [5a194f2] [dkayiwa] The build is still failing. - http://travis-ci.org/openmrs/openmrs-core/builds/15701256 |
| 11:14:12 | *** travis-ci has left #openmrs |
| 11:28:15 | *** banka has joined #openmrs |
| 11:28:20 | *** Shangeetha has joined #openmrs |
| 11:30:44 | *** Shangeetha has quit IRC |
| 11:46:26 | *** k-joseph has quit IRC |
| 11:51:35 | *** k-joseph has joined #openmrs |
| 11:57:24 | *** raonyguimaraes has joined #openmrs |
| 11:59:46 | *** raonyguimaraes has quit IRC |
| 12:01:03 | *** raonyguimaraes has joined #openmrs |
| 12:21:26 | *** raonyguimaraes has quit IRC |
| 12:38:41 | *** banka has quit IRC |
| 12:43:21 | *** GitHub34 has joined #openmrs |
| 12:43:21 | <GitHub34> [openmrs-core] k-joseph closed pull request #479: TRUNK-4126 Added duration and durationUnits properties to DrugOrder class (1.10.x...TRUNK-4126) http://git.io/s0uC0A |
| 12:43:21 | *** GitHub34 has left #openmrs |
| 12:47:51 | *** jpawelski_ has joined #openmrs |
| 12:52:37 | *** jpawelski_ has quit IRC |
| 12:52:40 | *** vinayv has quit IRC |
| 12:56:27 | *** GitHub102 has joined #openmrs |
| 12:56:28 | <GitHub102> [openmrs-core] k-joseph opened pull request #486: [#TRUNK-4126] Add duration and durationUnits properties to DrugOrder class (1.10.x...TRUNK-4126) http://git.io/ZLmm9A |
| 12:56:28 | *** GitHub102 has left #openmrs |
| 12:58:07 | *** vinayv has joined #openmrs |
| 13:17:57 | *** vinayv has quit IRC |
| 13:46:26 | *** k-joseph has quit IRC |
| 13:48:45 | *** robbyoconnor has quit IRC |
| 13:48:57 | *** Shangeetha has joined #openmrs |
| 13:49:11 | *** robbyoconnor has joined #openmrs |
| 13:49:11 | *** ChanServ sets mode: +v robbyoconnor |
| 13:49:54 | *** k-joseph has joined #openmrs |
| 13:51:27 | *** wluyima has joined #openmrs |
| 14:01:05 | *** dkayiwa has joined #openmrs |
| 14:06:05 | *** Shangeetha has quit IRC |
| 14:07:01 | *** Shangeetha has joined #openmrs |
| 14:07:20 | <Shangeetha> dkayiwa: Hi |
| 14:10:50 | <dkayiwa> Shangeetha: hi |
| 14:11:01 | *** GitHub56 has joined #openmrs |
| 14:11:02 | <GitHub56> [openmrs-core] k-joseph opened pull request #487: [#TRUNK-4169] Add quantity and units properties to DrugIngredient class (1.10.x...TRUNK-4169) http://git.io/WNDYvQ |
| 14:11:02 | *** GitHub56 has left #openmrs |
| 14:12:57 | <Shangeetha> dkayiwa: javax.servlet.jsp.JspException: javax.servlet.ServletException: File "/WEB-INF/view/module/radiology/portlets/RadiologyDashboardTab.jsp" not found |
| 14:13:23 | <Shangeetha> dkayiwa: This is the error we are getting when downgrading radiology module to 1.8.* from 1.9.* |
| 14:14:38 | <dkayiwa> Shangeetha: just a moment |
| 14:16:28 | <Shangeetha> dkayiwa: Ok sure |
| 14:29:44 | <wluyima> hi dj |
| 14:37:47 | <dkayiwa> Shangeetha: hi |
| 14:40:09 | <Shangeetha> dkayiwa: Hi |
| 14:40:34 | <dkayiwa> Shangeetha: which exact version of 1.8 are you using? |
| 14:40:38 | <Shangeetha> dkayiwa: http://snag.gy/pHtrG.jpg , we have pasted here |
| 14:40:52 | <Shangeetha> dkayiwa: Openmrs 1.8.3 |
| 14:41:44 | *** suranga__ has joined #openmrs |
| 14:41:56 | <dkayiwa> Shangeetha: so does 1.8.3 give you error? |
| 14:41:59 | <Shangeetha> dkayiwa: We want to make radiology module, available in https://github.com/akhilrv/Radiology compatible with 1.8.* |
| 14:42:27 | <Shangeetha> dkayiwa: Yeah 1.8.3 gave this error |
| 14:42:48 | <dkayiwa> Shangeetha: and which exact version of 1.9 did you try out? |
| 14:43:29 | <Shangeetha> dkayiwa: 1.9.3 but radiology works fine with any 1.9.* because we tried so many versions once in a while |
| 14:43:59 | <dkayiwa> Shangeetha: so why don't you just use 1.9? |
| 14:44:12 | <dkayiwa> Shangeetha: since it works well with it |
| 14:45:03 | <Shangeetha> dkayiwa: because other flows such as opd, ipd works well with 1.8.* |
| 14:45:18 | <Shangeetha> dkayiwa: We have deployed at slhms.uom.lk:8080/slhms |
| 14:45:42 | <Shangeetha> dkayiwa: where u can have a look at our deployment and customisation |
| 14:45:46 | <dkayiwa> Shangeetha: which exact version of 1.8 does opd, ipd, etc work with? |
| 14:46:18 | <Shangeetha> dkayiwa: 1.8.3 |
| 14:46:31 | <dkayiwa> Shangeetha: what happens when you run them in 1.9? |
| 14:48:13 | <Shangeetha> dkayiwa: Modules are not getting loaded properly and not giving tabs, required outputs etc. |
| 14:48:38 | <dkayiwa> Shangeetha: do they give any errors? |
| 14:50:44 | <Shangeetha> dkayiwa: yeah version incompatibility errors. One thing we can do is that we have to upgrade all modules to 1.9.* rather than downgrading radiology module to 1.8.* |
| 14:51:34 | <dkayiwa> Shangeetha: in the radiology module which fails on 1.8, does that portlet exist? |
| 14:51:43 | <Shangeetha> dkayiwa: Since we are running behind time and working under tight schedule, we thought of downgrading radiology module, which requires editing only one module |
| 14:52:45 | <dkayiwa> Shangeetha: in the radiology module which fails on 1.8, does that portlet exist? |
| 14:55:11 | <Shangeetha> dkayiwa: that portlet is not there. under /WEB-INF/view/module no such folder called, /radiology/portlets/RadiologyDashboardTab.jsp |
| 15:00:40 | *** vinayv has joined #openmrs |
| 15:06:09 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-UILIBRARY-88 has FAILED : Manual build by Rafał Korytkowski <https://ci-stg.openmrs.org/browse/SON-UILIBRARY-88> |
| 15:11:02 | *** robbyoconnor has quit IRC |
| 15:22:49 | <dkayiwa> Shangeetha: do you know who developed that module? |
| 15:25:54 | <Shangeetha> dkayiwa: Yeah akhil ravindran. |
| 15:26:05 | <dkayiwa> Shangeetha: did you tell him about that error? |
| 15:27:05 | <Shangeetha> dkayiwa: no because we got this module done for 1.9.*. Last week only we changed our decision to downgrade. So thought of getting help from here for downgrading |
| 15:40:12 | *** djazayeri has joined #openmrs |
| 15:40:12 | *** ChanServ sets mode: +o djazayeri |
| 15:51:18 | <Shangeetha> dkayiwa: It is ok. We will try getting contacted with the person who developed this radiology module. Thanks :) bye |
| 15:54:31 | <djazayeri> wluyima: hey, btw, have you (or someone) set up a hangout for 5 minutes from now? |
| 15:54:35 | <djazayeri> wluyima: for standup? |
| 15:55:07 | *** Shangeetha has quit IRC |
| 15:55:43 | <wluyima> djazayeri: i think cpower did because he included google+ urls in the google invites he sent out |
| 15:57:31 | <djazayeri> cpower: did you set that up? |
| 15:57:51 | <wluyima> djazayeri: do you see the where url in your email notification, that is the url |
| 15:58:34 | <wluyima> djazayeri: did you get the Order Entry BiWeekly Scrum email notification? |
| 15:58:43 | <wluyima> djazayeri: if yes, that has the url in it |
| 15:59:02 | <djazayeri> wluyima: I got your replies to it |
| 15:59:21 | <djazayeri> wluyima: apparently I got the invite but didn't know that |
| 15:59:29 | <wluyima> yeah |
| 15:59:45 | <djazayeri> wluyima: I think cpower set it up wrong though. We can't have a persistent hangout URL like he has in the description |
| 16:00:00 | <wluyima> djazayeri: why not? |
| 16:01:25 | <djazayeri> wluyima: because google doesn't let you reuse the same hangout repeatedly |
| 16:04:31 | <djazayeri> Order Entry standup is *now*: https://plus.google.com/hangouts/_/event/cdolj7fgikvlkhd30pfhna0jja0?authuser=0&hl=en |
| 16:04:35 | <OpenMRSBot> <http://ln-s.net/A5aH> (at plus.google.com) |
| 16:04:39 | <djazayeri> wluyima: ^^ |
| 16:10:52 | <dkayiwa> k-joseph: hi |
| 16:11:07 | <k-joseph> dkayiwa: hi |
| 16:12:51 | <k-joseph> dkayiwa: got it |
| 16:14:15 | <dkayiwa> k-joseph: you can turn off video to reduce band width usage |
| 16:14:35 | <k-joseph> dkayiwa: yeah, i have done that :) |
| 16:17:22 | <dkayiwa> k-joseph: you can eve turn off the videos of others to reduce it further :) |
| 16:20:58 | <k-joseph> dkayiwa: currently am not getting any video, it is only audio |
| 16:21:29 | <dkayiwa> k-joseph: so you do not even see others' videos. correct? |
| 16:21:53 | <k-joseph> dkayiwa: yeah, that is right :) |
| 16:22:02 | <dkayiwa> k-joseph: cool :) |
| 16:22:10 | <k-joseph> dkayiwa: :) |
| 16:22:14 | <dkayiwa> k-joseph: it too me long to figure that out :) |
| 16:40:36 | *** akwatuha__ has quit IRC |
| 16:41:07 | *** akwatuha___ has joined #openmrs |
| 16:46:13 | *** rfriedman has joined #openmrs |
| 16:46:24 | <rfriedman> mseaton, hi, you really there? |
| 16:51:05 | *** raonyguimaraes has joined #openmrs |
| 16:53:39 | *** robbyoconnor has joined #openmrs |
| 16:53:39 | *** ChanServ sets mode: +v robbyoconnor |
| 16:53:39 | *** k-joseph has quit IRC |
| 16:55:40 | *** vinayv has quit IRC |
| 16:57:09 | *** k-joseph has joined #openmrs |
| 16:59:05 | <jkeiper> dkayiwa: we have some questions for you :-) |
| 16:59:17 | <dkayiwa> jkeiper: shoot in the air :) |
| 17:01:08 | <jkeiper> dkayiwa: haha, i'll pm yiou |
| 17:01:09 | *** robbyoconnor has quit IRC |
| 17:04:31 | <djazayeri> wluyima: the dbms=mysql have to do with the way that saptarshi got things to work with postgres |
| 17:04:59 | <wluyima> djazayeri: hmm. so why did we stope requiring it |
| 17:05:21 | <djazayeri> wluyima: i.e. every changeset before a certain point (that isn't already in liquibase-schema) got dbms=mysql |
| 17:05:37 | <djazayeri> wluyima: still on a call now |
| 17:05:45 | <djazayeri> wluyima: search the dev list archive for this, and for saptarshi's description |
| 17:05:55 | <wluyima> djazayeri: ok will talk about it later, am still confused |
| 17:08:00 | <djazayeri> wluyima: look for an email thread with subject "dbms=mysql in liquibase-update-to-latest" that I started Oct 7 2011 |
| 17:08:49 | <mseaton> rfriedman: here i am |
| 17:09:00 | <rfriedman> hey mike, got a minute? |
| 17:09:28 | <rfriedman> mseaton, i have 2 report definitions I can see in the serialized_object table |
| 17:09:36 | *** wluyima has quit IRC |
| 17:09:45 | <rfriedman> mseaton, but getAllDefinitions only returns one of them |
| 17:10:02 | <rfriedman> mseaton, likewise, I can't find the one that isn't returned when I search by uuid |
| 17:10:20 | <rfriedman> mseaton, but I dont get any exceptions thrown, just a null return |
| 17:10:23 | <mseaton> rfriedman: doesn't sound right |
| 17:10:27 | <rfriedman> any ideas? |
| 17:10:47 | <mseaton> rfriedman: how did you get those definitions in there? are they old or new? |
| 17:10:56 | <rfriedman> mseaton, the only thing that appears to me offhand is that the missing one is of subtype BaseReportDefinition |
| 17:11:41 | <rfriedman> mseaton, I am trying to step thru it but all the proxying is very annoying |
| 17:11:59 | <mseaton> rfriedman: that would be the issue |
| 17:12:10 | <mseaton> rfriedman: i wonder why that happened... |
| 17:12:29 | <mseaton> rfriedman: if you manually change that in the DB to ReportDefinition, you'll see those methods work correctly |
| 17:12:58 | <mseaton> rfriedman: unfortunately db queries to fetch serialized objects of a given type don't understand java inheritance |
| 17:13:19 | <rfriedman> mseaton, ok, I will try that and let you know |
| 17:15:18 | <rfriedman> mseaton, does that mean when I call save I have to cast it to ReportDefinition? |
| 17:16:16 | <rfriedman> mseaton, or does the subclass list have to be updated? |
| 17:17:00 | *** k-joseph has quit IRC |
| 17:17:12 | <rfriedman> mseaton, no joy |
| 17:17:47 | <mseaton> rfriedman: what is the type and what is the subtype of that row in serialized object, exactly |
| 17:18:47 | <rfriedman> mseaton, org.openmrs.module.reporting.report.definition.ReportDefinition |
| 17:25:36 | <mseaton> rfriedman: ok maybe that was right all along. eg. type should = ReportDefinition and subtype should = BaseReportDefinition, but i'm not sure it matters. |
| 17:26:07 | <rfriedman> mseaton, ok, i can change it back, but of course it wasn't working |
| 17:26:18 | <mseaton> rfriedman: any other differences between the 2 rows? |
| 17:27:36 | <rfriedman> mseaton, the serialized object is much bigger in the missing one because the one that's found has saved components while the one that's not found is all internal to the report |
| 17:29:04 | <rfriedman> mseaton, the one not found has a null in description |
| 17:29:09 | <mseaton> rfriedman: looking at the code, specifically at org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImple, it appears that the code is set up to simply swallow serialization issues upon loading definitions, and return null if the data in the serialized_object table cannot be deserialized |
| 17:29:32 | <mseaton> rfriedman: i'd put some breakpoints in there and see what you can find out. |
| 17:30:09 | <mseaton> rfriedman: it should be logging warnings to the log however. do you not see anything in the log file? |
| 17:30:19 | <rfriedman> mseaton, stand by |
| 17:30:34 | <mseaton> rfriedman: grabbing a bite to eat. back in a sec |
| 17:31:37 | <rfriedman> mseaton, ok, am running in junit, should log to console, right? nothing there |
| 17:32:33 | <djazayeri> rfriedman: (in my experience running junit tests doesn't log the right stuff. but if you see _some_ log messages, I guess you would see these) |
| 17:35:11 | <rfriedman> mseaton, don't see nuthin |
| 17:35:21 | <rfriedman> mseaton, just sent you the serialized object |
| 17:37:42 | <rfriedman> mseaton, i better recreate it, it may not be the most recent |
| 17:39:16 | <djazayeri> orfriedman: can you put a breakpoint where Mike says ( org.openmrs.module.reporting.definition.service.SerializedDefinitionServiceImpl ) and Debug the junit test? that way you can see the serialization error |
| 17:41:49 | <rfriedman> djazayeri, yes |
| 17:42:10 | <djazayeri> rfriedman: that's my suggestion. (otherwise I have nothing useful to add) |
| 17:44:46 | <OpenMRSBot> Recent updates in the world of openmrsci: REPUI-REPUI-330 has FAILED (1 tests failed) : Updated by djazayeri <https://ci-stg.openmrs.org/browse/REPUI-REPUI-330> |
| 17:45:42 | <mseaton> rfriedman: got your mail. that's ugly. i'll wait to hear how the breakpoint goes. |
| 17:48:23 | <mseaton> rfriedman: you can also try going here and see what it has to say: |
| 17:48:32 | <mseaton> rfriedman: /module/reporting/definition/invalidSerializedDefinitions.form |
| 17:49:26 | *** GitHub0 has joined #openmrs |
| 17:49:27 | <GitHub0> [openmrs-core] dkayiwa closed pull request #468: [#TRUNK-4111] Exclude longitude, latitude, start date and end date fields from address template (master...TRUNK-4111) http://git.io/NzZH6Q |
| 17:49:27 | *** GitHub0 has left #openmrs |
| 17:49:27 | *** GitHub189 has joined #openmrs |
| 17:49:27 | <GitHub189> [openmrs-core] dkayiwa pushed 3 new commits to master: http://git.io/AnNcyA |
| 17:49:27 | <GitHub189> openmrs-core/master 309a389 k-joseph: TRUNK-4111 Excluded longitude, latitude, start date and end date fields from address template |
| 17:49:27 | <GitHub189> openmrs-core/master b661e72 k-joseph: TRUNK-4111 format fixed by running mvn clean install |
| 17:49:27 | <GitHub189> openmrs-core/master 66ce8cd dkayiwa: Merge pull request #468 from k-joseph/TRUNK-4111... |
| 17:49:27 | *** GitHub189 has left #openmrs |
| 17:54:24 | *** GitHub133 has joined #openmrs |
| 17:54:24 | <GitHub133> [openmrs-core] dkayiwa closed pull request #467: TRUNK-3646:Add ability to select/unselect all privileges on Role Managem... (master...master) http://git.io/trUPXA |
| 17:54:24 | *** GitHub133 has left #openmrs |
| 17:56:52 | <rfriedman> mseaton, i have a breakpoint set in the DAO as well, but I don't see the code, just the proxy with a message about code not found |
| 18:05:28 | <mseaton> rfriedman: you'll likely need to download sources and attach them appropriately for the debugger in your IDE |
| 18:05:50 | <mseaton> rfriedman: did you go to the link i pasted above? |
| 18:09:09 | <rfriedman> mseaton, I am getting a null pointer exception turning into a converter exception at ReferenceByIdUnmarshaller line 86 |
| 18:10:01 | <mseaton> rfriedman: when was this definition saved? is this something you saved with the current version of the module you are using? or has it been around in your db for a while and was created with an earlier version? |
| 18:10:22 | *** travis-ci has joined #openmrs |
| 18:10:22 | <travis-ci> [travis-ci] [openmrs/openmrs-core] [66ce8cd] [dkayiwa] The build was fixed. - http://travis-ci.org/openmrs/openmrs-core/builds/15722758 |
| 18:10:22 | *** travis-ci has left #openmrs |
| 18:10:29 | <rfriedman> mseaton, no, I created it Mon or Tue |
| 18:10:51 | <mseaton> rfriedman: do you have the latest serialization.xstream and reporting both? |
| 18:11:37 | <rfriedman> mseaton, prob not |
| 18:12:37 | <mseaton> rfriedman: worth checking on those versions. |
| 18:14:16 | <rfriedman> mseaton, will do ... any changes in this part of reporting that you know of? |
| 18:18:47 | <mseaton> rfriedman: i recall perhaps an issue with serialization for some of these indicator data sets that used inner classes, but not sure at all if this is your issue. and this was a while ago. |
| 18:19:22 | <rfriedman> mseaton, could be, i'm definitely into inner classes |
| 18:31:25 | *** GitHub37 has joined #openmrs |
| 18:31:25 | <GitHub37> [openmrs-core] dkayiwa closed pull request #466: Enhancing SourceMySqldiffFile to be used from command line - TRUNK-4186 (1.9.x...sqldiff) http://git.io/uL8s8A |
| 18:31:25 | *** GitHub37 has left #openmrs |
| 18:31:25 | *** GitHub99 has joined #openmrs |
| 18:31:25 | <GitHub99> [openmrs-core] dkayiwa pushed 2 new commits to 1.9.x: http://git.io/BSeH4g |
| 18:31:25 | <GitHub99> openmrs-core/1.9.x 9e6473c Shruthi: Angshu/Shruthi |
| 18:31:25 | <GitHub99> openmrs-core/1.9.x ea92dd0 dkayiwa: Merge pull request #466 from shruthidipali/sqldiff... |
| 18:31:25 | *** GitHub99 has left #openmrs |
| 18:34:18 | *** Ch3ck has joined #openmrs |
| 18:37:35 | *** k-joseph has joined #openmrs |
| 18:51:43 | *** GitHub30 has joined #openmrs |
| 18:51:43 | <GitHub30> [openmrs-core] dkayiwa pushed 1 new commit to master: http://git.io/Qrdmug |
| 18:51:43 | <GitHub30> openmrs-core/master 0d981ac dkayiwa: Forward porting to 1.11.x: Enhancing SourceMySqldiffFile to be used from... |
| 18:51:43 | *** GitHub30 has left #openmrs |
| 18:55:30 | *** wluyima has joined #openmrs |
| 18:57:13 | *** GitHub2 has joined #openmrs |
| 18:57:13 | <GitHub2> [openmrs-core] dkayiwa pushed 1 new commit to 1.10.x: http://git.io/gatYgw |
| 18:57:13 | <GitHub2> openmrs-core/1.10.x 3767864 dkayiwa: Forward porting to 1.11.x: Enhancing SourceMySqldiffFile to be used from... |
| 18:57:13 | *** GitHub2 has left #openmrs |
| 19:03:19 | *** Saki has joined #openmrs |
| 19:07:14 | *** GitHub62 has joined #openmrs |
| 19:07:15 | <GitHub62> [openmrs-core] dkayiwa closed pull request #463: TRUNK-4161: Fixing liquibase checksum error due to changes in TRUNK-4012 (1.9.x...TRUNK4161) http://git.io/gO4vJw |
| 19:07:15 | *** GitHub62 has left #openmrs |
| 19:13:54 | *** travis-ci has joined #openmrs |
| 19:13:54 | <travis-ci> [travis-ci] [openmrs/openmrs-core] [0d981ac] [dkayiwa] The build passed. - http://travis-ci.org/openmrs/openmrs-core/builds/15725874 |
| 19:13:54 | *** travis-ci has left #openmrs |
| 19:19:10 | <OpenMRSBot> Recent updates in the world of openmrsci: CA-CA-549 has FAILED (5 tests failed) : Dependant of TRUNK-LATEST-119 <https://ci-stg.openmrs.org/browse/CA-CA-549> |
| 19:35:01 | *** Saki has quit IRC |
| 19:39:10 | *** dkayiwa has quit IRC |
| 19:49:20 | *** Mkop has quit IRC |
| 19:56:38 | *** raonyguimaraes has quit IRC |
| 19:56:48 | <OpenMRSBot> Recent updates in the world of openmrsci: RDD-RDD-256 has FAILED (3 tests failed) : Dependant of TRUNK-LATEST-119 <https://ci-stg.openmrs.org/browse/RDD-RDD-256> |
| 20:01:20 | *** harsz89 has quit IRC |
| 20:17:23 | *** harsz89 has joined #openmrs |
| 20:17:23 | *** ChanServ sets mode: +v harsz89 |
| 20:23:46 | *** djazayeri has quit IRC |
| 20:50:45 | *** k-joseph has quit IRC |
| 20:57:36 | <OpenMRSBot> Recent updates in the world of openmrsci: TRUNK-MASTER-232 has FAILED (233 tests failed) : Updated by dkayiwa <https://ci-stg.openmrs.org/browse/TRUNK-MASTER-232> || TRUNK-ONEPOINTTENX-22 has FAILED (1 tests failed) : Updated by dkayiwa <https://ci-stg.openmrs.org/browse/TRUNK-ONEPOINTTENX-22> |
| 20:58:40 | *** GitHub131 has joined #openmrs |
| 20:58:41 | <GitHub131> [openmrs-core] wluyima closed pull request #485: [#TRUNK-4159] Add OrderFrequency to the data model (1.10.x...TRUNK-4159) http://git.io/o7U1Mg |
| 20:58:41 | *** GitHub131 has left #openmrs |
| 20:58:41 | *** GitHub133 has joined #openmrs |
| 20:58:41 | <GitHub133> [openmrs-core] wluyima pushed 2 new commits to 1.10.x: http://git.io/WeL3tw |
| 20:58:41 | <GitHub133> openmrs-core/1.10.x fad9d24 k-joseph: TRUNK-4159 seperated the three foreign keys and added them to order_frequency's changeset |
| 20:58:41 | <GitHub133> openmrs-core/1.10.x 657d819 wluyima: Merge pull request #485 from k-joseph/TRUNK-4159... |
| 20:58:41 | *** GitHub133 has left #openmrs |
| 21:01:08 | *** rfriedman has quit IRC |
| 21:01:13 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-METASHARING-85 has FAILED : Scheduled build <https://ci-stg.openmrs.org/browse/SON-METASHARING-85> |
| 21:10:00 | *** GitHub89 has joined #openmrs |
| 21:10:00 | <GitHub89> [openmrs-core] wluyima pushed 1 new commit to 1.10.x: http://git.io/Owv-_g |
| 21:10:00 | <GitHub89> openmrs-core/1.10.x 9d08da2 wyclif: Removed the num_refills and administration_instructions columns added via the liquibase schema only file |
| 21:10:00 | *** GitHub89 has left #openmrs |
| 21:16:06 | *** GitHub108 has joined #openmrs |
| 21:16:06 | <GitHub108> [openmrs-core] wluyima pushed 1 new commit to 1.10.x: http://git.io/uHwJYw |
| 21:16:06 | <GitHub108> openmrs-core/1.10.x 485e01a k-joseph: [#TRUNK-4169] Add added quantity and units properties to DrugIngredient |
| 21:16:06 | *** GitHub108 has left #openmrs |
| 21:16:36 | *** GitHub123 has joined #openmrs |
| 21:16:36 | <GitHub123> [openmrs-core] wluyima closed pull request #487: [#TRUNK-4169] Add quantity and units properties to DrugIngredient class (1.10.x...TRUNK-4169) http://git.io/WNDYvQ |
| 21:16:36 | *** GitHub123 has left #openmrs |
| 21:29:45 | <OpenMRSBot> Recent updates in the world of openmrsci: META-META-36 has FAILED (3 tests failed) : Dependant of METAM-METAM-20 <https://ci-stg.openmrs.org/browse/META-META-36> |
| 21:30:02 | *** GitHub128 has joined #openmrs |
| 21:30:02 | <GitHub128> [openmrs-core] wluyima pushed 1 new commit to 1.10.x: http://git.io/eCK6Gw |
| 21:30:02 | <GitHub128> openmrs-core/1.10.x a2e4eb1 k-joseph: TRUNK-4126 Added duration and durationUnits properties to DrugOrder class |
| 21:30:02 | *** GitHub128 has left #openmrs |
| 21:30:51 | *** GitHub36 has joined #openmrs |
| 21:30:51 | <GitHub36> [openmrs-core] wluyima closed pull request #486: [#TRUNK-4126] Add duration and durationUnits properties to DrugOrder class (1.10.x...TRUNK-4126) http://git.io/ZLmm9A |
| 21:30:51 | *** GitHub36 has left #openmrs |
| 21:47:48 | *** robbyoconnor has joined #openmrs |
| 21:47:48 | *** ChanServ sets mode: +v robbyoconnor |
| 22:02:12 | *** robbyoconnor has quit IRC |
| 22:12:23 | *** GitHub50 has joined #openmrs |
| 22:12:23 | <GitHub50> [openmrs-core] wluyima pushed 1 new commit to 1.10.x: http://git.io/OPHpUA |
| 22:12:23 | <GitHub50> openmrs-core/1.10.x 2e3c7cf Rafal Korytkowski: TRUNK-4129 Modify datatype of drug_order.quantity column and add... |
| 22:12:23 | *** GitHub50 has left #openmrs |
| 22:14:51 | *** GitHub78 has joined #openmrs |
| 22:14:51 | <GitHub78> [openmrs-core] wluyima pushed 1 new commit to 1.10.x: http://git.io/MIeM_g |
| 22:14:52 | <GitHub78> openmrs-core/1.10.x 9c31bbf wyclif: Removed changeset that sets quantity units for existing drug order quantities |
| 22:14:52 | *** GitHub78 has left #openmrs |
| 22:23:19 | *** GitHub82 has joined #openmrs |
| 22:23:19 | <GitHub82> [openmrs-core] wluyima closed pull request #474: TRUNK-4129 Modify datatype of drug_order.quantity column and add quantity_units column to drug_order (1.10.x...TRUNK-4129) http://git.io/Sy-c-g |
| 22:23:19 | *** GitHub82 has left #openmrs |
| 22:29:56 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-PROVIDERMANAGE-87 has FAILED (255 tests failed) : Scheduled build <https://ci-stg.openmrs.org/browse/SON-PROVIDERMANAGE-87> || REPUI-REPUI-333 has FAILED (1 tests failed) : Dependant of APPUI-APPUI-239 <https://ci-stg.openmrs.org/browse/REPUI-REPUI-333> |
| 22:37:46 | *** Ch3ck has quit IRC |
| 22:49:59 | <OpenMRSBot> Recent updates in the world of openmrsci: RDD-RDD-258 has FAILED (3 tests failed) : Dependant of PMM-PMM-47 <https://ci-stg.openmrs.org/browse/RDD-RDD-258> |
| 22:54:12 | *** wluyima has quit IRC |
| 23:29:00 | *** wluyima has joined #openmrs |
| 23:54:22 | <OpenMRSBot> Recent updates in the world of openmrsci: SON-REFAPP-93 has FAILED (3 tests failed) : Scheduled build <https://ci-stg.openmrs.org/browse/SON-REFAPP-93> |
| 23:56:16 | <OpenMRSBot> Recent updates in the world of openmrsci: RDD-RDD-259 has FAILED (3 tests failed) : Dependant of EMRAPI-EMRAPI-356 <https://ci-stg.openmrs.org/browse/RDD-RDD-259> |
