search

EPFLRocketTeam / SupersonicTestRocket

Repository for the avionics going on the supersonic test rocket
3 stars 1 forks source link

Altimax wrapper #33

Closed joshdboss closed 2 years ago

joshdboss commented 2 years ago

Finished the Altimax wrapper. Must of course be tested, but it should now be compatible with the main branch.

joshdboss commented 2 years ago

@spacewolfXfr can you please double-check that there are no conflicts regarding the documentation? When I made this branch a few months ago, I had started playing around with documenting with Doxygen. Since you recently included it, I removed it from this old branch but may have forgotten some things. As you have just done this and it's fresh in your mind, you likely remember the files associated to Doxygen better than I do and there might be 1-2 files that I missed.

Thanks!

spacewolfXfr commented 2 years ago

@joshdboss I don't know the purpose of the files in data_analysis/.idea/ (I guess it is Sphinx documentation, but not sure) Otherwise everything seems correct. I'll document the Wrappers in Doxygen style (or you can take inspiration from the other Wrappers, I will merge my branch soon).

joshdboss commented 2 years ago

I think the .idea folder is since I am using PyCharm for Python, although I'm not 100% certain. I think everything should be fine though. Regarding the wrappers, I can also document them if you would like, but it won't be before next weekend I think. Alternatively, I can go over what you wrote and see if there's anything I need to add.