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

Support large kmodel #1079

Merged
merged 6 commits into from
Sep 12, 2023
Merged

Support large kmodel #1079

merged 6 commits into from
Sep 12, 2023

Conversation

sunnycase
Copy link
Member

@sunnycase sunnycase commented Aug 31, 2023

  • Upgrade kmodel version to 7
  • Update size/location aware fields' type to uint64_t of kmodel

@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Patch coverage: 97.36% and project coverage change: +0.01% 🎉

Comparison is base (94afeb4) 78.74% compared to head (ac0c3f7) 78.76%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1079      +/-   ##
==========================================
+ Coverage   78.74%   78.76%   +0.01%     
==========================================
  Files         659      659              
  Lines       23307    23319      +12     
==========================================
+ Hits        18354    18368      +14     
+ Misses       4953     4951       -2     
Flag Coverage Δ
integration 41.27% <97.36%> (+0.02%) ⬆️
unit 71.59% <97.36%> (+0.02%) ⬆️

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

Files Changed Coverage Δ
....StackVM/CodeGen/StackVM/StackVMFunctionBuilder.cs 89.74% <ø> (ø)
...s.StackVM/CodeGen/StackVM/StackVMLinkableModule.cs 92.85% <ø> (ø)
src/Nncase.Core/CodeGen/CodeGenTypes.cs 100.00% <ø> (ø)
src/Nncase.Core/CodeGen/IModuleBuilder.cs 100.00% <ø> (ø)
src/Nncase.CodeGen/CodeGenExtensions.cs 72.41% <83.33%> (+0.41%) ⬆️
...StackVM/CodeGen/StackVM/StackVMLinkableFunction.cs 93.33% <100.00%> (ø)
...les.StackVM/CodeGen/StackVM/StackVMLinkedModule.cs 100.00% <100.00%> (ø)
src/Nncase.CodeGen/CodeGen/FunctionBuilder.cs 92.59% <100.00%> (ø)
src/Nncase.CodeGen/CodeGen/LinkableModule.cs 100.00% <100.00%> (ø)
src/Nncase.CodeGen/CodeGen/LinkedFunction.cs 93.33% <100.00%> (ø)
... and 3 more

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.

📢 Have feedback on the report? Share it here.

@github-actions
Copy link

github-actions bot commented Aug 31, 2023

Test Results

5 527 tests   5 527 ✔️  9m 19s ⏱️
       5 suites         0 💤
       5 files           0

Results for commit ac0c3f7.

♻️ This comment has been updated with latest results.

@sunnycase sunnycase merged commit d388629 into master Sep 12, 2023
20 checks passed
@sunnycase sunnycase deleted the feature/large-kmodel branch September 12, 2023 02:31
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.

None yet

3 participants