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.

Sign up for GitHub

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

Jump to bottom

feat(core): Add LLM application building support #6998

Closed
wants to merge 262 commits into from
Closed

feat(core): Add LLM application building support #6998

wants to merge 262 commits into from

Conversation

janober
Copy link
Member

@janober janober commented Aug 23, 2023

Github issue / Community forum post (link here to close automatically):
https://community.n8n.io/t/langchain-memory-chat/23733

@janober
⚡ Make sure that only nodes from other inputs than main can get
5e7973d 
queried
@janober
⚡ Add support for more connection types
e04a80c
@janober
⚡ Fix bug with connections
061eb27
@janober
⚡ Add incomplete Motorhead support
63c197a
@janober
⚡ Split out the Language Model
1802557
@janober
⚡ Add HuggingFace
c6dbe58
@janober
⚡ Finally move logic to nodes
81a5c1f
@janober
⚡ Remove not needed code
7034fad
@janober
⚡ Add Cohere support (WIP)
5629f85
@janober
⚡ Move nodes to own package
48dc854
@janober
⚡ Proper move and make the nodes load
a295a5e
@janober
⚡ Improve design
06a8b6f
@janober
⚡ Fix some display issues
64224cb
@janober
⚡ Some small design improvements
6c980e2
@janober
⚡ Open node creation panel only for "main" connections
2164322
@janober
⚡ Add watch command to new nodes package
865f75b
@janober
⚡ Make tools observable
e99d27e
@janober
⚡ Make languageModels observable
9845fd9
@janober
⚡ Remove not needed imports
6372080
@janober
Merge branch 'master' into ai-tool-creation
a6d2ed2
@janober
⚡ Remove AITest nodes
26a2f42
@janober
⚡ Fix Run selector for Input Data
d1b5529
@janober
⚡ Separate node style more strongly
535c52a
@janober
⚡ Properly set exection metadata on config nodes
a137476
@janober
⚡ Improve selected style
26e69a3
@janober
⚡ Remove some debug messages
4f986eb
@janober
⚡ Fix color of connected config nodes
c6c076b
@janober
⚡ Display config nodes as executing
f3df7a3
@janober
⚡ Display item count only on main connections
343bdd1
@janober
⚡ Improve execution state display on nodes
4f1d3cc
janober and others added 11 commits September 19, 2023 16:58
@janober
Merge branch 'master' into ai-tool-creation
c39dcf8
@janober
feat: Make it possible to supply additional data to Workflow called by
da2a1f5 
Tool
@janober
⚡ Improve error reporting of tools
2c62f3c
@alexgrozav @janober
feat: Update node colors on canvas (#7199)
ed3c2e8 
<img width="915" alt="image"
src="https://github.com/n8n-io/n8n/assets/6179477/81172c7a-b48f-4bce-af2c-c000ece2e16b">

---------

Co-authored-by: Jan Oberhauser <jan.oberhauser@gmail.com>
@janober
⚡ Display JSON toggle in AI Logs only if the displayed content
c97a523 
got parsed
@janober
⚡ Throw error when wrong loader on Binary to Document node is
88df24e 
selected
@janober
⚡ Improve general error reporting
50a1118
@janober
⚡ Add descriptions for Summarization Chain types
2d68e58
@janober
⚡ Fix credential documentation links
e1de989
@janober
⚡ Automatically set the first node as active in AI Logs
9e97235
@netroy
Merge remote-tracking branch 'origin/master' into ai-tool-creation
6541864
@netroy netroy mentioned this pull request Sep 21, 2023
Merged
janober and others added 14 commits September 22, 2023 10:40
@janober
bug(core): Fix issue that log indexes were messed up
6b3582f
@janober
⚡ Fix issues & improve parameter order
6b848e4
@janober
⚡ Give property a more generic name (breaks dislpay of old
f2d14dd 
executions)
@janober
⚡ Allow drop of output connections to display compatible nodes
1d7a5f0
@alexgrozav @janober
feat(editor): Update node styling, connector styling and grouping (#7222
24b5166 
)

<img width="1496" alt="image"
src="https://github.com/n8n-io/n8n/assets/6179477/c5fb08e2-1ddb-40a1-8d98-4a1033ae87ad">

---------

Co-authored-by: Jan Oberhauser <jan.oberhauser@gmail.com>
@janober
Merge branch 'master' into ai-tool-creation
0f66b45
@janober
bug: Fix automatic connections
61b5158
@janober
⚡ Fix icons on scaled nodes
d5abf9a
@janober
⚡ Make sub-nodes clearer in node creator
177076f
@janober
⚡ Make displaying of sub-nodes more generic
95c516b
@janober
⚡ Improve size of icons of configurable nodes
0d7e801
@janober
⚡ Standardise document loader names and descriptions
9f28cc6
@janober
⚡ Make sure nodes move as a group when node gets injected
a37ac8d
@janober
⚡ Make also keyboard shortcut for multi select work
21bd03f
@cypress
Copy link

cypress bot commented Sep 25, 2023

18 failed and 1 flaky tests on run #2259 ↗︎

18 127 0 0 Flakiness 1

Details:

🌳 ai-tool-creation 🖥️ browsers:node18.12.0-chrome107 🤖 oleg 🗃️ e2e/*
Project: n8n Commit: 21bd03f7f9
Status: Failed Duration: 18:16 💡
Started: Sep 25, 2023 9:14 AM Ended: Sep 25, 2023 9:32 AM
Failed  10-undo-redo.cy.ts • 1 failed test

View Output Video

Test Artifacts
Undo/Redo > should undo/redo deleting a connection by moving it away Output Screenshots Video
Failed  4-node-creator.cy.ts • 2 failed tests

View Output Video

Test Artifacts
Node Creator > should search for nodes Output Screenshots Video
Node Creator > should have most relevenat nodes on top when searching Output Screenshots Video
Failed  12-canvas.cy.ts • 1 failed test

View Output Video

Test Artifacts
Canvas Node Manipulation and Navigation > should add merge node and test connections Output Screenshots Video
Failed  5-ndv.cy.ts • 3 failed tests

View Output Video

Test Artifacts
NDV > should test webhook node Output Screenshots Video
NDV > can link and unlink run selectors between input and output Output Screenshots Video
NDV > should flag issues as soon as params are set Output Screenshots Video
Failed  16-webhook-node.cy.ts • 10 failed tests

View Output Video

Test Artifacts
Webhook Trigger node > should listen for a GET request Output Screenshots Video
Webhook Trigger node > should listen for a POST request Output Screenshots Video
Webhook Trigger node > should listen for a DELETE request Output Screenshots Video
Webhook Trigger node > should listen for a HEAD request Output Screenshots Video
Webhook Trigger node > should listen for a PATCH request Output Screenshots Video
Webhook Trigger node > should listen for a PUT request Output Screenshots Video
Webhook Trigger node > should listen for a GET request and respond custom status code 201 Output Screenshots Video
Webhook Trigger node > should listen for a GET request and respond with an empty body Output Screenshots Video
Webhook Trigger node > should listen for a GET request with Basic Authentication Output Screenshots Video
Webhook Trigger node > should listen for a GET request with Header Authentication Output Screenshots Video

The first 5 failed specs are shown, see all 28 specs in Cypress Cloud.

Flakiness  cypress/e2e/24-ndv-paired-item.cy.ts • 1 flaky test

View Output Video

Test Artifacts
NDV > resolves expression with default item when input node is not parent, while still pairing items Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@netroy
Copy link
Member

netroy commented Oct 2, 2023

closing in favor of #7246

@netroy netroy closed this Oct 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
n8n team Authored by the n8n team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@janober @netroy @valya @OlegIvaniv @alexgrozav @krynble