Skip to content

Conversation

lizlooney
Copy link
Collaborator

Add expansion hub python code.

Improved python tools:
Removed _DICT_FULL_MODULE_NAME_TO_MODULE_NAME. Use .all instead.
Pass the json_generator_robotpy to the constructor for json_generator_external_samples so it can understand the module exports from robotpy.

Renamed SparkMiniComponent to SparkMini.

Removed _DICT_FULL_MODULE_NAME_TO_MODULE_NAME. Use <module>.__all__ instead.
Pass the json_generator_robotpy to the constructor for json_generator_external_samples so it can understand the module exports from robotpy.

Renamed SparkMiniComponent to SparkMini.

Added expansion_hub.py.
@lizlooney lizlooney requested a review from alan412 July 27, 2025 05:41
Copy link
Collaborator

@alan412 alan412 left a comment

Choose a reason for hiding this comment

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

Looks great, but it took me a while to find the expansion hub stuff running because I expected to see it as a component. I think that shouldn't block this PR but should be a new one.

@lizlooney lizlooney merged commit 9a15ee3 into wpilibsuite:main Jul 27, 2025
1 check passed
@lizlooney lizlooney deleted the pr_improve_python_tools branch July 28, 2025 00:19
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