Skip to content

Conversation

kevinbleckmann
Copy link
Contributor

Missing AMI code from the example for genai fast chat
updated Readme as well to be more clear on running the module

missing some ami code from the example and added verbiage to readme file for better user experience
1. As mentioned above, **wait ~10 minutes** for the Recipe to download/install FastChat and the LLM model before continuing
2. SSH into newly created AWS EC2 instance.
3. The terraform module creates a key pair and adds the public key to the EC2 instance. It keeps the private key in the same folder from where the **terraform apply** was run.
3. The terraform module creates a key pair and adds the public key to the EC2 instance. It keeps the private key in the same folder from where the **terraform apply** was run. (tf.private)
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the filename is tfkey.private

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i updated that. After Talking to Lucas he wants me to write the Demo for AWS Cloudshell and see if we can simplify the instructions. I cant test until the module is updated with the AMI code changes.

7. Your app will be proxied through gradio. See https://xxxxxxx.gradio.live URL that is generated during the run_demo.sh script execution.
8. Open a browser and put the gradio url referenced in the prior step
7. Now you can access the Fastchat by opening your browser and entering the following URL "http://<yourpublicip>:7860.
8. Now you can enter your message or question in the chat prompt to see the Fastchat in action?
Copy link
Contributor

Choose a reason for hiding this comment

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

? mark may not be needed at the end since this is not a question.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rewriting the demo piece to use Cloudshell

@mandalrajiv mandalrajiv merged commit a4efd21 into main Oct 5, 2023
@mandalrajiv mandalrajiv deleted the 10-2-23-update branch October 5, 2023 00:35
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.

2 participants