Skip to content

Conversation

@BenMendel
Copy link

Added a bunch of doc

@BenMendel
Copy link
Author

Hey @jrdnwrth!
just checking if you got the chance to see my improvements on the project.

@jrdnwrth
Copy link
Owner

jrdnwrth commented Jan 8, 2024

Hi @BenMendel,

Sorry I missed this update! I like what you have done. There were a lot of changes.

You seem to understand the library better than I do. Would you like to own the project? If so, I would change my readme.md to point to your project so newcomers know where to go.

I don't know why it threw an AccessViolationException for you when trying to build as x64. I'm on Windows 10 (22H2) 64 bit. My nicaiu.dll is version 19.5.0.49152. I was not able to reproduce this issue.

@BenMendel
Copy link
Author

Hi @jrdnwrth!

Thanks for the complements 😄
Sure, I'd like to own this repository. Thank you!

@jrdnwrth
Copy link
Owner

Hi @BenMendel

I updated my repo to point to yours. Enjoy!

Thank you.

- Updated solution to include "NetDaqmx.Tests" project.
- Added global using directive for Xunit in GlobalUsings.cs.
- Created NetDaqmx.Tests.csproj with necessary testing dependencies.
- Implemented NetDaqMxTests class with WritePort0Test and AnalogInputTest methods to validate NIDAQ operations.
Copy link
Owner

@jrdnwrth jrdnwrth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants