Skip to content

Cumulative changes from my fork#4

Open
TheNathannator wants to merge 443 commits intoferzkopp:mainfrom
TheNathannator:main
Open

Cumulative changes from my fork#4
TheNathannator wants to merge 443 commits intoferzkopp:mainfrom
TheNathannator:main

Conversation

@TheNathannator
Copy link
Contributor

@TheNathannator TheNathannator commented Apr 19, 2022 •
edited
Loading

Overview of cumulative changes:

  • Switch from Pcap.Net to SharpPcap
    • This is a fully managed packet capture library, which means the Visual C++ redistributables are no longer required.
    • It also has some additional functionality that could be useful in the future for convenience of identifying the receiver.
  • Rewrite packet parsing
  • Add ability to log packets to a file
  • Update the installer:
    • Change platform to x64 and move the install location from Program Files (x86) to Program Files
    • Add start menu shortcut
  • Add application icon
  • Update the readme and packet format docs
  • Other improvements and fixes

This list is not comprehensive, as there are quite a few changes to go through, and a large number of them are overshadowed by future changes.

@TheNathannator TheNathannator marked this pull request as ready for review April 19, 2022 09:37
@TheNathannator
Copy link
Contributor Author

TheNathannator commented May 24, 2022

Updating this PR to include my v2.0.0.0 stuff.

@TheNathannator TheNathannator changed the title Switch from Pcap.Net to SharpPcap, add more proper logging, update packet docs, etc. Rewrite packet parsing, add more proper logging, update packet docs and installer, etc. May 24, 2022
@TheNathannator TheNathannator changed the title Rewrite packet parsing, add more proper logging, update packet docs and installer, etc. Rewrite packet parsing, add packet logging to file, update packet docs and installer, etc. May 24, 2022
@TheNathannator TheNathannator changed the title Rewrite packet parsing, add packet logging to file, update packet docs and installer, etc. Cumulative changes from my fork Aug 7, 2022
@TheNathannator TheNathannator force-pushed the main branch 3 times, most recently from 7451f43 to 0c90797 Compare June 25, 2023 13:26
@TheNathannator
Refactor handling of keystrokes
356a173
@TheNathannator
Fix sequence ID handling
9db468e
@TheNathannator
Client side of message sending
3e5743a
@TheNathannator
Add XboxMessage container for constant messages
7993719
@TheNathannator
Retry a few times on failed packet sends
b71bb7c
@TheNathannator
Reorganize/refactor packet definitions and add missing doc comments
b4e9456
@TheNathannator
Refactor command ID definitions
66afaa4
@TheNathannator
Implement basic initialization sequence
255bae5
@TheNathannator
Rework descriptor parsing to use hash sets
cf4cbd5
@TheNathannator
Respect descriptor's list of output commands
bdb6189
@TheNathannator
Add Xbox prefix to all packet types to prevent name collisions
5906c52
@TheNathannator
Hook up WinUSB backend to UI
ef5442c
@TheNathannator
Fix USB device disconnection
36492fc
@TheNathannator
Better read/write logging
8c0e870
@TheNathannator
of course a single missing character breaks the whole thing lol
b5a2d60
@TheNathannator
Adjust message error logging and log disconnect message
bff6d20
@TheNathannator
More proper client disconnection handling
8b6074c
@TheNathannator
Use the correct endpoint when writing
c7b89d7
@TheNathannator
Move constant message definitions to their respective types
0bce81a
@TheNathannator
Reset USB device when stopping capture and on app exit
e448045 
Fixes Stop blocking until new message is received
@TheNathannator
Fix a couple issues with (dis)connecting devices after starting capture
3e45e42
@cjgammon
Copy link

cjgammon commented Apr 12, 2025

I'm trying to use two Riffmaster guitars on a dedicated Clone Hero Cab, i could only get them to work with this RB4InstrumentMapper however I have to escape out of the Clone Hero so i can hit the start button on the program. Is there anyway the developers can add a click box next the to Start Button to have it auto start the program for those using it in a dedicated cab? Thanks

Same, very interested in a less manual solution.

@cjgammon
Copy link

cjgammon commented Apr 15, 2025

I created a fork & PR allowing it to run without the GUI via command line interface:
#8

cjgammon and others added 19 commits April 15, 2025 21:15
@cjgammon
Add dedicated CLI version (#53)
80517b3 
* add cli

* updates and fixes

* add bat

* fix errors

* update

* fix errors

* update path in bat

* update readme

* update gitignore
@TheNathannator
Overhaul project structure to better accomodate CLI as a separate app…
335674a 
…lication
@TheNathannator
Update building section of the readme to include more detailed instru…
8b84390 
…ctions for MSBuild and the .NET SDK
@TheNathannator
Fix already-connected WinUSB devices ceasing to function after starting
04c61b1
@TheNathannator
Tidy up CLI and various other bits of code
f4ee885
@TheNathannator
Adjust some installer things and make a build script for releases
96fcb16 
CLI is now included in the installer package
@TheNathannator
Bump version to 5.2.0, and unify where version number is specified
b05b9da
@TheNathannator
Bump package versions
5987829
@TheNathannator
Update CLI readme info and adjust --help output
42c64d9
@TheNathannator
Correct settings namespace to match project restructure
57fa984 
Note: may require a manual deletion of the `obj` folder to build correctly.
@TheNathannator
Mark GameInput reading callback failure message as verbose
c5ab206
@TheNathannator
Make it easier to enable gamepad mapping for release builds
a7bed54 
Primarily to assist in performance-related testing
@TheNathannator
Limit polling rate for GameInput backend, with configuration between …
b5b782c 
…60-500 hz
@TheNathannator
Update program screenshot
13cd9be
@TheNathannator
Make XboxMessage<T> covariant with XboxMessage
d1bd52e
@TheNathannator
Add x64 suffix to release artifacts in publish script
2f02e31
@TheNathannator
Remove building of standalone versions from publish script
21bebc0 
Not particularly worth it on second thought, there are additional dependencies the installer handles for you.
@TheNathannator
Tweak tilt scaling
83e0a57
@TheNathannator
v5.2.1
f2d236d
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@TheNathannator @masilu77 @wisker27 @cjgammon @carlmylo