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

feat: add export arg to export to md #1352

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cyai
Copy link
Contributor

@cyai cyai commented Jul 23, 2024

Describe the changes you have made:

Added conversation export functionality to markdown files:

  • Implemented export_to_markdown() method in OpenInterpreter class
  • Added --export command-line argument to specify export at startup
  • Modified main() function to handle export on KeyboardInterrupt (Ctrl+C)
  • Export uses default or user-specified filename
  • Exports to OS-specific default directories

Pre-Submission Checklist (optional but appreciated):

  • I have included relevant documentation updates (stored in /docs)
  • I have read docs/CONTRIBUTING.md
  • I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux

@cyai
Copy link
Contributor Author

cyai commented Jul 24, 2024

@KillianLucas I noticed that Additional functions in core beyond running code is considered out of scope. But my current features includes adding functionalities to the core. However I can use the same functions as utils. Should I do that? Just wanted to get this though you before changing it

@MikeBirdTech
Copy link
Collaborator

@cyai It would be helpful to hear how this differs from https://github.com/OpenInterpreter/open-interpreter/blob/main/interpreter/terminal_interface/utils/export_to_markdown.py

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