YAGSL-Example icon indicating copy to clipboard operation
YAGSL-Example copied to clipboard

Integration of AdvantageKit

Open falOn-Dev opened this issue 1 year ago • 10 comments

I was wondering if there had been any thoughts of integrating advantagekit into YAGSL, I'd even be willing to contribute to it and use my team's bot to test it if necessary.

falOn-Dev avatar Dec 21 '23 16:12 falOn-Dev

I think it's something that would be really cool to incorporate into YAGSL.

I haven't looked into Advantage Kit but I've heard it complicates the robot project. YAGSL sits in a difficult spot. It should be easy to get into and modify but have a high enough ceiling that a more advanced team can get more performance out of it.

Technologyman00 avatar Dec 21 '23 17:12 Technologyman00

I still am just looking into it for the first time, but it certainly does complicate a lot of stuff. Although I think it could end up being harder having it set up in a way that doesn't force the team to use advantagekit

falOn-Dev avatar Dec 21 '23 17:12 falOn-Dev

That is my conclusion as well, I have talked with the AdvantageKit devs before about it and we have concluded YAGSL is possible to integrate into AdvantageKit however it would require users to call functions synchronously with periodic's and modify the SwerveTelemtry too. I dont think its worth adding bc i want YAGSL to remain first and foremost, a library NOT a template.

thenetworkgrinch avatar Dec 21 '23 21:12 thenetworkgrinch

If you don't mind then, I may fork YAGSL and mess with implementing it myself

falOn-Dev avatar Dec 22 '23 03:12 falOn-Dev

Go ahead, and Good luck! Its a pain maintaining a fork though but itd be a good reference for teams.

thenetworkgrinch avatar Dec 22 '23 03:12 thenetworkgrinch

I'd like to reopen this issue now that AdvantageScope is an official WPILIB Tool. I feel like it should be pretty basic to implement just a basic SwerveDriveIO Class, since a lot of abstraction is already done for motors and the like

falOn-Dev avatar Jan 19 '24 02:01 falOn-Dev

AdvantageScope != AdvantageKit however it is a consideration I still do not want the user to have to do anything other than include the library.

thenetworkgrinch avatar Jan 19 '24 02:01 thenetworkgrinch

This is being worked on as a PR, not by us. The PR has an idea and will be reviewed and merged provided advantagekit-less projects still function

thenetworkgrinch avatar Jun 14 '24 15:06 thenetworkgrinch

Which PR is this? I didn't see it in the list, but I may have just missed it. I'd love to take a look.

toddas90 avatar Jan 12 '25 22:01 toddas90

It was being made by @falOn-Dev but i think it has been dropped

thenetworkgrinch avatar Jan 13 '25 05:01 thenetworkgrinch