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

Goal: get to 100% code coverage for VectorizationBase.jl #24

Open
DilumAluthge opened this issue Dec 12, 2020 · 3 comments
Open

Goal: get to 100% code coverage for VectorizationBase.jl #24

DilumAluthge opened this issue Dec 12, 2020 · 3 comments
Labels
help wanted Extra attention is needed

Comments

@DilumAluthge
Copy link
Member

DilumAluthge commented Dec 12, 2020

From @chriselrod's comment here:

VectorizationBase.jl also needs a lot more tests. It has <50% coverage at the moment.

For context, here is the current code coverage:

Codecov

@chriselrod chriselrod added the help wanted Extra attention is needed label Dec 15, 2020
@chriselrod
Copy link
Member

I'll try and get it to at least 90%.

I added a help wanted on the off chance there's anyone out there who for some reason enjoys writing unit tests.

@DilumAluthge DilumAluthge changed the title VectorizationBase needs more tests Goal: get to 100% code coverage for VectorizationBase.jl Jan 4, 2021
@DilumAluthge
Copy link
Member Author

Since VectorizationBase.jl is so important to the LoopVectorization.jl stack, I think it would be good to get to 100% code coverage.

If you can get it to 90%, I can work on the last 10%.

It would also be good to set the 100% goal for the other important packages in the LoopVectorization.jl ecosystem.

@chriselrod
Copy link
Member

Sounds like a deal. I don't recall what the coverage was exactly when you filed the issue, but apparently <50%, so I've been making some progress.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants