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

added n_threads functionality for gpt4all #5427

Merged
merged 2 commits into from
May 30, 2023
Merged

added n_threads functionality for gpt4all #5427

merged 2 commits into from
May 30, 2023

Conversation

Vokturz
Copy link
Contributor

@Vokturz Vokturz commented May 30, 2023

Added support for modifying the number of threads in the GPT4All model

I have added the capability to modify the number of threads used by the GPT4All model. This allows users to adjust the model's parallel processing capabilities based on their specific requirements.

Changes Made

  • Updated the validate_environment method to set the number of threads for the GPT4All model using the values["n_threads"] parameter from the GPT4All class constructor.

Context

Useful in scenarios where users want to optimize the model's performance by leveraging multi-threading capabilities.
Please note that the n_threads parameter was included in the GPT4All class constructor but was previously unused. This change ensures that the specified number of threads is utilized by the model .

Dependencies

There are no new dependencies introduced by this change. It only utilizes existing functionality provided by the GPT4All package.

Testing

Since this is a minor change testing is not required.

Who can review?

Community members can review the PR once tests pass. Tag maintainers/contributors who might be interested:

Models

@dev2049 dev2049 merged commit 8121e04 into langchain-ai:master May 30, 2023
vowelparrot pushed a commit that referenced this pull request May 31, 2023
# Added support for modifying the number of threads in the GPT4All model

I have added the capability to modify the number of threads used by the
GPT4All model. This allows users to adjust the model's parallel
processing capabilities based on their specific requirements.

## Changes Made
- Updated the `validate_environment` method to set the number of threads
for the GPT4All model using the `values["n_threads"]` parameter from the
`GPT4All` class constructor.

## Context
Useful in scenarios where users want to optimize the model's performance
by leveraging multi-threading capabilities.
Please note that the `n_threads` parameter was included in the `GPT4All`
class constructor but was previously unused. This change ensures that
the specified number of threads is utilized by the model .

## Dependencies
There are no new dependencies introduced by this change. It only
utilizes existing functionality provided by the GPT4All package.

## Testing
Since this is a minor change testing is not required.

---------

Co-authored-by: Dev 2049 <dev.dev2049@gmail.com>
@danielchalef danielchalef mentioned this pull request Jun 5, 2023
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
# Added support for modifying the number of threads in the GPT4All model

I have added the capability to modify the number of threads used by the
GPT4All model. This allows users to adjust the model's parallel
processing capabilities based on their specific requirements.

## Changes Made
- Updated the `validate_environment` method to set the number of threads
for the GPT4All model using the `values["n_threads"]` parameter from the
`GPT4All` class constructor.

## Context
Useful in scenarios where users want to optimize the model's performance
by leveraging multi-threading capabilities.
Please note that the `n_threads` parameter was included in the `GPT4All`
class constructor but was previously unused. This change ensures that
the specified number of threads is utilized by the model .

## Dependencies
There are no new dependencies introduced by this change. It only
utilizes existing functionality provided by the GPT4All package.

## Testing
Since this is a minor change testing is not required.

---------

Co-authored-by: Dev 2049 <dev.dev2049@gmail.com>
This was referenced Jun 25, 2023
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.

3 participants