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

add documentation about file and template node #112

Merged
merged 3 commits into from
Dec 12, 2023

Conversation

au-re
Copy link
Contributor

@au-re au-re commented Dec 11, 2023

No description provided.

@au-re au-re linked an issue Dec 11, 2023 that may be closed by this pull request
Copy link
Contributor

@MoElkhidir MoElkhidir left a comment

Choose a reason for hiding this comment

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

very nice documentation, left a few comments.


- [Checklist](/instructions/table): will run your prompt sequentially over a document of variable length and fill a table with parameters you requested. The full document is provided to the LLM as is to ensure no information is missed. This Instruction type is ideal for analyzing documents, running checklists or extracting information from unstructured data.
- [Checklist](/nodes/table): will run your prompt sequentially over a document of variable length and fill a table with parameters you requested. The full document is provided to the LLM as is to ensure no information is missed. This Instruction type is ideal for analyzing documents, running checklists or extracting information from unstructured data.
Copy link
Contributor

Choose a reason for hiding this comment

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

this sentence is a bit confusing, could be simplified? something like "this will run your prompt over a document, validate a checklist you specify and fill the result in a table format"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✔️

A tool is a series of [instructions](instructions.md) that can be combined to create more complex features. Once created, your tool can be deployed and becomes available through an API. You can use our SDK or our Rest API to integrate the workflow into your codebase. You can also share a preview of the tool using our [shareable ui](../tools/preview.md) feature.
Tools can automate tasks, process documents, classify data and do a lot more. We are constantly expanding the set of features available within the tool editor.

A tool is a series of `nodes` that can either be [instructions](instructions.md) or [data](file.md) and that can be combined together to create more complex features. Once created, your tool can be deployed through an API. You can use our [SDK](/sdk/js) or our [Rest API](/api/getting-started) to integrate the tool into your codebase. You can also share a preview of the tool using our [shareable ui](../tools/preview.md) feature.
Copy link
Contributor

Choose a reason for hiding this comment

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

complex features can be complex logic or advance functionality?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

much better! ✔️


The file node has the following inputs:

**`Text`** content of the file, _ignore this if you just want to upload a file_
Copy link
Contributor

Choose a reason for hiding this comment

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

This might not be the time to fix this but I have always wondered why do we have text field in this nod and the user can use the node as template node without uploading any file?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

currently all "fields" are considered inputs, even if in this case it doesn't make sense. We could extend the node API to fix this

- **Control**: if this field is not connected the instruction will not run.
- **AI Settings**: what [language model](/concepts/llms) and parameters to use to complete the instruction
- **Prompt**: The [prompt](/concepts/prompts) to provide to the language model
**`Control`** if this field is not connected the instruction will not run
Copy link
Contributor

Choose a reason for hiding this comment

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

can we expand on this and explain why it won't run? maybe something similar to "this controls the execution order of the node"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

what do you think about this version?

@@ -0,0 +1,35 @@
# Checklist

The **checklist instruction** will run your prompt sequentially over a document of any length and fill a table with parameters you requested. The full document is provided to the LLM as is, to ensure no information is missed. This is ideal for analyzing documents, running checklists or extracting information from unstructured data.
Copy link
Contributor

Choose a reason for hiding this comment

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

similar to previous comment related to this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✔️

@@ -0,0 +1,35 @@
# Checklist

The **checklist instruction** will run your prompt sequentially over a document of any length and fill a table with parameters you requested. The full document is provided to the LLM as is, to ensure no information is missed. This is ideal for analyzing documents, running checklists or extracting information from unstructured data.
Copy link
Contributor

Choose a reason for hiding this comment

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

checklist node*

Copy link
Contributor Author

Choose a reason for hiding this comment

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

✔️


## Inputs

The Instruction node has the following inputs:
Copy link
Contributor

Choose a reason for hiding this comment

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

checklist node*

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed 👍


The Instruction node has the following inputs:

**`Control`** if this field is not connected the instruction will not run.
Copy link
Contributor

Choose a reason for hiding this comment

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

similar to the previous comment related to the control field

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed 👍

@au-re au-re merged commit 9d0c07e into main Dec 12, 2023
@au-re au-re deleted the 111-add-documentation-about-file-and-template-nodes branch December 12, 2023 08:35
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.

Add documentation about file and template nodes
2 participants