Skip to content

Fix grammar and spelling in agentchat_groupchat_RAG.ipynb#349

Closed
shruti222patel wants to merge 1 commit into
microsoft:mainfrom
shruti222patel:patch-8
Closed

Fix grammar and spelling in agentchat_groupchat_RAG.ipynb#349
shruti222patel wants to merge 1 commit into
microsoft:mainfrom
shruti222patel:patch-8

Conversation

@shruti222patel

Copy link
Copy Markdown
Contributor

Why are these changes needed?

For readability.

Related issue number

Checks

"\n",
"\n",
"- `use_spark`: boolean, default=False | Whether to use spark to run the training in parallel spark jobs. This can be used to accelerate training on large models and large datasets, but will incur more overhead in time and thus slow down training in some cases. GPU training is not supported yet when use_spark is True. For Spark clusters, by default, we will launch one trial per executor. However, sometimes we want to launch more trials than the number of executors (e.g., local mode). In this case, we can set the environment variable `FLAML_MAX_CONCURRENT` to override the detected `num_executors`. The final number of concurrent trials will be the minimum of `n_concurrent_trials` and `num_executors`.\n",
"- `use_spark`: boolean, default=False | Whether to use spark to run the training in parallel spark jobs. This can be used to accelerate training on large models and large datasets but will incur more overhead in time and thus slow down training in some cases. GPU training is not supported yet when use_spark is True. For Spark clusters, by default, we will launch one trial per executor. However, sometimes we want to launch more trials than the number of executors (e.g., local mode). In this case, we can set the environment variable `FLAML_MAX_CONCURRENT` to override the detected `num_executors`. The final number of concurrent trials will be the minimum of `n_concurrent_trials` and `num_executors`.\n",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The outputs are not supposed to be altered.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@qingyun-wu

Copy link
Copy Markdown
Contributor

Closing this PR due to conflicts and unnecessary changes. Thank you!

@qingyun-wu qingyun-wu closed this Dec 4, 2023
jackgerrits pushed a commit that referenced this pull request Oct 2, 2024
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.

4 participants