# Design of RadiologyReport object semantics

**URL:** <https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032>\
**Category:** Development\
**Tags:** radiologydcm4chee, radiology\
**Created:** [March 2, 2016, 10:58am UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032 "2016-03-02T10:58:32Z")\
**Posts on this page:** 20\
**Page:** 1

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 2, 2016, 10:58am UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/1 "2016-03-02T10:58:32Z")

</div>

Hi @judy,

just wanted to get your opinion on the RadiologyReport object we are creating in the radiology module.

How would you call the provider writing the report?

- personally I would go with **author**

- **principalResultsInterpreter** as done by [https://github.com/openmrs/openmrs-module-radiologyapp/blob/master/api/src/main/java/org/openmrs/module/radiologyapp/RadiologyReport.java#L30](https://github.com/openmrs/openmrs-module-radiologyapp/blob/master/api/src/main/java/org/openmrs/module/radiologyapp/RadiologyReport.java#L30)

- other suggestion?

And since we start with a simple free text report we have a string holding the actual report.

Should we simple call it **reportBody** like:

> <https://github.com/openmrs/openmrs-module-radiologyapp/blob/master/api/src/main/java/org/openmrs/module/radiologyapp/RadiologyReport.java#L40>

sounds right to me.

---

<div class="post-metadata">

**Author:** ![judy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/judy/32/2072_2.png) [@judy](https://talk.openmrs.org/u/judy)\
**Post date:** [March 3, 2016, 4:58am UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/2 "2016-03-03T04:58:13Z")

</div>

There are two levels of report writers

1. The person that may write it on paper and someone else is typing the report in the system (Transcribed by / Transcriptionist)

2. The person who interprets the study - so maybe **principalResultsInterpreter** is more close to the description. I am also ok with the author

Any preference from @mogoodrich or @mseaton

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 3, 2016, 3:33pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/3 "2016-03-03T15:33:06Z")

</div>

Principal Results Interpreter comes from the HL7 field that transports this value, so I figured it was a typical medical term for specifically noting the “author” of a report. I don’t a huge preference of that over author.

Bigger question though–would it make sense to try to migrate RadiologyReport (and other domain objects) from RadiologyApp module into the Radiology module, rather than have two separate “Report” modelling?

Take care, Mark

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 3, 2016, 4:30pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/4 "2016-03-03T16:30:59Z")

</div>

The thing is our object is modeled a bit different.

PR is pending, since functionality is not yet totally bug free, you can see the RadiologyReport as it is at the moment here

[https://github.com/thomas-burko/openmrs-module-radiologydcm4chee/blob/RAD-116/api/src/main/java/org/openmrs/module/radiology/report/RadiologyReport.java](https://github.com/thomas-burko/openmrs-module-radiologydcm4chee/blob/RAD-116/api/src/main/java/org/openmrs/module/radiology/report/RadiologyReport.java)

difference is we have a report status. So a user can claim reporting an order once the order is complete and has a study with images. This ensures no one else will report the same study. The report is than in status claimed and the user can edit as long as he wants, like a draft. Once the user is happy with it he puts the report into status complete.

We also dont have any concept members. The imaging procedure is stored with the RadiologyOrder as concept and we dont have a report type so far.

I dont think we can simply migrate RadiologyReport and your other domain objects with the features we (@judy) have currently in mind. Not sure what the best way would be to go about this.

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 3, 2016, 4:38pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/5 "2016-03-03T16:38:58Z")

</div>

@mogoodrich I think we would need to find a common ground for the functionality we want the module to have and see what of the existing domain objects, services from the radiology app should go into the radiology module and what changes would need to be done in the radiology app.

If you would like we could create a radiology task force/design group? 🙂

currently we have a lot of helping hands who want to get started, so we would need to decide soon on how to proceed. otherwise I guess the modules will diverge. @judy wants to deploy it in Malawi in June which is soon and we havent released the module yet since it still lacks stability and features.

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 4, 2016, 2:59pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/6 "2016-03-04T14:59:11Z")

</div>

@teleivo if you are up for it, I would take part in a working group!

---

<div class="post-metadata">

**Author:** ![judy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/judy/32/2072_2.png) [@judy](https://talk.openmrs.org/u/judy)\
**Post date:** [March 4, 2016, 3:08pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/7 "2016-03-04T15:08:15Z")

</div>

Do you men form a radiology working group ?

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 4, 2016, 3:29pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/8 "2016-03-04T15:29:32Z")

