Skip to content

Conversation

@VeronicaMuthee
Copy link
Collaborator

@VeronicaMuthee VeronicaMuthee commented Oct 27, 2025

Updated cause of death concepts

Removed Concept: "Probable cause of death" (CIEL 1599) (row 67 - 67)
Removed 3 mappings:

  1. CIEL → CIEL (self-reference)
  2. CIEL → SNOMED-CT (184305005)
  3. CIEL → AMPATH (5002)

Updated cause of death concepts
@denniskigen
Copy link
Member

denniskigen commented Oct 28, 2025

All of these instances of the deleted concept should be fixed to leverage the newly added "Probable cause of death" concept.

@ibacher
Copy link
Member

ibacher commented Dec 2, 2025

@VeronicaMuthee Usually we want these self-reference mappings, i.e., that's how we "know" that the concept is, e.g., CIEL:1599. The other two mappings should be retained too, I think...

@dkayiwa
Copy link
Member

dkayiwa commented Dec 2, 2025

With the changes in this pull request, the error has changed to this: https://gist.github.com/dkayiwa/363b521a03955269e8a2f679024475c3

@VeronicaMuthee
Copy link
Collaborator Author

With the changes in this pull request, the error has changed to this: https://gist.github.com/dkayiwa/363b521a03955269e8a2f679024475c3

Seems like the issue is with importation of this concept ....160218, let me import that "Caused by: org.openmrs.module.openconceptlab.importer.SavingException: Cannot create mapping from concept with URL /orgs/CIEL/sources/CIEL/concepts/160218/, because the concept has not been imported"

Added Probable cause of death concept (160218)
@dkayiwa
Copy link
Member

dkayiwa commented Dec 3, 2025

Now all is well and we have no more errors.
Thanks @denniskigen and @VeronicaMuthee 😊

@dkayiwa dkayiwa merged commit 263af2e into main Dec 3, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants