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 zeroarray for heterogeneous axes #182

Merged
merged 2 commits into from
Jun 17, 2022
Merged

Conversation

timholy
Copy link
Member

@timholy timholy commented Jun 16, 2022

Previously ZeroArrayPromise required homogeneous axes,
this supports promotion prior to construction.
The test requires BlockArrays 0.16.1718 (specifically
JuliaArrays/BlockArrays.jl#213)
but it's not a dependency of the package, so let's just
trust in the lack of conflicts.

Previously `ZeroArrayPromise` required homogeneous axes,
this supports promotion prior to construction.
The test requires BlockArrays 0.16.17 (specifically
JuliaArrays/BlockArrays.jl#213)
but it's not a dependency of the package, so let's just
trust in the lack of conflicts.
@codecov
Copy link

codecov bot commented Jun 16, 2022

Codecov Report

Merging #182 (2b58b47) into master (9e64fec) will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #182      +/-   ##
==========================================
+ Coverage   62.60%   62.67%   +0.06%     
==========================================
  Files          10       10              
  Lines         583      584       +1     
==========================================
+ Hits          365      366       +1     
  Misses        218      218              
Impacted Files Coverage Δ
src/stackedviews.jl 79.48% <100.00%> (+0.26%) ⬆️

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 9e64fec...2b58b47. Read the comment docs.

@timholy
Copy link
Member Author

timholy commented Jun 16, 2022

Let's try it now that BlockArrays 0.16.18 is out.

@timholy timholy closed this Jun 16, 2022
@timholy timholy reopened this Jun 16, 2022
@timholy timholy merged commit cfb2a9b into master Jun 17, 2022
@timholy timholy deleted the teh/zeroarray_hetero_ax branch June 17, 2022 08:41
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.

2 participants