Closed mrfh92 closed 2 months ago
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Thank you for the PR!
Attention: Patch coverage is 94.28571%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 91.91%. Comparing base (
a774559
) to head (b388446
). Report is 2 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
heat/core/vmap.py | 94.11% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Thank you for the PR!
@ClaudiaComito problem should be fixed now. In fact, we require PyTorch >= 2.0.0 for vmap as vmap is only available in these versions.
Thank you for the PR!
@ClaudiaComito codecov was a bit lower than expected by adding the uncovered line (~-3%), so I improved coverage
Thank you for the PR!
Thank you for the PR!
@ClaudiaComito @JuanPedroGHM thx for reviewing :+1:
Due Diligence
Description
Issue/s resolved: #1383 The implemented vmap-functionality is similar to the one of PyTorch. It allows vectorized application of PyTorch-functions to one or multiple DNDarrays with vectorization taking place along the respective split axes.
An artificial example, how vmap does work, is found in the tests.
Type of change
new feature
Does this change modify the behaviour of other functions? If so, which?
no