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

Add MOSAIC, ARCRC, SENSO, ADCAD, SALMON expand query expansion rules #33

Merged
merged 7 commits into from
Aug 16, 2022

Conversation

amoeba
Copy link
Contributor

@amoeba amoeba commented Oct 8, 2021

Hey @taojing2002, this adds indexing support for two new ontologies we've developed over on https://github.com/DataONEorg/sem-prov-ontologies for #29 and also tweaks some of the indexing logic described in DataONEorg/dataone-indexer#3.

The PR includes automated tests and I've done a fair bit of manual testing so this doesn't need an in-depth review, though I'd appreciate you taking a quick look.

Closes #29 and DataONEorg/dataone-indexer#3.

@amoeba amoeba requested a review from taojing2002 October 8, 2021 23:32
@amoeba amoeba self-assigned this Oct 8, 2021
@amoeba amoeba changed the title Add ARCRC and MOSAiC Ontologies, expand query expansion rules Add ARCRC, MOSAiC, and SENSO Ontologies, expand query expansion rules Nov 9, 2021
@amoeba amoeba changed the title Add ARCRC, MOSAiC, and SENSO Ontologies, expand query expansion rules Add ARCRC, MOSAiC, SENSO Ontologies, expand query expansion rules Nov 9, 2021
@amoeba amoeba changed the title Add ARCRC, MOSAiC, SENSO Ontologies, expand query expansion rules Add ARCRC, MOSAiC Ontologies, expand query expansion rules Nov 9, 2021
amoeba added a commit that referenced this pull request Aug 12, 2022
amoeba added 5 commits August 11, 2022 22:57
Ref #31

Also adds tests for previously added MOSAiC and ARCRC ontologies
This adds support for transitive and symmetric skos:exactMatch and non-transitive but symmetric skos:closeMatch
@amoeba amoeba force-pushed the feature-29-add-mosiac-and-arcrc branch from 9cbcdbe to f7ef529 Compare August 12, 2022 07:10
@amoeba amoeba changed the title Add ARCRC, MOSAiC Ontologies, expand query expansion rules Add MOSAIC, ARCRC, SENSO, ADCAD, SALMON expand query expansion rules Aug 12, 2022
@amoeba
Copy link
Contributor Author

amoeba commented Aug 12, 2022

This PR now includes support for all ontologies we've been developing for the last while and is ready to get merged @taojing2002. This closes #29.

Before or after merge, these changes need to get cherry-picked onto https://github.com/DataONEorg/dataone-indexer and also ported to https://github.com/NCEAS/metacat. All content in CN and MN Solr instances matching the query q=sem_annotation:* needs to get reindexed manually.

@taojing2002 taojing2002 merged commit 7f4712b into develop_2.3 Aug 16, 2022
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.

2 participants