deneland / streamlit-labelstudio

A Streamlit component that provides an annotation interface using the LabelStudio Frontend.
MIT License
93 stars 16 forks source link

The component is unstable #4

Open Usif-A opened 3 years ago

Usif-A commented 3 years ago

It sometimes crops out the submit button, it loads only half the picture sometimes, and recently just I run the streamlit file and the label studio front end didn't even load (the screen was blank)

fathimanourin commented 3 years ago

hey @Usif-A I am working with the package too, would love to collaborate with you !

Usif-A commented 3 years ago

dear Fathimanourin

oh sorry I can't help. I have too many commitments right now. I worked around the issue by having my code open label studio in another tab instead of using the package. another solution I looked into was having an iframe in the streamlit app.

have a great day, Youssif Ahmed

On Wed, 16 Jun 2021 at 16:11, fathimanourin @.***> wrote:

hey @Usif-A https://github.com/Usif-A I am working with the package too, would love to collaborate with you !

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/deneland/streamlit-labelstudio/issues/4#issuecomment-862414668, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATXQ7WCSFY7IBENJECLHHW3TTCWIHANCNFSM46TKO46A .

fathimanourin commented 3 years ago

oh sure, I'll try these, thanks anyways !

mathmanu commented 3 years ago

another solution I looked into was having an iframe in the streamlit app.

Could you elaborate on that please. I want to have label studio in the same tab.

mathmanu commented 3 years ago

I found that providing a key avoids the cropping issue: https://github.com/rchuzh99/streamlit-labelstudio/blob/master/streamlit_labelstudio/__init__.py#L44

Can you add **kwargs at the end of https://github.com/deneland/streamlit-labelstudio/blob/master/streamlit_labelstudio/__init__.py#L43 https://github.com/deneland/streamlit-labelstudio/blob/master/streamlit_labelstudio/__init__.py#L44

So that additional arguments can be passed?

Thanks.

mathmanu commented 3 years ago

@deneland @w0ut0 Could you add the above enhancement and update the pip package please?

helenkimber commented 2 years ago

Did this update ever happen? I want to run two instances of this component so need to pass the key argument.