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 concept sets search (mergeable) #2315

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

alex-odysseus
Copy link
Contributor

No description provided.

anton-abushkevich and others added 30 commits October 31, 2022 17:34
…arch

# Conflicts:
#	src/main/resources/i18n/messages_en.json
#	src/main/resources/i18n/messages_ko.json
#	src/main/resources/i18n/messages_ru.json
#	src/main/resources/i18n/messages_zh.json
…_sets_search

# Conflicts:
#	src/main/java/org/ohdsi/webapi/service/ConceptSetService.java
#	src/main/java/org/ohdsi/webapi/vocabulary/solr/SolrSearchProvider.java
…_sets_search

# Conflicts:
#	pom.xml
#	src/main/java/org/ohdsi/webapi/service/ConceptSetService.java
#	src/main/java/org/ohdsi/webapi/service/VocabularyService.java
#	src/main/java/org/ohdsi/webapi/vocabulary/DatabaseSearchProvider.java
#	src/main/java/org/ohdsi/webapi/vocabulary/VocabularySearchService.java
#	src/main/java/org/ohdsi/webapi/vocabulary/VocabularySearchServiceImpl.java
@anthonysena
Copy link
Collaborator

Linking to #2177.

From discussion on the Atlas/WebAPI WG call: This set of changes for advanced concept set search is complex and is currently lacking information on how the functionality is intended to work. As an example, is concept set indexing something that run synchronously and could this delay saving of a concept set? It is also unclear how we can run this branch locally to test the functionality since it is unclear how to configure SOLR to make the functions of this branch work properly.

We're proposing to move this to v2.15 but need some feedback on this @alex-odysseus

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