-
Notifications
You must be signed in to change notification settings - Fork 323
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
Release canonical "stable" versions (don't use floating tags) #1733
Comments
Hey @macropin the CLI command that you shared should pull the same version Docker container. I agree though, that it is much cleaner to just pin the correct tag and I think we'll look into moving to that direction. Looking at when these were published, I believe everything here should be stable. Are these bugs preventing the server from starting? Are there certain workflows that you're running where I can replicate some of these bugs? Our CI/CD has been in a passing state, and I haven't had many other issues reported, but would like to get to the bottom of why it seems to be in a bad state for you. |
Yes, the lite install using the default config had issues with some of the containers starting due to (IIRC) configuration issues (possibly missing env vars). The full version did a little better. The r2r dashboard also had a consistent issue with the logs viewer not working, and there were some other inconsistent issues where agent / chat would break for no apparent reason. Graph generation never worked for either. |
Can you share some specs around your OS and environment? |
It should be nothing unusual. I'm running Jammy |
It would also be helpful to use git tags to track releases. I've noted that Also the release does seems broken for me. Running the following: python3 -m venv venv
./venv/bin/pip install r2r==3.3.24
cp venv/lib/python3.10/site-packages/core/configs/full_local_llm.toml ./
./venv/bin/r2r serve --docker --full --config-path=/home/andrew/code/r2r/full_local_llm.toml Gives the following result
|
Thanks @macropin, sorry for not following up here sooner. We'll be moving to canonical versions on images soon. This is also a great catch—we were just about to cut a release, so I've gone ahead and fixed the import of There should be some bug fixes here that might be related to the streaming issues that you saw. I've also made some improvements to the graph generation process, that may have been the root of your troubles if you were using local models. This might be of interest: https://r2r-docs.sciphi.ai/cookbooks/local-llms#extracting-entities-and-relationships Overall, from what we've seen and heard from users, the system should be in a good state. Please let me know if you have any issues with the latest. |
Thanks for your help here. It's mostly working now with 3.3.27, and I've been able to kick the tyres and see what it's all about! A few issues I've spotted so far:
|
TLDR, tried installing via Docker and almost everything is broken. Too many bugs to report. Need a stable release that encapsulates all dependencies.
The Docker installation method (
r2r serve --docker
) uses floating tags for all containers, including 3rd party containers. This means there is no stable release bundle that encapsulates all dependencies, and it's possible that two installs might pull down different container versions, since it's literally the 'latest' container in the registry, not a requested version:Please consider updating the compose configurations to use known stable and tested versions of all dependencies.
The text was updated successfully, but these errors were encountered: