-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
Labeled comprehension channel axis #381
base: main
Are you sure you want to change the base?
Labeled comprehension channel axis #381
Conversation
Can one of the admins verify this patch? Admins can comment |
add to allowlist
gpuCI ***@***.***> schrieb am Fr. 12. Juli 2024 um 14:58:
… Can one of the admins verify this patch?
Admins can comment ok to test to allow this one PR to run or add to
allowlist to allow all future PRs from the same author to run.
—
Reply to this email directly, view it on GitHub
<#381 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AC7SWBB3O7L7RKIFQYAIZJDZL7HIDAVCNFSM6AAAAABKY4IG56VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMRVGUZDQMBYGA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
@joshua-gould I'd agree that supporting a channel axis could be useful here. However I've just seen that this isn't implemented upstream in Would you have opinions here @GenevieveBuckley @jakirkham ? |
Could we try raising an issue upstream with SciPy and see if there is interest in this feature? Maybe this is something @joshua-gould could help with 🙂 |
Fixed up merge conflicts here to pick up recent CI fixes Hope that is ok Please feel free to tweak further as needed |
Support trailing channel axis in labeled_comprehension