Skip to content

ux improvements#37

Closed
TheTripleV wants to merge 1 commit into
robotpy:mainfrom
TheTripleV:goodux
Closed

ux improvements#37
TheTripleV wants to merge 1 commit into
robotpy:mainfrom
TheTripleV:goodux

Conversation

@TheTripleV

Copy link
Copy Markdown
Member
  • Add --only flag - work with a single project, ignore dependencies

  • Add --till flag - work with a project and its dependencies

  • Make rdev install its dependencies instead of erroring out

  • Make rdev install robotpy build instead of erroring out

  • Add progress bar
    This is buggy for some reason. It worked on allrobotpy. But here, the progress bar keeps getting reprinted and doesn't just show up once at the bottom of the screen.

@@ -0,0 +1,4 @@
# file generated by setuptools_scm

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Probably should add this to .gitignore

Comment thread devtools/__main__.py
from .progress import progress
except (ImportError, ModuleNotFoundError):
print("Installing robotpy dev requirements...")
subprocess.check_call(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Definitely do not install the requirements automatically. Just detect the first import error and output the appropriate command for the user to copy/paste, but do not execute it.

Comment thread rdev.toml

[params]

parallel = true

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm concerned about these interacting poorly with CI. Maybe have a user-specific file that is added to .gitignore instead?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The default should be parallel.

Also, if ci sets an environment variable, this won't override it.

Cache should also be on by default imo. But I do need to check if ccache is available and not use it if it doesn't exist.

@virtuald

Copy link
Copy Markdown
Member

I think most of this is OBE due to semiwrap.

@virtuald virtuald closed this Jul 16, 2025
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.

2 participants