Update mappings pane in a concept details page

[paynejd]

This should be the new default view on the concept details page -- this should be available to all by default without any configuration.

Requirements:

• Rename to "Associations"
• include the count of associations to the right of the header
  • Edit: Partially done but also addressing this in Small concept details tweaks #1013
• include hierarchical relationships in the list using the map_type defined by the source
  • Edit: Looking at this ICD test concept, I am not seeing the hierarchy in there.
• combine "direct" and "inverse" mappings into the same section -- use an icon after the map_type to indicate that it is inverse (eg "inverse" in math is a -1 superscript, or it could just be a label of Inverse)
• sort mappings by internal/external (where internal means pointing to a concept in the same source), then by map_type, then by direct/inverse
• group mappings by map_type + direct/inverse and only show the map_type + direct/inverse once -- no need to repeat it for each row
• Action icon (the 3-dots) should only appear on mouseover
  • Edit: I believe we decided not to do this, although it may be good to reduce the size of the dots.

[jamlung-ri]

@snyaggarwal I reviewed these and put edits into the checklist above where applicable. A few more small changes, and we can close this one out.

[snyaggarwal]

@paynejd @jamlung-ri I will work on the remaining points.
Just on the hierarchy -- if you want to see the hierarchy attributes you will have to ask for it using query params:
includeParentConceptURLs=true, includeChildConceptURLs=true, includeParentConcepts=true, includeChildConcepts=true, includeHierarchyPath=true

[snyaggarwal]

@paynejd @jamlung-ri In order to show hierarchy in associations, should we show both children and parents (as the inverse)?

[jamlung-ri]

I think we should be showing both children and parents, but we should be labeling them like that and not using any inverse. It should look something like this if the source has no hierarchy_meaning:

Relationship	Code	Name	Source
Has Child	LP33231-9	Maternal cell contamination (LP33231-9)	LOINC
	LP192128-9	Cancer risk assessment (LP192128-9)	LOINC
	LP36464-3	Microsatellite instability (LP36464-3)	LOINC
	LP76383-6	Identity testing | Comparison case | Index case (LP76383-6)	LOINC
	LP62970-6	X linked heterotaxy | Blood or Tissue (LP62970-6)	LOINC
Has parent	LP29693-6	Laboratory Categories (LP29693-6)	LOINC

And if there is a hierarchy meaning, then we can use the inverse notation:

Relationship	Code	Name	Source
is-a	LP33231-9	Maternal cell contamination (LP33231-9)	LOINC
	LP192128-9	Cancer risk assessment (LP192128-9)	LOINC
	LP36464-3	Microsatellite instability (LP36464-3)	LOINC
	LP76383-6	Identity testing | Comparison case | Index case (LP76383-6)	LOINC
	LP62970-6	X linked heterotaxy | Blood or Tissue (LP62970-6)	LOINC
is-a -1	LP29693-6	Laboratory Categories (LP29693-6)	LOINC

[paynejd]

Agree with Joe -- just please include the source.hierarchy_meaning in the relationship as well. Something like this: Has child (Is a -1) Has parent (Is a)

…

On Tue, Oct 5, 2021 at 9:42 AM Joseph Amlung ***@***.***> wrote: I think we should be showing both children and parents, but we should be labeling them like that and not using any inverse. It should look something like this: Relationship Code Name Source Has Child LP33231-9 Maternal cell contamination (LP33231-9) LOINC LP192128-9 Cancer risk assessment (LP192128-9) LOINC LP36464-3 Microsatellite instability (LP36464-3) LOINC LP76383-6 Identity testing | Comparison case | Index case (LP76383-6) LOINC LP62970-6 X linked heterotaxy | Blood or Tissue (LP62970-6) LOINC Has parent LP29693-6 Laboratory Categories (LP29693-6) LOINC — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#991 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJCOOJNHM732ZUIB3LOT4DUFL6EDANCNFSM5EURK5HQ> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[snyaggarwal]

Done

[jamlung-ri]

Awesome! Looks good in QA, looking at this ICD concept.