8000 Fix check if plugin is available from conda by Czaki · Pull Request #6545 · napari/napari · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix check if plugin is available from conda #6545

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

Merged
merged 1 commit into from
Dec 31, 2023

Conversation

Czaki
Copy link
Collaborator
@Czaki Czaki commented Dec 19, 2023

Description

I found that on current main, my plugin is not available from the conda, even when it is.

I found that it happens because the plugin is named PartSeg, so it normalized name is parsteg. As we normalize plugin names but https://npe2api.vercel.app/api/conda serves not normalized names, the check fails even if should not.

In this PR, I create a set of normalized names to not false exclude plugins.

@Czaki Czaki added the bugfix PR with bugfix label Dec 19, 2023
@Czaki Czaki added this to the 0.5.0 milestone Dec 19, 2023
@Czaki Czaki requested a review from DragaDoncila December 19, 2023 13:31
Copy link
codecov bot commented Dec 19, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (d507dd6) 92.29% compared to head (95fa6f7) 92.24%.
Report is 2 commits behind head on main.

Files Patch % Lines
napari/plugins/npe2api.py 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6545      +/-   ##
==========================================
- Coverage   92.29%   92.24%   -0.06%     
==========================================
  Files         601      601              
  Lines       53687    53688       +1     
==========================================
- Hits        49553    49527      -26     
- Misses       4134     4161      +27     

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

@Czaki Czaki added the ready to merge Last chance for comments! Will be merged in ~24h label Dec 27, 2023
@Czaki Czaki merged commit a30074f into napari:main Dec 31, 2023
@Czaki Czaki deleted the fix_conda_availability_check branch December 31, 2023 15:57
@Czaki Czaki removed the ready to merge Last chance for comments! Will be merged in ~24h label Dec 31, 2023
kne42 added a commit to kne42/napari that referenced this pull request Jan 4, 2024
* main:
  Use blobs instead of random integers (napari#6527)
  Set default dtype for empty `_ImageSliceResponse` (napari#6552)
  Add creating image from clipboard (napari#6532)
  Fix check if plugin is available from conda (napari#6545)
  Fix generation of layer creation functions docstrings (napari#6558)
  Print a whole stack if throttler is not finished (napari#6549)
  Update `app-model`, `babel`, `coverage`, `dask`, `fsspec`, `hypothesis`, `imageio`, `ipython`, `lxml`, `magicgui`, `pandas`, `pint`, `psutil`, `pydantic`, `pyqt6`, `pytest-qt`, `tensorstore`, `tifffile`, `virtualenv`, `xarray` (napari#6478)
  Minimal changes for mlx (and jax) compatibility for Image layers (napari#6553)
  [pre-commit.ci] pre-commit autoupdate (napari#6528)
  Moving IntensityVisualizationMixin from _ImageBase to Image (napari#6548)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix PR with bugfix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0