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

first commit #998

Merged
Merged

Conversation

emrgnt-cmplxty
Copy link
Contributor

@emrgnt-cmplxty emrgnt-cmplxty commented Aug 28, 2024

🚀 This description was created by Ellipsis for commit ef7b67b

Summary:

This pull request refactors the codebase by renaming chunking_settings to chunking_config, adding Hatchet workflows, and restructuring services and API routes for improved modularity and functionality.

Key points:

  • Renamed chunking_settings to chunking_config in go/sdk/pkg/sdk/ingestion.go, js/sdk/src/r2rClient.ts, and py/sdk/ingestion.py.
  • Added Hatchet workflows for ingestion and restructuring in py/core/main/hatchet/ingestion_workflow.py and py/core/main/hatchet/restructure_workflow.py.
  • Updated Docker setup to include Hatchet in py/cli/commands/server.py and py/cli/utils/docker_utils.py.
  • Introduced R2RSerializable class in py/core/base/abstractions/base.py for serialization.
  • Refactored API routes by moving and renaming files for better organization.
  • Enhanced IngestionService and RestructureService with adapters for input parsing.
  • Updated pyproject.toml to require Python 3.10 and added hatchet-sdk dependency.
  • Added new Docker compose file py/compose.hatchet.yaml for Hatchet services.

Generated with ❤️ by ellipsis.dev

@emrgnt-cmplxty emrgnt-cmplxty marked this pull request as ready for review August 28, 2024 20:27
@emrgnt-cmplxty emrgnt-cmplxty merged commit f7aac9e into feature/add-orchestration Aug 28, 2024
0 of 3 checks passed
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to ef7b67b in 39 seconds

More details
  • Looked at 2449 lines of code in 58 files
  • Skipped 1 files when reviewing.
  • Skipped posting 0 drafted comments based on config settings.

Workflow ID: wflow_tlTjcuyZ29u3hQS2


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@emrgnt-cmplxty emrgnt-cmplxty deleted the feature/hatchet-orchestration branch September 6, 2024 18:38
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.

1 participant