fix: make sure pillow is not partially upgraded - #7
Open
AngeloDanducci wants to merge 2 commits into
Open
AngeloDanducci wants to merge 2 commits into
AngeloDanducci wants to merge 2 commits into
Conversation
Signed-off-by: AngeloDanducci <[email protected]>
psschwei
reviewed
Sep 14, 2026
psschwei
left a comment
psschwei
left a comment
Member
There was a problem hiding this comment.
I can't comment on the cell for some reason, but what about doing something like this (using the ICML notebook as an example)
!uv pip install --reinstall-package pillow "mellea[hf,docling,sandbox,tools]==0.6.0"
that keeps it all in one line and may help with future resolution too
Signed-off-by: AngeloDanducci <[email protected]>
Contributor
Author
|
@psschwei I don't think this was solvable via UV - hence the sys.executable call. However in doing some additional investigating around this I found the best way to fix this seems to be just restarting the kernel to pickup updates instead of dealing with a stale cached version (being served via the colab env instead of uv). |
Member
|
I tried to run the ICML notebook in collab and it crashed on me running the first cell a couple of times. Here's the logs:
I'm not really sure what to make of these errors tbh. Also will tag in @jakelorocco and @nrfulton in case they have an alternative viewpoint on whether we should try to maintain these notebooks on collab... |
Contributor
Author
|
Did you try running sequential cells after the "crash"? This May be a better way of doing it though. Edit: assuming you also have the zstd fix in #6 |
Member
|
I did not try running additional cells. Is there any way to avoid or hide the warnings? Feels like that would be a better UX if possible |
Contributor
Author
|
I did some digging and could not find a way to restart it without the warning or suppress it. |
Contributor
Personally, I think the notebooks on collab are useful when doing demos / presentations. If the errors / difficulties are different between the notebooks, we should at least have a |
Member
I like this idea |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes generative-computing/mellea#1640