Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
stormcrow79 opened this issue Sep 5, 2022 · 10 comments · Fixed by #402
Closed

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

stormcrow79 opened this issue Sep 5, 2022 · 10 comments · Fixed by #402
Labels
defect A deviation from the Requirements / Specification.
Milestone

Comments

@stormcrow79
Copy link

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.

@milkshakeuk
Copy link
Member

@stormcrow79 I will take a look into this.

@milkshakeuk
Copy link
Member

milkshakeuk commented Sep 6, 2022

@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
Copy link
Member

@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

@stormcrow79
Copy link
Author

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

@milkshakeuk milkshakeuk added defect A deviation from the Requirements / Specification. and removed awaiting review labels Sep 7, 2022
@stormcrow79
Copy link
Author

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

@milkshakeuk
Copy link
Member

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

@milkshakeuk
Copy link
Member

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

@stormcrow79
Copy link
Author

@milkshakeuk thanks for the update - happy holidays!

@milkshakeuk
Copy link
Member

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

@milkshakeuk
Copy link
Member

@stormcrow79 this change is in the latest 3.2.0 version.

@milkshakeuk milkshakeuk added this to the v3.2.0 milestone Jan 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
defect A deviation from the Requirements / Specification.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants