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.

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

Run LLM with Ollama inside Daytona workspace #136

Merged
merged 6 commits into from
Jan 15, 2025

Conversation

Kiran1689
Copy link
Contributor

/claim #2
fix #2

Writer's Checklist

Writing Structure

  • Use short sentences and paragraphs, and include bucket brigades.
  • Include more than two descriptive H2 headings to organize content.
  • Capitalize headings according to the AP Stylebook
    (use this tool)
  • Include an introduction with at least two paragraphs before the first H2
    section.
  • Use appropriate formatting (bold, italic, underline), notes, quotes,
    TLDRs, and key points.
  • Incorporate code elements and Markdown format where appropriate.
  • Ensure at least one visual element per “screen” of the article
    (screenshots, diagrams, tables, graphs, lists, sidenotes, blockquotes).

Fact-Checking

  • Verify all facts and data points included in the article.

Asset Management

  • Save images in the /assets folder.
  • Follow naming conventions:
    YYYYMMDD_title_of_the_article_IMG_NAME_NO.png.
  • (Optional) Create a GitHub repo for the code referenced in the article and
    share it.
  • (Optional) Include a link to this Loom video in the PR comments.

Interlinking

Glossary/Definitions

  • Create new definition in /defitnitions folder.

Review and Edit

  • Ensure articles address the needs of the target audience and their search
    intent.
  • Read the article out loud to catch any awkward phrasing.
  • Run the draft through Grammarly or a similar
    grammar tool.
  • Double-check adherence to the style guide and repository guidelines.
  • Use the name of the article for the title of the PR.

Sorry, something went wrong.

Signed-off-by: Kiran1689 <kirannaragund197@gmail.com>
Signed-off-by: Kiran1689 <kirannaragund197@gmail.com>
Signed-off-by: Kiran1689 <kirannaragund197@gmail.com>
Signed-off-by: Kiran1689 <kirannaragund197@gmail.com>
Signed-off-by: Kiran1689 <kirannaragund197@gmail.com>
@Kiran1689
Copy link
Contributor Author

@mojafa PR is ready for review.

@mojafa
Copy link
Collaborator

mojafa commented Dec 21, 2024

@Kiran1689 please wait for one of your prs to be closed. submitting many prs doesn't help.

@mojafa
Copy link
Collaborator

mojafa commented Dec 21, 2024

@Kiran1689 please close this PR until we successfully merge the other one. I'm reviewing now. you can always resubmit this after.

@Kiran1689
Copy link
Contributor Author

Okay @mojafa
Can you please assign the issue to me?
Thanks!!

@Kiran1689
Copy link
Contributor Author

@mojafa closing this for temporarily..

Can you please assign the issue this PR solves as well. It helps.

@Kiran1689 Kiran1689 closed this Dec 21, 2024
@Kiran1689 Kiran1689 reopened this Dec 21, 2024
@Kiran1689
Copy link
Contributor Author

@mojafa Reopened this PR, as we successfully merged the other one.

@Kiran1689
Copy link
Contributor Author

@mojafa Can you please review?

@mojafa
Copy link
Collaborator

mojafa commented Dec 24, 2024

@Kiran1689 was able to create a new project and replicate your steps, I'm getting an error here though:
Screenshot 2024-12-25 at 00 11 24

@Kiran1689
Copy link
Contributor Author

Hi @mojafa, The error you are getting is because of insufficient RAM. You need at least 16GB of RAM to run the model smoothly.

Signed-off-by: Kiran1689 <kirannaragund197@gmail.com>
@Kiran1689
Copy link
Contributor Author

@mojafa I have added the hostRequirements field in the JSON file and included a description for it as well..

@Kiran1689
Copy link
Contributor Author

@mojafa Any updates on this?

@mojafa
Copy link
Collaborator

mojafa commented Dec 30, 2024

@Kiran1689 thanks, the project runs well and the guide is proper. Please allow for us to rereview either merge or request changes by 8th Jan. Still on holiday...
Screenshot 2024-12-30 at 14 36 20

@nkkko nkkko merged commit ce0352e into daytonaio:main Jan 15, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Run LLM with Ollama inside Daytona workspace
3 participants