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.

Sign up for GitHub

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

Jump to bottom

LangChain Quickstart #2005

Merged
merged 19 commits into from
Sep 7, 2023
Merged

LangChain Quickstart #2005

merged 19 commits into from
Sep 7, 2023

Conversation

sjyothi54
Copy link
Contributor

Ticket URL: https://issues.newrelic.com/browse/NR-156585
Description: Created LangChain quickstart

Pre merge checklist

  • Did you check you NRQL syntax? - Does it work?
  • Did you include a Data source and Documentation reference?
  • Are all documentation links publicly accessible?
  • Did you check your descriptive content for voice and tone?
  • Did you check your descriptive content for spelling and grammar errors?
  • Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

  • Does the PR contain a screenshot for each of your dashboards?
  • Do your screenshots show data?
  • Has the sanitization script been run on each dashboard?

Alerts

  • Did you check that your alerts actually work?

@github-actions
Copy link

github-actions bot commented Sep 4, 2023

Thank you for your contribution, our team will be reviewing this shortly, please be available for any follow up questions or code review feedback!

@RamanaReddy8801 RamanaReddy8801 added the Tara Submitted by a team member from Tara team (Previously CXPUI team) in HYD label Sep 4, 2023
@seemantk seemantk added the aimlops AI & MLOps label Sep 5, 2023
d3caf
d3caf previously approved these changes Sep 5, 2023
View reviewed changes
rahulbasu-nr
rahulbasu-nr suggested changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@rahulbasu-nr rahulbasu-nr left a comment

Choose a reason for hiding this comment

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

.

pkosanam
pkosanam suggested changes Sep 7, 2023
View reviewed changes
data-sources/langchain/config.yml Outdated Show resolved Hide resolved
data-sources/langchain/config.yml Outdated Show resolved Hide resolved
@sjyothi54 sjyothi54 requested a review from seemantk as a code owner September 7, 2023 14:32
mdumpati
mdumpati previously approved these changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@mdumpati mdumpati left a comment

Choose a reason for hiding this comment

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

LGTM

pkosanam
pkosanam previously approved these changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@pkosanam pkosanam left a comment

Choose a reason for hiding this comment

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

LGTM

seemantk
seemantk previously approved these changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@seemantk seemantk left a comment

Choose a reason for hiding this comment

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

lgtm

@pkosanam pkosanam dismissed stale reviews from mdumpati, seemantk, and themself via ac6ad98 September 7, 2023 19:54
mdumpati
mdumpati approved these changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@mdumpati mdumpati left a comment

Choose a reason for hiding this comment

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

LGTM

seemantk
seemantk approved these changes Sep 7, 2023
View reviewed changes
Copy link
Contributor

@seemantk seemantk left a comment

Choose a reason for hiding this comment

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

lgtm

@mdumpati mdumpati merged commit 917a35d into main Sep 7, 2023
@mdumpati mdumpati deleted the NR-156585 branch September 7, 2023 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pkosanam pkosanam pkosanam left review comments

@mdumpati mdumpati mdumpati approved these changes

@seemantk seemantk seemantk approved these changes

@d3caf d3caf d3caf left review comments

@rahulbasu-nr rahulbasu-nr Awaiting requested review from rahulbasu-nr

Assignees
No one assigned
Labels
aimlops AI & MLOps new-quickstart Tara Submitted by a team member from Tara team (Previously CXPUI team) in HYD
Projects
NR1 Quickstarts Community
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@sjyothi54 @seemantk @d3caf @mdumpati @pkosanam @rahulbasu-nr @RamanaReddy8801