Skip to content

Index traits#410

Open
Tokazama wants to merge 4 commits into
masterfrom
index-traits
Open

Index traits#410
Tokazama wants to merge 4 commits into
masterfrom
index-traits

Conversation

@Tokazama

Copy link
Copy Markdown
Member

This gets a good chunk of the methods that can be made independent of Static.jl but
were in StaticArrayInterface.jl.

  • is_splat_index
  • known_length: the followning are necessary to get known_length to work
    • known_size
    • known_first
    • known_last
    • known_step

Tokazama added 2 commits May 13, 2023 06:54
This gets a good chunk of the methods that can be made independent of Static.jl but
were in StaticArrayInterface.jl.

- is_splat_index
- known_length: the followning are necessary to get known_length to work
    - known_size
    - known_first
    - known_last
    - known_step
@codecov

codecov Bot commented May 13, 2023

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 2.09790% with 140 lines in your changes missing coverage. Please review.
✅ Project coverage is 23.48%. Comparing base (796eb67) to head (116da7d).
⚠️ Report is 181 commits behind head on master.

Files with missing lines Patch % Lines
src/ArrayInterface.jl 2.56% 114 Missing ⚠️
ext/ArrayInterfaceStaticArraysExt.jl 0.00% 10 Missing ⚠️
ext/ArrayInterfaceOffsetArraysExt.jl 0.00% 6 Missing ⚠️
ext/ArrayInterfaceStaticArraysCoreExt.jl 0.00% 5 Missing ⚠️
ext/ArrayInterfaceStaticExt.jl 0.00% 5 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (796eb67) and HEAD (116da7d). Click for more details.

HEAD has 31 uploads less than BASE
Flag BASE (796eb67) HEAD (116da7d)
52 21
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #410       +/-   ##
===========================================
- Coverage   76.55%   23.48%   -53.07%     
===========================================
  Files           7       10        +3     
  Lines         563      694      +131     
===========================================
- Hits          431      163      -268     
- Misses        132      531      +399     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ChrisRackauckas

Copy link
Copy Markdown
Member

I'm not looking at any new things until a round of bug fixes for like a week or two.

@Tokazama

Copy link
Copy Markdown
Member Author

I'm not looking at any new things until a round of bug fixes for like a week or two.

Feel free to ping me if any are applicable to me

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