tstadel

Results 35 comments of tstadel

> > @FHardow can we add a test for get_run_results() too please? > > Sure. I couldn't find tests for the other functions in that class and skipped them. The...

@anakin87 this is almost good to go. Only the tutorial script is missing. I'm pushing it over the finish line.

@anakin87 unfortunately I don't have write access. Can you run the script yourself or [grant me access](https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork)?

> 'Allow edits and access to secrets by maintainers' is already selected. What can I do to allow you? @anakin87 Ok, weird. To fix this quickly, can you grant me...

@anakin87 Yes thanks! Now it's working.

@brandenchan this is to be merged before we release, right?

Hey @mumtazcem, good to here that you managed to deploy your pipeline :-) We already started on creating a simple helm chart, but it's still a draft. It works on...

@espoirMur have you tried setting `refresh_type="false"`. This way each bulk index request does not wait until the documents are indexed but continues. This often dramatically speeds up document ingestion. First...

It's a param of `ElasticsearchDocumentStore`'s constructor.

> @tstadel I added some basic formatting using openpyxl. > > pylint still produces an error because I am "instantiating" an abstract class (whose `__new__()` function actually returns an instance...