</div>

Yes, radiology design group, as @teleivo mentions above…

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 4, 2016, 5:25pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/9 "2016-03-04T17:25:24Z")

</div>

> [@mogoodrich](#):
>
> if you are up for it, I would take part in a working group!

yes I am in, great! @judy what about you? @sunbiz and @mseaton are you interested to join as well?

I am traveling until 12th March, we could schedule our first meeting sometime after that. any preferences?

would you like to make those meetings every two weeks? or should we just hold our first meeting and discuss this then.

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 4, 2016, 9:29pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/10 "2016-03-04T21:29:29Z")

</div>

Let’s just start with one meeting… the less meeting, the more we can do online the better… 🙂

---

<div class="post-metadata">

**Author:** ![judy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/judy/32/2072_2.png) [@judy](https://talk.openmrs.org/u/judy)\
**Post date:** [March 5, 2016, 3:56am UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/11 "2016-03-05T03:56:14Z")

</div>

Do we want to schedule an open design call fur one of the available slots ?

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 7, 2016, 2:19pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/12 "2016-03-07T14:19:29Z")

</div>

That makes sense to me @judy…

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 7, 2016, 7:20pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/13 "2016-03-07T19:20:05Z")

</div>

would the [https://wiki.openmrs.org/display/RES/2016-03-16+Design+Forum](https://wiki.openmrs.org/display/RES/2016-03-16+Design+Forum) work for you all? if so I would apply for the slot

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 7, 2016, 7:35pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/14 "2016-03-07T19:35:29Z")

</div>

Works for me, thanks!

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 7, 2016, 7:53pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/15 "2016-03-07T19:53:09Z")

</div>

@judy does the [https://wiki.openmrs.org/display/RES/2016-03-16+Design+Forum](https://wiki.openmrs.org/display/RES/2016-03-16+Design+Forum) work for you as well ? and @sunbiz, @mseaton are you interested too?

---

<div class="post-metadata">

**Author:** ![judy](https://talk.openmrs.org/user_avatar/talk.openmrs.org/judy/32/2072_2.png) [@judy](https://talk.openmrs.org/u/judy)\
**Post date:** [March 8, 2016, 12:24am UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/16 "2016-03-08T00:24:54Z")

</div>

yes it does

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 8, 2016, 5:58am UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/17 "2016-03-08T05:58:21Z")

</div>

dear @jthomas, can you please give the [https://wiki.openmrs.org/display/RES/2016-03-16+Design+Forum](https://wiki.openmrs.org/display/RES/2016-03-16+Design+Forum) slot to the radiology module.

---

<div class="post-metadata">

**Author:** ![jthomas](https://talk.openmrs.org/user_avatar/talk.openmrs.org/jthomas/32/21_2.png) [@jthomas](https://talk.openmrs.org/u/jthomas)\
**Post date:** [March 8, 2016, 6:56pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/18 "2016-03-08T18:56:02Z")

</div>

Sure. I’ll tag @judy, @mogoodrich, @sunbiz and @mseaton so they are aware too.

---

<div class="post-metadata">

**Author:** ![teleivo](https://talk.openmrs.org/user_avatar/talk.openmrs.org/teleivo/32/2015_2.png) [@teleivo](https://talk.openmrs.org/u/teleivo)\
**Post date:** [March 16, 2016, 4:02pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/19 "2016-03-16T16:02:39Z")

</div>

@jthomas I have a family emergency and cannot make it. I am very sorry but I need to cancel the meeting. Can you please make sure everyone knows. Thanks a lot!!!

@judy @mogoodrich @sunbiz

---

<div class="post-metadata">

**Author:** ![mogoodrich](https://talk.openmrs.org/user_avatar/talk.openmrs.org/mogoodrich/32/3324_2.png) [@mogoodrich](https://talk.openmrs.org/u/mogoodrich)\
**Post date:** [March 16, 2016, 4:11pm UTC](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032/20 "2016-03-16T16:11:59Z")

</div>

@teleivo sorry to hear… hope everything is okay!

[Next page](https://talk.openmrs.org/t/design-of-radiologyreport-object-semantics/5032.md?page=2)
