-
Notifications
You must be signed in to change notification settings - Fork 831
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
fix: modified the search engine in the demo notebook to bing #2013
Conversation
Hey @sherylZhaoCode 👋! We use semantic commit messages to streamline the release process. Examples of commit messages with semantic prefixes:
To test your commit locally, please follow our guild on building from source. |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
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.
Summary by GPT-4
This diff shows changes made to a Jupyter Notebook file. The main changes include:
- Removing the mention of SerpAPI and its related code, as it is no longer needed.
- Adding Bing Search API related code and environment variables.
- Updating the sequential_chain to include the new prompt_generation_chain and web_search_agent with Bing Search API.
- Some minor formatting and text changes.
These changes indicate that the notebook has been updated to use Bing Search API instead of SerpAPI for web search functionality in the LangchainTransformer example.
Suggestions
No suggestions are needed as the changes in this PR seem appropriate.
Codecov Report
@@ Coverage Diff @@
## master #2013 +/- ##
===========================================
- Coverage 87.02% 76.31% -10.72%
===========================================
Files 306 306
Lines 16064 16064
Branches 840 840
===========================================
- Hits 13980 12259 -1721
- Misses 2084 3805 +1721 |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
notebooks/features/cognitive_services/CognitiveServices - LangchainTransformer.ipynb
Outdated
Show resolved
Hide resolved
/azp run |
No commit pushedDate could be found for PR 2013 in repo microsoft/SynapseML |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
…ft#2013) * modified the search engine in the demo notebook to bing * reformatted * Update notebooks/features/cognitive_services/CognitiveServices - LangchainTransformer.ipynb --------- Co-authored-by: Mark Hamilton <[email protected]>
Related Issues/PRs
#xxx
What changes are proposed in this pull request?
Briefly describe the changes included in this Pull Request.
How is this patch tested?
Does this PR change any dependencies?
Does this PR add a new feature? If so, have you added samples on website?
website/docs/documentation
folder.Make sure you choose the correct class
estimators/transformers
and namespace.DocTable
points to correct API link.yarn run start
to make sure the website renders correctly.<!--pytest-codeblocks:cont-->
before each python code blocks to enable auto-tests for python samples.WebsiteSamplesTests
job pass in the pipeline.