Eric StJohn
Eric StJohn
Have a look at https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.defaultdllimportsearchpathsattribute that might help your scenario.
Fixed the XUnit version here -- let's see if that gets us further.
I'm going to abandon this in favor of https://github.com/dotnet/machinelearning/pull/6354. This was on .NET Eng - Latest which moved onto net 8.0 codebase. I'd like to keep us on net 7.0...
I haven't yet fixed the tests here, needed to get some fresh results.
@eerhardt have you thought of this before or are you aware of any discussion with Prasanth about it? If we have an idea of how it would work we could...
Thank you @michaelgsharp for making this change. Can we close this issue now? One thought here that we could consider in the future is flowing TorchSharp through darc, the way...
I'll add that we can workaround this by using `Assert.Throws(Action)` but we'd actually like to Assert a `null` parameter name, since it's clear that we didn't forget to test it....
Looks like aspnetcore is hitting this as well. Here's a repro of how they hit it: ```c# [Theory] [InlineData(0)] [InlineData(null)] public void Test1(long? aNumber) { } ```
So if I can summarize the state here, @richlander and @agocke are saying that this PR should not be needed due to the changes to how we expect distro's to...
Closing this stale PR for now