search

BroncBotz3481 / YAGSL-Lib

YAGSL-Vendor Library
Apache License 2.0
7 stars 3 forks source link

OFFSEASON: Rewrite most of YAGSL in C++ then use JNI #3

Open thenetworkgrinch opened 1 year ago

thenetworkgrinch commented 1 year ago

YAGSL currently runs slower than it should because of slow java calls and jumping. Most of YAGSL can and should be rewritten into C++ then mapped to their current locations using JNI. This will also expand YAGSL to support C++ teams and be the only "mainstream" swerve library that supports C++.

When this change is made development will take place solely in the vendordep repository (YAGSL-Lib) due to it's support of JNI compilation and delivery to teams. This development will only take place after v1 released. All further development will be done in C++ if possible.

DanPeled commented 2 weeks ago

Im currently working on this exact thing, havent done any JNI stuff tho, only implementation of the classes in C++, like 30% done

thenetworkgrinch commented 2 weeks ago

I can invite you to the in dev version rn, it hasnt been touched in a few months though

DanPeled commented 2 weeks ago

Ive just been working here: https://github.com/DanPeled/YAGSL-CPP/tree/2024 i think ill stick with my repo, mostly interested to learn how YAGSL works by myself

thenetworkgrinch commented 2 weeks ago

I just shared the development repo with you so you can read and use it.

DanPeled commented 2 weeks ago

Tbh seems like mine is in a somewhat more developed state currently, ill look into it tho, thanks