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

Update TorchSharp to latest version #6636

Merged
merged 2 commits into from
Apr 29, 2023

Conversation

michaelgsharp
Copy link
Member

Updates TorchSharp to the latest version and fixes any of the breaking changes.

@michaelgsharp
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@codecov
Copy link

codecov bot commented Apr 29, 2023

Codecov Report

Merging #6636 (8cf501c) into main (2ede226) will increase coverage by 0.14%.
The diff coverage is 86.51%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6636      +/-   ##
==========================================
+ Coverage   68.38%   68.52%   +0.14%     
==========================================
  Files        1176     1200      +24     
  Lines      248240   250287    +2047     
  Branches    25917    26093     +176     
==========================================
+ Hits       169754   171516    +1762     
- Misses      71710    71937     +227     
- Partials     6776     6834      +58     
Flag Coverage Δ
Debug 68.52% <86.51%> (+0.14%) ⬆️
production 63.00% <87.36%> (+0.24%) ⬆️
test 88.83% <63.95%> (-0.05%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...icrosoft.ML.TorchSharp/NasBert/Models/BaseModel.cs 66.66% <ø> (ø)
...osoft.ML.TorchSharp/NasBert/Models/NasBertModel.cs 89.58% <ø> (+1.82%) ⬆️
...oft.ML.TorchSharp/NasBert/Models/PredictionHead.cs 100.00% <ø> (ø)
...ML.TorchSharp/NasBert/Models/TransformerEncoder.cs 70.00% <ø> (ø)
...Microsoft.ML.TorchSharp/NasBert/Optimizers/Adam.cs 100.00% <ø> (ø)
src/Microsoft.ML.TorchSharp/Utils/HashHelpers.cs 0.00% <0.00%> (ø)
test/Microsoft.ML.Tests/TextClassificationTests.cs 86.01% <0.00%> (-8.75%) ⬇️
src/Microsoft.ML.TorchSharp/Utils/Index.cs 29.41% <29.41%> (ø)
src/Microsoft.ML.TorchSharp/Utils/Range.cs 31.57% <31.57%> (ø)
src/Microsoft.ML.TorchSharp/TorchSharpCatalog.cs 77.77% <71.42%> (-22.23%) ⬇️
... and 26 more

... and 4 files with indirect coverage changes

@JakeRadMSFT JakeRadMSFT merged commit 3386245 into dotnet:main Apr 29, 2023
@ghost ghost locked as resolved and limited conversation to collaborators May 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants