Skip to content
This repository has been archived by the owner on Nov 13, 2024. It is now read-only.

[Bug] Please upgrade the dependencies for canopy-sdk #353

Open
2 tasks done
NB-123 opened this issue Nov 1, 2024 · 8 comments
Open
2 tasks done

[Bug] Please upgrade the dependencies for canopy-sdk #353

NB-123 opened this issue Nov 1, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@NB-123
Copy link

NB-123 commented Nov 1, 2024

Is this a new bug?

  • I believe this is a new bug
  • I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior

Since your last release was in March, a bunch of canopy-idk dependencies are pinned to very old versions. Many users have raised issues with package conflicts + security vulnerabilities. Please fix.

Expected Behavior

That a company like Pinecone actively maintains what it ships out to the open source world...

Steps To Reproduce

Try installing canopy-idk with any modern version of tokenizer/gunicorn etc, you will likely face issues.

Relevant log output

No response

Environment

- **OS**:
- **Language version**:
- **Canopy version**:

Additional Context

No response

@NB-123 NB-123 added the bug Something isn't working label Nov 1, 2024
@NB-123
Copy link
Author

NB-123 commented Nov 1, 2024

@miararoy

@miararoy
Copy link
Contributor

miararoy commented Nov 3, 2024

fixing gunicorn rn, #354

@miararoy miararoy mentioned this issue Nov 3, 2024
7 tasks
@NB-123
Copy link
Author

NB-123 commented Nov 4, 2024

Could you do tokenizer as well please? @miararoy @izellevy

@NB-123
Copy link
Author

NB-123 commented Nov 5, 2024

And tiktoken (and any other commonly used AIML library)? Just run a poetry update - doesn't take long... @miararoy @izellevy

@miararoy
Copy link
Contributor

miararoy commented Nov 6, 2024

@NB-123 poetry update only updates poetry.lock which is not committed, I will run update and commit lock file, which is less of a best practice for a lib like Canopy and will likely break some people. I will wait for feedback on this in the future but will unblock you.

I am also fixing some failing, outdated tests.

@NB-123
Copy link
Author

NB-123 commented Nov 6, 2024

Thanks. When can we expect the release such that we can upgrade poetry via our package manager? @miararoy

@NB-123
Copy link
Author

NB-123 commented Nov 7, 2024

Just checking in again, thanks @miararoy @izellevy

1 similar comment
@NB-123
Copy link
Author

NB-123 commented Nov 12, 2024

Just checking in again, thanks @miararoy @izellevy

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants