Skip to content
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

Driver madness 7: Deep dark #1

Open
wants to merge 121 commits into
base: master
Choose a base branch
from
Open

Driver madness 7: Deep dark #1

wants to merge 121 commits into from

Conversation

okawo80085
Copy link
Member

All socket logic was is now handled by lazy-sockets.

Posers are actually notified about bad packets and posers will get notified on driver exit.
Also improved manager handling.

Split the entire driver from a giant cpp file, into a bunch of small dedicated files for simpler management.

Plus comments and generally nicer code.

Oleg Vorobiov added 30 commits December 4, 2021 22:11
Finally split devices into their own source files, no more  10 billion line files.

Signed-off-by: Oleg Vorobiov <[email protected]>
Added eye first iterations for eye and face tracking devices

Signed-off-by: Oleg Vorobiov <[email protected]>
Added shroom dep submodule for addon IPC

Signed-off-by: Oleg Vorobiov <[email protected]>
Moved device packet structs into it's own header.
Also added a new udu device, to make that happen receiver class had to be modified

Signed-off-by: Oleg Vorobiov <[email protected]>
First working implementation of the GazeMaster device,
still missing some cosmetic changes, but already functional

Signed-off-by: Oleg Vorobiov <[email protected]>
Removed useless paths from the include

Signed-off-by: Oleg Vorobiov <[email protected]>
Fixed a silly typo

Signed-off-by: Oleg Vorobiov <[email protected]>
Added a age_seconds field to HoboVR_GazeState_t

Signed-off-by: Oleg Vorobiov <[email protected]>
Latest Linux build, new devices :P

Signed-off-by: Oleg Vorobiov <[email protected]>
Added GazeMaster device to the last stage of the test

Signed-off-by: Oleg Vorobiov <[email protected]>
…r GazeMaster

Added an initial device model for GazeMaster

Signed-off-by: Oleg Vorobiov <[email protected]>
Fixed json typos and model being too fat

Signed-off-by: Oleg Vorobiov <[email protected]>
Fixed a bug where it's pose won't update

Signed-off-by: Oleg Vorobiov <[email protected]>
Now different platform versions of our code are
handled in the build files while the code can stay generic

Signed-off-by: Oleg Vorobiov <[email protected]>
Updated receiver class and revamped device communication
protocol a bit, posers are finally notified about device list misses

Signed-off-by: Oleg Vorobiov <[email protected]>
Updated controller packet reducing the size by 20 bytes

Signed-off-by: Oleg Vorobiov <[email protected]>
Removed old receiver sources

Signed-off-by: Oleg Vorobiov <[email protected]>
Updated manager message structs as well
as combined all manager structs into one
for more sane message interpretation

Signed-off-by: Oleg Vorobiov <[email protected]>
Updated GazeMaster paket structs

Signed-off-by: Oleg Vorobiov <[email protected]>
Signed-off-by: Oleg Vorobiov <[email protected]>
Because of course i forgot add these when splitting everything...

Signed-off-by: Oleg Vorobiov <[email protected]>
Fixed an edge case when we fail to create shared memory.

Signed-off-by: Oleg Vorobiov <[email protected]>
Incorporated the new receiver class simplifying initial setup by... a lot.

Signed-off-by: Oleg Vorobiov <[email protected]>
…ucts

Renamed a bunch of enums and merged a bunch of message structs.

Signed-off-by: Oleg Vorobiov <[email protected]>
Minor fixes, bumped the version up, removed some unused ifdefs.

Signed-off-by: Oleg Vorobiov <[email protected]>
…g pain

Just read the diff... it's just pain...

Signed-off-by: Oleg Vorobiov <[email protected]>
@okawo80085
Copy link
Member Author

okawo80085 commented Aug 3, 2022

CI builds are currently broken cuz virtual-environments/issues#2667

Will push a dependency fix later

@Minothor
Copy link
Contributor

Minothor commented Aug 3, 2022

if needs be, I can write a hotfix to the ci to pull in the necessary lib manually.

@okawo80085
Copy link
Member Author

Thats pretty much the only way to get it right now, but we can just take the steps for getting it from that github issue

@okawo80085 okawo80085 force-pushed the driver_madness_7 branch 6 times, most recently from 21110dd to d9e0b3e Compare August 4, 2022 00:03
Oleg Vorobiov added 2 commits August 4, 2022 03:19
Added missing boost fetch before build

Signed-off-by: Oleg Vorobiov <[email protected]>
Signed-off-by: Oleg Vorobiov <[email protected]>
@okawo80085 okawo80085 force-pushed the driver_madness_7 branch 9 times, most recently from 7e349a4 to c40f09c Compare August 5, 2022 01:44
Added boost dependency as a submodule

Signed-off-by: Oleg Vorobiov <[email protected]>
No need to download it seperately, its a submodule now

Signed-off-by: Oleg Vorobiov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants