-
Notifications
You must be signed in to change notification settings - Fork 104
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
yqiao/master #1248
base: master
Are you sure you want to change the base?
yqiao/master #1248
Conversation
add timeout parameter.
when deleting first check if index exists and ignore if it doesn't.
Fixed KeywordQuery and TermQuery so that they work for substructure search. Note query must be constructed with is_substructure=True.
…ile is rxn from chemaxon it may need sanitized with automap().
# Conflicts: # .ci/version.txt # api/dotnet/src/Indigo.Net.csproj # api/http/requirements.txt # api/http/setup.py # api/indigo-version.cmake # api/java/pom.xml # api/python/indigo/__init__.py # api/python/setup.py # api/r/DESCRIPTION # api/wasm/indigo-ketcher/package.json # bingo/bingo-elastic/java/pom.xml # bingo/bingo-elastic/python/bingo_elastic/__init__.py # bingo/bingo-elastic/python/bingo_elastic/elastic.py # bingo/bingo-elastic/python/setup.py # utils/indigo-ml/setup.py # utils/indigo-service/backend/service/requirements.txt
For aromotize() call on registration and post-processing perhaps it can preserve backward compatibility should we have a toggle on the ElasticRepository and AsyncElasticEpiository class with a default value set to True (so it can be set to false manually for legacy connections)? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apologies for long review and thank you for your PR. Great to have new Query (TermQuery), but we would clarify some questions.
For Aleksandr Savelev as requested.