Skip to content
This repository has been archived by the owner on Mar 29, 2024. It is now read-only.

Adding earnings call chatbot blog post #43

Merged
merged 9 commits into from
Aug 3, 2023
Merged

Adding earnings call chatbot blog post #43

merged 9 commits into from
Aug 3, 2023

Conversation

htrivedi99
Copy link
Contributor

No description provided.

@htrivedi99 htrivedi99 requested a review from filopedraz July 31, 2023 20:51
@filopedraz filopedraz requested review from tiero and removed request for filopedraz July 31, 2023 21:14
casperdcl
casperdcl suggested changes Aug 2, 2023
View reviewed changes
Copy link
Contributor

@casperdcl casperdcl left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Thanks! high-level feedback: mostly missing info on how to run models on-prem (don't need to describe in detail, could explain briefly & link to other tutorials)

blog/2023-07-31-quarterly-earnings-chatbot/banner.png Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
blog/2023-07-31-quarterly-earnings-chatbot/index.md Outdated Show resolved Hide resolved
@casperdcl casperdcl force-pushed the het-blogs branch 4 times, most recently from 7160179 to 39ea1bd Compare August 3, 2023 15:29
casperdcl
casperdcl approved these changes Aug 3, 2023
View reviewed changes
Copy link
Contributor

@casperdcl casperdcl left a comment •
edited
Loading

Choose a reason for hiding this comment

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

lgtm; opened a couple of follow-up issues (#49, #50).
Also:

  • where did you get the NVIDIA recording from? Should cite the original source 55cf619
  • (minor) from my tests, the banner isn't rendering on the blog home page

@filopedraz filopedraz merged commit d75f1ee into main Aug 3, 2023
@casperdcl casperdcl deleted the het-blogs branch August 4, 2023 00:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@casperdcl casperdcl casperdcl approved these changes

@tiero tiero Awaiting requested review from tiero

Assignees

@htrivedi99 htrivedi99

Labels
content-blog
Projects
No open projects
Prem ML
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@htrivedi99 @casperdcl @filopedraz