Skip to content

Conversation

@kkeroo
Copy link

@kkeroo kkeroo commented Nov 4, 2025

This pull request refactors and simplifies the codebase by removing unused or redundant code related to online model conversion and the HubAI integration, and by reorganizing imports for better maintainability. It also updates the documentation to clarify the preferred method for online conversion.

Documentation update:

  • The README.md file now recommends using the HubAI SDK for online model conversion, replacing the previous detailed CLI and Python usage instructions.

Code cleanup and refactoring:

  • Removed unused imports and references to the hub module from modelconverter/__init__.py and modelconverter/__main__.py, streamlining the main entry points. [1] [2] [3]
  • Changed the import path for Request to use modelconverter.utils.hub_requests instead of modelconverter.hub.hub_requests in modelconverter/cli/utils.py, reflecting the new module structure.
  • Removed several helper functions related to HubAI resource management and output formatting from modelconverter/cli/utils.py, including print_hub_resource_info, hub_ls, is_valid_uuid, get_resource_id, request_info, get_variant_name, get_version_number, wait_for_export, and get_target_specific_options. This reduces complexity and removes code that is no longer needed. [1] [2]

@kkeroo kkeroo requested a review from a team as a code owner November 4, 2025 12:41
@kkeroo kkeroo requested review from conorsim, klemen1999, kozlov721 and tersekmatija and removed request for a team November 4, 2025 12:41
Copy link
Collaborator

@klemen1999 klemen1999 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are some mentions of the online usage in the main README as well that should be removed

@kkeroo kkeroo marked this pull request as draft November 4, 2025 14:12
Eliminated imports and references to the hub module from __init__.py and __main__.py, likely due to deprecation or refactoring of hub functionality.
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.

4 participants