Skip to content

use-cases: add falcon-ai-end-to-end notebook#5

Open
SuhaniNagpal7 wants to merge 4 commits intocookbook/quickstart-notebooksfrom
cookbook/falcon-ai-end-to-end
Open

use-cases: add falcon-ai-end-to-end notebook#5
SuhaniNagpal7 wants to merge 4 commits intocookbook/quickstart-notebooksfrom
cookbook/falcon-ai-end-to-end

Conversation

@SuhaniNagpal7
Copy link
Copy Markdown

Pull Request

Description

Describe the changes in this pull request:

  • What feature/bug does this PR address?
  • Provide any relevant links or screenshots.

Checklist

  • Code compiles correctly.
  • Created/updated tests.
  • Linting and formatting applied.
  • Documentation updated.

Related Issues

Closes #<issue_number>

Copy link
Copy Markdown

@KarthikAvinashFI KarthikAvinashFI left a comment

Choose a reason for hiding this comment

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

Two issues on the latest push:

  • All screenshot URLs across the three notebooks point to empty S3 paths. The paths under s3://fi-cookbook-assets/use-cases/ for all three cookbooks are empty, so none of the images render. Please upload the screenshots and verify the URLs resolve before merging.

  • falcon-ai-end-to-end.ipynb still has a ModuleNotFoundError in cell 8 and a cascading NameError in cell 18. Please do a clean re-run after kernel restart.

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