-
Notifications
You must be signed in to change notification settings - Fork 29
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
Discussion: Extensions vs Hardcoded for WPI components. #508
Comments
I believe I may have found that limitation. Seems that an extension does not "Currently" have knowledge of the macros with in the PalletteDescription.yanl file #Button("")
I suppose the Extension could recreate the required code but that violates -DRY- |
What is the advantage of doing everything as extensions? |
I'm not sure there is or is not an advantage. Possibly more people would develop components if they had templates as examples. Sure most devices we use fall into a hard-coded. If users have an extention that would fit inclusion to the project it would be easier for them to submit them to a central extention repo verses cloning the project and inserting the bits into the various locations. I point every team and student to a repository of extention I created that are mostly 3rd party enhanced versions of their existing extentions if they have one. Navx and Rev currently don't. I do. |
I was panning through the 2023 changes through the Beta links and the PS4Controller class jumped out at me. Since this is a WPILib class I though about adding it to Robotbuilder like the Xbox Controller. I already created a working Extension and thought would it be possible to include an Extensions folder in the repository to easily add new components as needed vs editing the individual files that make RobotBuilder operate.
Maybe there are other reasons to keep all of the WPILib components coded to the few internal files that I'm not aware of. If that is the case I can move my edits into the necessary files and create a Pull Request.
The text was updated successfully, but these errors were encountered: