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

Features/347 average #352

Merged
merged 28 commits into from
Aug 2, 2019
Merged

Features/347 average #352

merged 28 commits into from
Aug 2, 2019

Conversation

ClaudiaComito
Copy link
Contributor

Description

Addresses Issue #347

Unlike mean(), average() allows to calculate a weighted average of a tensor, or along a specific axis.

The implementation is equivalent to numpy.average().

Averaging across multiple axes (axis is tuple) not supported yet (see Issue #351)

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Have you handled and tested all split configurations?
Yes.

@codecov-io
Copy link

codecov-io commented Aug 1, 2019

Codecov Report

Merging #352 into master will increase coverage by 0.03%.
The diff coverage is 99.25%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #352      +/-   ##
==========================================
+ Coverage   96.96%   96.99%   +0.03%     
==========================================
  Files          53       53              
  Lines        8917     9051     +134     
==========================================
+ Hits         8646     8779     +133     
- Misses        271      272       +1
Impacted Files Coverage Δ
heat/core/dndarray.py 95.37% <100%> (+0.01%) ⬆️
heat/core/tests/test_statistics.py 100% <100%> (ø) ⬆️
heat/core/statistics.py 96.66% <97.72%> (+0.13%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b69ecfc...17fe219. Read the comment docs.

@ClaudiaComito ClaudiaComito merged commit 0dc41cf into master Aug 2, 2019
@ClaudiaComito ClaudiaComito deleted the features/347-average branch August 2, 2019 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants