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

Move methods generating completion replies to provider #3

Closed
wants to merge 9 commits into from

Conversation

krassowski
Copy link
Owner

No description provided.

@krassowski krassowski added the enhancement New feature or request label Apr 5, 2024
@krassowski krassowski force-pushed the completion-generation-in-provider branch 3 times, most recently from a614369 to 30a5792 Compare April 5, 2024 12:18
…jupyterlab#712)

* Update directory.py to add new file formats

1.  Added single file functionality
2.  Added HTML files
3.  Added PDF files

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update dependencies

Added pypdf==4.1.0, required for handling pdf files in /learn

* Update directory.py to add new file formats

1.  Added single file functionality
2.  Added HTML files
3.  Added PDF files

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* Update dependencies

Added pypdf==4.1.0, required for handling pdf files in /learn

* Amended directory.py

Made changes for
1. matching all file extensions in lower case, to ensure no case sensitivity
2. Streamlined the PDF loader to remove a loop over pages, using join and list comprehension.

* Update directory.py

* Update directory.py

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@krassowski krassowski force-pushed the completion-generation-in-provider branch from a50f81a to 45e2e56 Compare April 10, 2024 19:27
3coins and others added 4 commits April 12, 2024 09:24
… `/learn` (jupyterlab#733)

* Handle single files, pdfs, errors

(1) Enables handling single files, not just directories.
(2) Learns PDFs with langchain's PyPDFLoader.
(3) Gives a clean error w/o traceback when the file type that is being handled needs addtional packages.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* error handling for missing packages in learn.py

Removed the extra attribute and additional response comments based on feedback from Piyush Jain and Andrii Ieroshenko

* Amend error message for failure in learn.py

Made the error message more generic as there are many different failure types.

* Fixed build error.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Piyush Jain <[email protected]>
…#735)

* catch embedding model validation errors on extension init

* pre-commit
@krassowski krassowski force-pushed the completion-generation-in-provider branch 2 times, most recently from 7357dec to c16bb54 Compare April 22, 2024 22:29
cloutier and others added 4 commits April 22, 2024 15:34
…lab#726)

* Remove closing markdown identifiers (jupyterlab#686)

* Remove whitespace after closing markdown identifier

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Add an example completion provider

Adjust name of variable to reflect new implementation
@krassowski krassowski force-pushed the completion-generation-in-provider branch from c16bb54 to 4dcf885 Compare April 22, 2024 23:06
@krassowski krassowski closed this Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants