IRC Chat : 2008-12-11 - OpenMRS

00:34:18 *** bmckown has quit IRC
00:43:03 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1148 (defect closed): Leading zeroes should be ignored in patient identifiers <http://dev.openmrs.org/ticket/1148#comment:3> || OpenMRS Changesets: Changeset [6246]: Fixed patient search search with padded identifiers. ticket #1148 <http://dev.openmrs.org/changeset/6246>
01:33:00 *** jacobb has quit IRC
01:38:25 *** jacobb has joined #openmrs
01:45:25 *** [m_harrison] has joined #openmrs
01:57:52 *** kapten_yuba has joined #openmrs
01:58:09 *** kapten_yuba has left #openmrs
02:08:53 *** pygi has quit IRC
02:36:20 *** pygi has joined #openmrs
02:46:55 *** Mario__ has joined #openmrs
02:47:34 *** pygi has quit IRC
02:55:55 *** Mario__ is now known as pygi
03:04:53 *** Mario__ has joined #openmrs
03:05:28 *** pygi has quit IRC
03:05:32 *** Mario__ is now known as pygi
03:50:17 *** pygi has quit IRC
04:33:36 *** pygi has joined #openmrs
04:40:27 *** pygi has quit IRC
06:48:38 *** njero has quit IRC
10:41:28 *** pygi has joined #openmrs
11:37:19 *** pygi has quit IRC
11:50:37 *** Keelhaul has joined #openmrs
12:53:11 *** james_regen has joined #openmrs
12:53:11 *** ChanServ sets mode: +v james_regen
13:05:23 *** bwolfe has joined #openmrs
13:05:24 *** ChanServ sets mode: +o bwolfe
13:54:15 <Keelhaul> hi bwolfe
13:55:02 <bwolfe> hi Keelhaul
13:55:37 <Keelhaul> bwolfe: it would be nice if a concept could support multiple units, along with normal/critical ranges for each
13:55:48 <Keelhaul> and maybe also different ranges for men and women
13:56:12 <Keelhaul> btw, i ended up flushing my db and re-creating =/
13:56:14 <bwolfe> Keelhaul: different units on the same concept can't be done
13:56:15 <Keelhaul> couldnt find the error
13:56:31 <Keelhaul> why not
13:56:37 <bwolfe> because the numbers are completely different that get stored
13:56:46 <bwolfe> a different unit is a whole different kind of question
13:56:52 <Keelhaul> hmm
13:56:54 <bwolfe> "how many miles did you walk to get here"
13:56:55 <Keelhaul> depends
13:56:59 <Keelhaul> hm
13:57:00 <Keelhaul> yea
13:57:04 <bwolfe> "how many kilometers did you wakl to get here"
13:57:20 <bwolfe> but obs.value_numeric only stores an integer
13:57:26 <Keelhaul> but what if you ask "how far did you walk to get here (km)?"
13:57:44 <bwolfe> the user needs to enter in kilometers
13:57:45 <Keelhaul> yea, it would also need a value_unit or smt
13:58:16 <bwolfe> and in my eyes, thats not even close to worth it :-)
13:58:34 <bwolfe> because then you just put a larger demand on the reporting engine
13:58:41 <Keelhaul> true
13:58:43 <bwolfe> when its very easily solved by having two different concepts
13:58:56 <Keelhaul> but it could be made cross-unit with proper conversions
13:58:58 <Keelhaul> but yea, lots of work
13:59:21 <bwolfe> yes, the data entry screen could have boxes for miles, kilometers, inches, etc
13:59:22 <Keelhaul> what about different ranges for genders
13:59:27 <bwolfe> and do the conversion to km to be stored
13:59:38 <bwolfe> thats an interesting one
14:00:05 <bwolfe> I'm not sure how people are doing that
14:00:17 <bwolfe> surely they are not creating two different concepts...
14:00:17 <Keelhaul> i've been entering value ranges for some concepts, looking up the typical values on wiki
14:00:24 <Keelhaul> and some have different numbers for men and women
14:01:05 <Keelhaul> does anyone even use the ranges
14:01:38 <bwolfe> we only use a few ranges...mainly for ones that come off of machines
14:01:51 <bwolfe> Keelhaul: send that out to the impl list and see what people say
14:02:10 <Keelhaul> oh hm
14:02:14 <Keelhaul> i dont wanna register there =P
14:02:18 <Keelhaul> already enough mail to read every day
14:02:18 <bwolfe> add in alternate ranges for men/women wouldn't be /extremely/ difficult, but it would be nice to know if there are alternatives
14:02:26 <bwolfe> Keelhaul: filters...
14:02:31 <bwolfe> Keelhaul: its all about mail filters
14:02:48 <bwolfe> and the impl list gets less mail than the dev list anyway
14:02:57 <Keelhaul> yea i noticed form the feed
14:03:07 <Keelhaul> from*
14:03:10 <Keelhaul> btw
14:03:17 <Keelhaul> is there an automated way to calculate BMI
14:03:22 <Keelhaul> or is that a manual concept
14:03:49 <Keelhaul> i have a patient search for obese patients, which uses bmi
14:03:58 <Keelhaul> and i was wondering whether anyone even uses that concept
14:04:07 <Keelhaul> or just enters weight and height
14:04:22 <bwolfe> Keelhaul: I think people usually use weight/height
14:04:32 <bwolfe> darius wrote a helper function for it
14:04:35 <Keelhaul> the dashboard does the calculation i think
14:04:42 <bwolfe> I think its displayed on the dashboard using his function
14:04:44 <Keelhaul> but there's no way to do it in the reports
14:04:46 <bwolfe> yeah
14:04:47 <Keelhaul> so i have to use bmi
14:04:48 <bwolfe> hmm
14:06:14 <Keelhaul> what's more common, outside the US, mg/dl or mmol?
14:07:03 <bwolfe> Keelhaul: I don't know :-/
14:07:36 <Keelhaul> how about in the US
14:07:51 <Keelhaul> i tohught mg/dl sounds metrical but it seems the US use it
14:13:43 <bwolfe> Keelhaul: I don't know on that one either
14:15:28 <Keelhaul> =/
14:52:40 *** Keelhaul has quit IRC
14:54:07 *** Keelhaul has joined #openmrs
15:19:59 *** bwolfe has quit IRC
15:20:00 *** njero has joined #openmrs
15:20:00 *** ChanServ sets mode: +v njero
15:30:03 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6247]: formentry 1.4.x branch: Fixed taskpane to include concept-name-id when … <http://dev.openmrs.org/changeset/6247>
16:00:41 <Keelhaul> java.lang.OutOfMemoryError
16:00:42 <Keelhaul> ffs
16:00:46 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6249]: -- chica * added new java rules to support ETS JIT * updated chica1 obs … <http://dev.openmrs.org/changeset/6249> || OpenMRS Changesets: Changeset [6248]: formentry module trunk: ported fixes for #1165 from 1.4.x branch <http://dev.openmrs.org/changeset/6248>
16:13:59 *** Keelhaul has quit IRC
16:33:43 *** bwolfe has joined #openmrs
16:33:43 *** ChanServ sets mode: +o bwolfe
18:27:46 *** pygi has joined #openmrs
18:32:41 *** pygi has quit IRC
18:34:04 *** pygi has joined #openmrs
18:39:49 *** Mario__ has joined #openmrs
18:40:07 *** pygi has quit IRC
18:40:09 *** Mario__ is now known as pygi
19:10:15 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6251]: visitregister: Initial import containing skeleton code <http://dev.openmrs.org/changeset/6251> || OpenMRS Changesets: Changeset [6250]: visitregister module: Initial import containing skeleton code. <http://dev.openmrs.org/changeset/6250>
19:16:55 *** pygi has quit IRC
19:21:53 *** pygi has joined #openmrs
19:22:09 *** Mario__ has joined #openmrs
19:22:24 *** pygi has quit IRC
19:22:32 *** Mario__ is now known as pygi
19:22:50 *** Mario__ has joined #openmrs
19:29:26 *** Mario__ has quit IRC
20:14:18 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6252]: -- chica * changed mlms to reference a single CREATE_JIT rule <http://dev.openmrs.org/changeset/6252>
20:44:05 *** nribeka has joined #openmrs
20:44:20 <nribeka> exams is over ....
20:44:22 <nribeka> wooohooo :D
20:44:24 <OpenMRSBot> Recent updates in the world of openmrs: Justin Miranda: On user feedback <http://blog.justinmiranda.com/2008/12/on-user-feedback.html>
21:09:57 *** bwolfe_ has joined #openmrs
21:09:57 *** ChanServ sets mode: +o bwolfe_
21:16:21 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6253]: Applying patch #1133 author: efoxepstein <http://dev.openmrs.org/changeset/6253>
21:22:38 *** james_regen has quit IRC
21:24:37 *** bwolfe has quit IRC
21:32:35 *** njero has quit IRC
21:42:56 <nribeka> bwolfe_, i have a question
21:42:57 <nribeka> :D
21:43:03 *** nribeka has left #openmrs
21:44:16 *** bwolfe_ is now known as bwolfe
21:45:13 *** nribeka has joined #openmrs
21:45:37 <nribeka> is it possible for a patient to become a user, bwolfe?
21:45:51 <bwolfe> nribeka: yes, technically it is
21:45:58 <bwolfe> nribeka: but it throws errors right now
21:46:07 <bwolfe> due to a bug with the way we're doing users, etc
21:46:26 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Changesets: Changeset [6254]: bugfix: check if any subimage has an annotation; if so, do not show in … <http://dev.openmrs.org/changeset/6254>
21:54:07 *** nribeka has quit IRC
21:54:26 *** nribeka has joined #openmrs
21:55:33 <nribeka> so with the current release, there will be no chance a user is replaced by patient right bwolfe?
21:55:50 <nribeka> i mean it won't go through to the database
21:56:04 <bwolfe> nribeka: it can happen
21:56:14 <nribeka> it can?
21:56:21 <nribeka> so, if we got this exception:
21:56:23 <nribeka> [2008-12-07 12:37:05]IllegalArgumentException in class: org.openmrs.Person, setter method of property: personChangedBy
21:56:23 <nribeka> [2008-12-07 12:37:05]expected type: org.openmrs.User, actual value: org.openmrs.Patient
21:56:28 <bwolfe> but the problem is that hibernate throws an error if both a patient and user are the same person_id on the same object
21:56:43 <bwolfe> yes, that is the type of error
21:57:11 *** nribeka1 has joined #openmrs
21:58:19 <nribeka1> hmm ic ic
21:59:05 <nribeka> it is caused by that?
21:59:05 *** nribeka has quit IRC
21:59:39 <bwolfe> yeah, so the problem is that hibernate tries to be smart
21:59:58 <nribeka1> tries to be smart bwolfe?
22:00:43 <bwolfe> it sees Users and Patients that have the same person_id it will give you the object that was loaded first
22:01:34 <nribeka1> is there any workaround for it bwolfe?
22:01:49 <bwolfe> nribeka1: for the saving, or the loading problem?
22:01:57 <nribeka1> loading
22:02:20 <nribeka1> when james arbaugh tries to extract his data he got this message
22:11:50 <bwolfe> nribeka1: hmm, loading is tough
22:13:02 <nribeka1> :(
22:14:06 <nribeka1> so, there's no way to avoid it?
22:16:43 <bwolfe> nribeka1: yeah, for now you have to make different users :-/
22:16:45 <bwolfe> or different patients
22:17:01 <bwolfe> or just change your creator to be someone else
22:17:53 <nribeka1> wow, that's not an option i guess bwolfe :P
22:18:06 <nribeka1> it's a production data
22:18:18 <bwolfe> nribeka1: well, thats the only one I know of :-/
22:18:25 <nribeka1> ups ...
22:18:26 <nribeka1> :(
22:18:36 <bwolfe> nribeka1: he probably didn't get the data in there via openmrs, so I'm guessing it was loaded in by him manually
22:18:41 <bwolfe> so he could change it :-p
22:18:45 <nribeka1> hahahaha
22:18:48 <nribeka1> ah ic ic
22:19:01 <nribeka1> i will copy and paste this explanation then bwolfe
22:19:12 <nribeka1> and i will explain to james egg too :D
22:19:17 <nribeka1> or you can bwolfe
22:19:21 <bwolfe> hehe
22:19:26 <nribeka1> you guys are just a few cubicle away
22:19:28 <nribeka1> hahaha ...
22:19:29 <bwolfe> nribeka1: I need to send an email about this
22:19:36 <bwolfe> an email to the dev list
22:19:42 <bwolfe> but james is gone for the day I think
22:20:33 <nribeka1> ah ic ic
22:20:34 <nribeka1> ok ok
22:20:42 <nribeka1> i will be looking forward to read the email :D
22:21:28 <nribeka1> is it an active bug?
22:21:35 <bwolfe> nribeka1: you should still send them email
22:21:43 <bwolfe> yes, its in trac as a bug already I think
22:22:22 <nribeka1> ok ok. i will also refer to your email :)
22:35:21 *** njero has joined #openmrs
22:35:50 *** ChanServ sets mode: +v njero
22:46:51 *** bwolfe has quit IRC
22:46:56 <OpenMRSBot> Recent updates in the world of openmrs: OpenMRS Tickets: Ticket #1133 (task closed): Extension Point in <http://dev.openmrs.org/ticket/1133>