nHapi icon indicating copy to clipboard operation
nHapi copied to clipboard

Incorrect repeat counts for OBX-8/9/10/16 in v2.4

Open stormcrow79 opened this issue 3 years ago • 6 comments

NHapi.Model.V24.OBX has incorrect repeat counts for the following fields:

  • OBX-8 Abnormal Flags is 1, should be 5
  • OBX-9 Probability is 5, should be 1
  • OBX-10 Nature of Abnormal Test is 1, should be unlimited
  • OBX-16 Responsible Observer is 1, should be unlimited

image

NHapi 2.5.0.6 to 3.1.0 throw HL7Exception from LegacyPipeParser.Parse for a segment that is valid by the spec:

OBX|1|FT|ALLERG^Allergie^MOLIS||dummy report|||A~B~C~D~E|0.5|X~Y~Z|C|||20030331053409||A11^SMITH^JOHN~B22^JONES^FRED~C33^BEAN^MARY";

NHapi 3.1.1 does not raise an exception due to changes in PR #277.

This seems to be due to a problem with the HL7 database and/or codegen affecting only v2.4 - v2.3 and v2.5 have the expected repeat count. This is similar to issue #191.

I'm happy to raise a PR for the model changes.

stormcrow79 avatar Sep 05 '22 06:09 stormcrow79

@stormcrow79 I will take a look into this.

milkshakeuk avatar Sep 06 '22 09:09 milkshakeuk

@stormcrow79 I ran the query in the database and it looks like the database might be incorrect here, I will let the DB maintainer know to see if we can get a fix in to the database before we consider any PRs.

v2.4 image

milkshakeuk avatar Sep 06 '22 11:09 milkshakeuk

@stormcrow79 seems it is an issue with the database, when I get an updated copy of the DB I should be able to generate the fix.

Screenshot_20220906-190921.png

milkshakeuk avatar Sep 06 '22 18:09 milkshakeuk

Great, thanks @milkshakeuk . I've been sitting on this one since Sep 2020 when 2.x was unmaintained!

stormcrow79 avatar Sep 07 '22 00:09 stormcrow79

hey @milkshakeuk what's the release cadence for the db?

stormcrow79 avatar Oct 06 '22 08:10 stormcrow79

@stormcrow79 good question, I think its probably ready when its ready - but I will check back with Frank!

milkshakeuk avatar Oct 06 '22 09:10 milkshakeuk

@stormcrow79 sorry for the long delay on this, I will check in with Frank probably in the new year now.

milkshakeuk avatar Dec 20 '22 20:12 milkshakeuk

@milkshakeuk thanks for the update - happy holidays!

stormcrow79 avatar Dec 21 '22 04:12 stormcrow79

@stormcrow79 okay got the latest DB, will look to create a PR for this soon.

milkshakeuk avatar Jan 16 '23 10:01 milkshakeuk

@stormcrow79 this change is in the latest 3.2.0 version.

milkshakeuk avatar Jan 20 '23 13:01 milkshakeuk