search

curiosity-ai / catalyst

🚀 Catalyst is a C# Natural Language Processing library built for speed. Inspired by spaCy's design, it brings pre-trained models, out-of-the box support for training word and document embeddings, and flexible entity recognition models.
MIT License
743 stars 75 forks source link

Mosaik.Core has a dependency on .NET 5 Microsoft.Extensions.Logging fails in Azure Funcs #65

Closed williware closed 3 years ago

williware commented 3 years ago

Describe the bug Azure Functions running under .Net Core 3.1 applications utilizing Catalyst Nuget package are unable to load due the dependency Mosaik.Core has on ,net 5 Microsoft.Extensions.Logging.

To Reproduce build an Azure Func in VS 2019 with Core Tools Version: 3.0.3904 and Function Runtime Version: 3.3.1.0 and add the Catalyst Nuget Package. Reference Catalyst in the code. Try to debug the function in VS 2019. The runtime fails to start. The runtime fails with A host error has occurred during startup operation '7b34dd36-ae2b-4d81-8322-302f63c2cf02'. [2021-11-04T03:01:37.263Z] System.Private.CoreLib: Could not load file or assembly 'Microsoft.Extensions.Logging.Abstractions, Version=5.0.0.0, Culture=neutral, PublicKeyToken=adb9793829ddae60'. The system cannot find the file specified.

Expected behavior without a reference to .net 5 the code would load and run successfully_

Additional context image

theolivenbaum commented 3 years ago

Hi @williware, thanks for reporting it - it's quite annoying that microsoft is so behind with their own usage of .NET... I've bumped back the version of Microsoft.Extensions.Logging.Abstractions to 2.1.1 on the Mosaik.Core package, you should expect a new version of Catalyst in a few hours with the updated package!

Let me know if it works!

theolivenbaum commented 3 years ago

@williware: Should be fixed with https://www.nuget.org/packages/Catalyst/1.0.23175, can you test it? Will close for now, feel free to reopen the issue if you hit anything!

williware commented 3 years ago

I will download and give it a try.

Thanks

On Fri, Nov 5, 2021 at 8:01 AM theolivenbaum @.***> wrote:

@williware https://github.com/williware: Should be fixed with https://www.nuget.org/packages/Catalyst/1.0.23175, can you test it? Will close for now, feel free to reopen the issue if you hit anything!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/curiosity-ai/catalyst/issues/65#issuecomment-961838066, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAD4WOT5DPOKG6HHMEGWXZ3UKPBSDANCNFSM5HKOKJ6A . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.