Combined property-map.sssom.tsv

[joeflack4]

Related to:

• Handling multiple config/*property-map*.sssom.tsvs #482
• Add --allow-duplicate-entities as option in ROBOT rename ontodev/robot#1195

Data PR:

• Combined property-map.sssom.tsv - data files #505

Overview

There have been 2 property map TSV files used by robot rename. However, we are now utilizing the --allow-duplicates flag, which allows us to consolidate them into 1 file.

Pre-merge checklist

Documentation

Was the documentation added/updated under docs/?

• Yes
• No, updates to the docs were not necessary after careful consideration

QC

Was the full pipeline run before submitting this PR using sh run.sh make build-mondo-ingest on this branch (after
docker pull obolibrary/odkfull:dev), and no errors occurred?

• Yes
• No, there are no functional (code-related) changes to the pipeline in the PR, so no re-run is necessary

New Packages

Were any new Python packages added?

• Yes, requirements.txt.full was updated and an ODK issue submitted
• No

Were any other non-Python packages added?

• Yes, Dockerfile updated and an ODK issue submitted
• No

PR Review and Conversations Resolved

Has the PR been sufficiently reviewed by at least 1 team member of the Mondo Technical team and all threads resolved?

• Yes

Changes

Consolidate property maps
- There have been 2 property map TSV files used by robot rename. However, we are now utilizing the --allow-duplicates flag, which allows us to consolidate them into 1 file.
- Delete: one of the property map files
- Rename: the other of the files
- Update: components goals to utilize the new/updated file
- Update: Docs regarding this aspect of adding a new source

[matentzn]

I like this change very much but if you merge into dev before testing the pipeline, you may not be able to separate the resulting problems from other code merged into dev. This is a possibly impactful change (hopefully not)!

[twhetzel]

looks good with these changes and the data PR