8000 feat: remove `parameters` by ZanSara · Pull Request #11 · deepset-ai/canals · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Nov 28, 2023. It is now read-only.

feat: remove parameters #11

Merged
merged 20 commits into from
May 5, 2023
Merged

feat: remove parameters #11

merged 20 commits into from
May 5, 2023

Conversation

ZanSara
Copy link
Contributor
@ZanSara ZanSara commented Apr 28, 2023 •
edited
Loading
  • Change from data, params to run signature, Output dataclass
  • Connections are validated on type and default on same name in case of ambiguities
  • Add mermaid graph rendering and refactor drawing
  • Refactor pipeline code
  • Add examples in tests for fixed and variable decision components, fixed and variadic merge nodes, dynamic types, dynamic outputs, etc.
  • Add basic test class (wip)
  • Add barebone debug output
  • Update documentation

@ZanSara ZanSara marked this pull request as draft April 28, 2023 15:40
@ZanSara ZanSara marked this pull request as ready for review May 5, 2023 11:27
@ZanSara ZanSara merged commit 850b551 into main May 5, 2023
This was referenced May 5, 2023
Closed
Closed
Closed
Closed
Closed
Closed
Closed
@masci masci deleted the no-params branch November 16, 2023 10:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ZanSara
0