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

NTCAN hardware plugin #504

Merged
merged 1 commit into from
Dec 18, 2024
Merged

NTCAN hardware plugin #504

merged 1 commit into from
Dec 18, 2024

Conversation

GwnDaan
Copy link
Member

@GwnDaan GwnDaan commented Nov 8, 2024

Describe your changes

Continued #198

I will create another PR in the coming days that adds an example on how to bridge two hardware plugins together. This allows one to use a virtual bus, while connecting it to (multiple) physical busses as well.

How has this been tested?

I have tested it using their virtual can bus and everything is functioning as it should.

@GwnDaan GwnDaan requested a review from ad3154 November 8, 2024 21:00
@GwnDaan GwnDaan self-assigned this Nov 8, 2024
@GwnDaan GwnDaan added enhancement New feature or request os: windows Related to usage on Windows labels Nov 8, 2024
@GwnDaan GwnDaan force-pushed the daan/ntcan-driver branch 4 times, most recently from 02d0357 to ae43c52 Compare November 28, 2024 08:53
@GwnDaan GwnDaan marked this pull request as ready for review November 28, 2024 09:00
@GwnDaan
Copy link
Member Author

GwnDaan commented Dec 7, 2024

Hi all, any chance someone has time to review this PR for me? I'd like to continue work that depends on the contributed code

@ad3154
Copy link
Member

ad3154 commented Dec 9, 2024

I think my main concern from before was just that I didn't know if we could include ntcan stuff due to license issues.

It looks like you're not adding ntcan.h or the .lib file - so I guess, what's your plan here? If the end goal is just that we want to provide a virtual bus that works cross-process, we might want to develop something home-grown for that, IDK

@GwnDaan GwnDaan merged commit 68bc015 into main Dec 18, 2024
11 checks passed
@GwnDaan GwnDaan deleted the daan/ntcan-driver branch December 18, 2024 22:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request os: windows Related to usage on Windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants