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

fixed "cannot allocate memory" exception by specifying max_processes #910

Merged
merged 1 commit into from
Mar 19, 2021
Merged

fixed "cannot allocate memory" exception by specifying max_processes #910

merged 1 commit into from
Mar 19, 2021

Conversation

mosheber
Copy link
Contributor

When using the regular retriever.train method (as specified in Tutorial 9), the process fails in my local setting, due to an "cannot allocate memory error".

After looking through the code, it turns out that the max_processes is by default 128 in farm's DataSilo. When I decreased it to a lower value, the method worked as expected.

Thus, I inserted the option to inject the max_processes value directly through the train method.

Copy link
Member

@tholor tholor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect. Thanks a lot for the addition @mosheber !

@tholor tholor merged commit 47dc069 into deepset-ai:master Mar 19, 2021
@mosheber
Copy link
Contributor Author

@tholor With pleasure!

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