ViccNormalizerDataExtension should be moved to mappings

[korikuzma]

Currently, we have the a ViccNormalizerDataExtension in MappableConcept.extensions. For Disease conceptTypes, this looks like:

{
  "name": "vicc_normalizer_data",
  "value": {
    "id": "ncit:C4035",
    "label": "Thyroid Gland Papillary Carcinoma",
    "mondo_id": "mondo:0005075"
  }
}

However, we should actually be storing these as ConceptMappings in mappings:

[
    {
        "coding": {
            "id": "doid:3908",
            "code": "DOID:3908",
            "system": "https://disease-ontology.org/?id="
        },
        "relation": "exactMatch",
        "extensions": [
            {
                "name": "from_vicc_normalizer",
                "value": true
            }
        ]
    },
    {
        "coding": {
            "id": "ncit:C2926",
            "code": "C2926",
            "label": "Lung Non-Small Cell Carcinoma",
            "system": "https://ncit.nci.nih.gov/ncitbrowser/ConceptReport.jsp?dictionary=NCI_Thesaurus&code="
        },
        "relation": "exactMatch",
        "extensions": [
            {
                "name": "from_vicc_normalizer",
                "value": true
            }
        ]
    },
    {
        "coding": {
            "id": "mondo:0005233",
            "code": "MONDO:0005233",
            "system": "http://purl.obolibrary.org/obo/mondo.owl"
        },
        "relation": "exactMatch",
        "extensions": [
            {
                "name": "from_vicc_normalizer",
                "value": true
            }
        ]
    }
]

I might need to check with @ahwagner on relation values.

[korikuzma]

Need to figure out if we change relation to exact match or use what's in the normalizers

[korikuzma]

I realize we also need to retain if it was the merged concept ID (from priority source). Thinking of updating the extension to be:

"extensions": [
            {
                "name": "from_vicc_normalizer",
                "value": {"is_merged_identifier": True}
            }
        ]

or

"extensions": [
            {
                "name": "vicc_normalizer_merged_id",
                "value": True
            }
        ]

Where vicc_normalizer_merged_id would only be present in VICC concept mappings. I kind of like this option better because it's more compact.

@jsstevenson @katiestahl thoughts? Open to alternatives.

Update: Not sure about vicc_normalizer_merged_id since it will also include label and not just the merged concept ID... Going to go with vicc_normalizer_priority for now.

[github-actions]

Closed by #425.