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.

Sign up for GitHub

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

Jump to bottom

TODO: Tier 1 kernel implementations #120

Closed
StRigaud opened this issue Nov 28, 2022 · 7 comments
Closed

TODO: Tier 1 kernel implementations #120

StRigaud opened this issue Nov 28, 2022 · 7 comments
Labels
help wanted Extra attention is needed WIP Work In Progress

Comments

@StRigaud
Copy link
Member

StRigaud commented Nov 28, 2022 •
edited
Loading

List of kernel to be implemented in CLIc, based from the pyclesperanto-prototype list.

If the link return a 404 error, it means that the kernel is not yet available for the clespernanto-kernel branch. See this issue for more info.

Please create a branch or a PR draft for the development of new kernel so that we can see those which are under development.

Kernels with a ❓ next to them, means we do not know yet how relevant it is to keep them.
Kernels with a ❗ next to them, means we need to update their .cl code before implementation.

Tiers 1
@StRigaud StRigaud added help wanted Extra attention is needed WIP Work In Progress labels Nov 28, 2022
@StRigaud
Copy link
Member Author

For good example for Tier 1 contributions, see the following PRs:

I will try to build up a documentation from these later on.

@StRigaud
Copy link
Member Author

I have started a documentation on how to create new kernels to CLIc

@StRigaud StRigaud mentioned this issue Nov 30, 2022
Closed
@StRigaud
Copy link
Member Author

Update of T1 implemented kernel list

WARNING: some kernel have not been tested and/or are missing their test function

@StRigaud
Copy link
Member Author

@haesleinhuepf can you have a look a the kernel not yet implemented in the list and tell me which one we still should implement and those we should not because out of scope?

Thanks 😃

@haesleinhuepf
Copy link
Member

I think those would be useful:

@StRigaud
Copy link
Member Author

All 5 filters have been pushed (with tests), as well as the hessian eigenvalues filter.

@StRigaud
Copy link
Member Author

closing for now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Extra attention is needed WIP Work In Progress
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@StRigaud @haesleinhuepf