-
Notifications
You must be signed in to change notification settings - Fork 6
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
make categories searchable #232
Conversation
for more information, see https://pre-commit.ci
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #232 +/- ##
==========================================
+ Coverage 86.52% 86.62% +0.10%
==========================================
Files 18 19 +1
Lines 1603 1608 +5
==========================================
+ Hits 1387 1393 +6
+ Misses 216 215 -1 ☔ View full report in Codecov by Sentry. |
Hey @haesleinhuepf Yes, it does, but very minimalist has we just change the name when we write the file: |
Also I did an pyclesperanto/pyclesperanto/_functionalities.py Lines 266 to 269 in 5382c42
dont hesitate to remove it |
Just a note: the pre-commit is super annoying, because it is obviously a post-commit. I'm running into merge conflicts all the time. |
for more information, see https://pre-commit.ci
The |
Hey Stephane @StRigaud ,
this change is required to make the napari-assistant find the right functions. Under the hood, I read out all functions, that claim to be assistant-compatible like this
list(cle.operations("in assistant").keys())
I had to modify this:
to this in many places:
Does this require a change in a code-generator? If so, where?
Let me know what you think! (Or feel free to merge this).
Best,
Robert