Skip to content
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.

Sign up for GitHub

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

Jump to bottom

refactor: Rename VisionControllerClient to VisionClient #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

refactor: Rename VisionControllerClient to VisionClient #46

wants to merge 1 commit into from

Commits on Aug 29, 2023

  1. refactor: Rename VisionControllerClient to VisionClient 

    This is a second attempt to merge #39 .

However I'm not sure if we can actually make such changes. Since they are not backwards compatible. E.g. `from mujinvisioncontrollerclient import visioncontrollerclient` will not work anymore.

Maybe we should add alternative class name without renaming the file. Or go all in and rename the module completely.
    @andriymujin
    andriymujin committed Aug 29, 2023
    Configuration menu
    Copy the full SHA
    69066f6 View commit details
    Browse the repository at this point in the history