-
Notifications
You must be signed in to change notification settings - Fork 0
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
Simplify dependencies & split CPU/GPU backend Dockerfiles #3
Conversation
|
A bit concerned about removing Omniparser. The file structure that I'm using here is not default and a pure re-install of Omniparser won't restore the file structure. I committed Omniparser because the non-downloaded model files are relatively lightweight. Why is Omniparser being considered unused? Is this for demo purposes? |
Is Omniparser still being used for the main pipeline? I think I mistook it being unused because it was commented out. I'll restore the directory. Also, are only the weights of Omniparser used? We can move Omniparser code into the main source, and leave only model weights in |
I think the configs are also used and I wouldn't remove the license just in case. There's probably some unused files, but this is largely just a restructured directory from Omniparser, so I didn't really try to touch individual files. I can look at reducing the files as a secondary priority after we complete the big action items. |
@deluzhao I've restored OmniParser and the relevant task. |
pyproject.toml
fileuv