-
Notifications
You must be signed in to change notification settings - Fork 72
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
Bugfix/ Model upload logs #155
Conversation
I will defer to Paul to approve which versions we are supporting. I know previously the < was to prevent ultralytics from pushing breaking changes knowing we would support < Things have presumably settled down over there... |
Maybe we should pin to a specific version and have it match the version we use in the roboflow-model-conversion library. |
i'll pin it to the version in roboflow-model-conversion after I upgrade that ultralytics package |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great!
@@ -593,14 +593,14 @@ def deploy(self, model_type: str, model_path: str) -> None: | |||
|
|||
if self.public: | |||
print( | |||
f"View the status of your deployment at: {APP_URL}/{self.workspace}/{self.project}/deploy/{self.version}" | |||
f"View the status of your deployment at: {APP_URL}/{self.workspace}/{self.project}/{self.version}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, that's a big improvement!
Description
Corrects logging for the model upload feature.
Previous logs asked for ultralytics version < 8.0.20, even though we require a greater version. And they point to the old URL path of
../deploy/..
New logs point to correct path
Type of change
Please delete options that are not relevant.
How has this change been tested, please provide a testcase or example of how you tested the change?
Tested model upload with local package.
Any specific deployment considerations
None.