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

Fix mid for IntervalBox to return SVector #30

Merged
merged 1 commit into from
May 19, 2017
Merged

Conversation

dpsanders
Copy link
Member

No description provided.

@codecov-io
Copy link

codecov-io commented May 13, 2017

Codecov Report

Merging #30 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #30   +/-   ##
=======================================
  Coverage   92.14%   92.14%           
=======================================
  Files          22       22           
  Lines         917      917           
=======================================
  Hits          845      845           
  Misses         72       72
Impacted Files Coverage Δ
src/multidim/intervalbox.jl 81.25% <100%> (ø) ⬆️

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 c0bb130...2e096de. Read the comment docs.

@@ -31,7 +31,7 @@ IntervalBox(x::Interval) = IntervalBox( (x,) ) # single interval treated as tup
## arithmetic operations
# Note that standard arithmetic operations are implemented automatically by FixedSizeArrays.jl

mid(X::IntervalBox) = [mid(x) for x in X]
mid(X::IntervalBox) = mid.(X)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@dpsanders dpsanders merged commit 222d765 into master May 19, 2017
@dpsanders dpsanders deleted the fix_mid_intervalbox branch August 2, 2017 19:40
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.

3 participants