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

Refactor/task memory cleanup #1055

Merged
merged 78 commits into from
Aug 15, 2024
Merged

Refactor/task memory cleanup #1055

merged 78 commits into from
Aug 15, 2024

Conversation

collindutter
Copy link
Member

@collindutter collindutter commented Aug 9, 2024

Describe your changes

Added

  • ExtractionClient Tool for having the LLM extract structured data from text.
  • PromptSummaryClient Tool for having the LLM summarize text.
  • QueryClient Tool for having hte LLM queyr text.

Changed

  • BREAKING: Removed TextArtifactStorage.query and TextArtifactStorage.summarize.
  • BREAKING: Removed TextArtifactStorage.rag_engine, and TextArtifactStorage.retrieval_rag_module_name.
  • BREAKING: Removed TextArtifactStorage.summary_engine, TextArtifactStorage.csv_extraction_engine, and TextArtifactStorage.json_extraction_engine.
  • BREAKING: Removed TaskMemory.summarize_namespace and TaskMemory.query_namespace.
  • BREAKING: Removed Structure.rag_engine.
  • BREAKING: Split JsonExtractionEngine.template_generator into JsonExtractionEngine.system_template_generator and JsonExtractionEngine.user_template_generator.
  • BREAKING: Split CsvExtractionEngine.template_generator into CsvExtractionEngine.system_template_generator and CsvExtractionEngine.user_template_generator.
  • BREAKING: Changed JsonExtractionEngine.template_schema from a run argument to a class attribute.
  • BREAKING: Changed CsvExtractionEngine.column_names from a run argument to a class attribute.
  • BREAKING: Removed JsonExtractionTask, and CsvExtractionTask use ExtractionTask instead.
  • BREAKING: Removed TaskMemoryClient, use RagClient, ExtractionClient, or PromptSummaryClient instead.
  • RagClient now can be used to search through Artifacts stored in Task Memory.

Issue ticket number and link

Closes #623
Closes #1024


📚 Documentation preview 📚: https://griptape--1055.org.readthedocs.build//1055/

@collindutter collindutter force-pushed the refactor/task-memory-cleanup branch 2 times, most recently from 334843a to 01bd603 Compare August 13, 2024 22:58
@collindutter collindutter force-pushed the refactor/task-memory-cleanup branch from 01bd603 to 410f7d3 Compare August 13, 2024 22:58
@collindutter collindutter force-pushed the refactor/task-memory-cleanup branch from 3afb72e to 75f6bfa Compare August 13, 2024 23:49
@collindutter collindutter force-pushed the refactor/task-memory-cleanup branch from c03f26a to b2ee6f7 Compare August 15, 2024 16:55
CHANGELOG.md Outdated Show resolved Hide resolved
docs/griptape-framework/structures/task-memory.md Outdated Show resolved Hide resolved
griptape/tools/query/tool.py Show resolved Hide resolved
griptape/tools/extraction/tool.py Outdated Show resolved Hide resolved
@vachillo
Copy link
Member

87.50% of diff hit (target 87.18%)

impressive

@collindutter collindutter merged commit 60037e9 into dev Aug 15, 2024
13 checks passed
@collindutter collindutter deleted the refactor/task-memory-cleanup branch August 15, 2024 19:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants