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

Doc #818

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Doc #818

wants to merge 7 commits into from

Conversation

ccl-core
Copy link
Contributor

No description provided.

ccl-core and others added 7 commits February 21, 2025 10:40
you might have noticed the HF space that holds the croissant editor was
out of date and is now completely broken. I've been trying to figure out
why - it's a dependency issue with streamlit-nested-layout, but it's
impossible to replicate in local tests in the docker container or even
in venvs.

locally, version 0.1.2 of the library has a similar error with
up-to-date streamlit versions, and I'm thinking that the HF space (when
it creates the container) has some sort of caching that means it's
loading 0.1.2 rather than 0.1.4 (which works as intended).

So, this PR is just a pin in requirements.txt and an instruction to not
cache in Dockerfile to maybe avoid that problem. Locally this works, but
it seems it's a dice roll on whether it will work on HF.

NOTE - commits to editor files here do not change the HF space. 

LONG TERM - should remove dependency on streamlit-nested-layout, which
is serviced by two people and doesn't offer tons of value. I guess we
might move away from streamlit anyway.
…tly referenced by Hugging Face. (#814)

contact Pierre Marcena or Neil Majithia for info
Copy link

MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅

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.

3 participants