Skip to content

Conversation

@ranocha
Copy link
Member

@ranocha ranocha commented Aug 31, 2017

recursivecopy!now uses copy for StaticArrays, see #19. This should be correct for both mutable and immutable StaticArrays, but performance could probably be improved for mutable ones if copy! can be used.

Could you please release a new version v0.12.2 of RecursiveArrayTools including this fix?

@codecov
Copy link

codecov bot commented Aug 31, 2017

Codecov Report

Merging #20 into master will increase coverage by 0.94%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   61.61%   62.55%   +0.94%     
==========================================
  Files           3        3              
  Lines         211      211              
==========================================
+ Hits          130      132       +2     
+ Misses         81       79       -2
Impacted Files Coverage Δ
src/utils.jl 46.37% <100%> (+2.89%) ⬆️

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 6d304a5...51ccc06. Read the comment docs.

@coveralls
Copy link

coveralls commented Aug 31, 2017

Coverage Status

Coverage increased (+0.9%) to 62.559% when pulling 51ccc06 on ranocha:pull-request/51ccc062 into 6d304a5 on JuliaDiffEq:master.

@ChrisRackauckas ChrisRackauckas merged commit 7346d2c into SciML:master Aug 31, 2017
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