-
Notifications
You must be signed in to change notification settings - Fork 18
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
New napari-plugin does not show up on the napari-hub #611
Comments
Similar issue with this plugin : Cc @gschlafly |
@haesleinhuepf Thanks for opening this! There might have been a lag between when the plugin was released and when we retrieve new plugin data (though this should be happening every 5 minutes). Looks like those are showing up now. Do you remember how much time had passed since release, at the time you weren't seeing the data on napari-hub? |
@richaagarwal Our napari-LF plugin did not show up on napari-hub for at least four hours. When I checked 19 hours after the release, it was on napari-hub. |
Thank you for the info, @gschlafly! It looks like there's a related issue #200 so I think it's worth more investigation at this point. I'll chat with the team to see when we might be able to prioritize this investigation. |
here's another example: napari/napari#4862
given that napari now uses the napari hub's API to get the list of available plugins, this is a problem that propagates to napari as well |
oddly, for https://api.napari-hub.org/plugins/napari-deepfinder {
"conda": [],
"display_name": "",
"plugin_types": [],
"reader_file_extensions": [],
"writer_file_extensions": [],
"writer_save_layers": []
} otoh, the expected response for a non-existant plugin should be an empty blob: https://api.napari-hub.org/plugins/foobar {} |
Hmmm I thought the |
@DragaDoncila hmm not sure, I thought we also switched it off on the backend but I'm not sure what was involved in doing that. maybe @klai95 @richaagarwal knows? |
iirc the backend changes were just switching off the lambdas, I don't think we ever reverted the API changes. We should definitely gate it if we don't want to revert - not sure what the timelines are looking like for updating the backend work @richaagarwal |
@DragaDoncila That's correct, we never reverted the API changes. I don't know if there is a strong need to do this as we had discussed that no one else is really consuming the npe2 data yet, but maybe @neuromusic can weigh in here. But if needed, then I am in support of reverting all npe2-enabled features changes. |
Do you think it's preferable to revert rather than just add a "gate" to hold the fields back? I think a lot of the API stuff won't change so it might be more effort to revert then add it back? |
@richaagarwal I've been looking into using a proper feature flag service on the hub frontend, I wonder if this is something we could use for the backend too in the future so things like this can be easier. They have a python SDK, so something we can use for the backend: https://help.split.io/hc/en-us/articles/360020359652-Python-SDK |
why are we seeing these npe2 fields for |
@klai95 @potating-potato any idea why we wouldn't be seeing data from https://pypi.org/pypi/napari-deepfinder/json populated yet on the hub side? |
@neuromusic probably a quirk of when we're returning information. We decided early on that |
@DragaDoncila so even though it's not showing up, |
@neuromusic correct - somewhere somehow, the napari hub backend is aware of |
The plugin is still not available on the napari-hub, do you know why? Because I just saw that more recent released plugins are available on the hub... |
For some reason, the backend data for napari-deepfinder was corrupted, I deleted them and the repopulation of the data is then successful. The reason for the corruption is not yet known. I don't have any additional context but that the corrupted json contains nothing but "conda: []", this is not expected from the typical indexing, I believe this is probably related to npe2 somehow |
Thanks for the fix! This is indeed somewhat strange xD |
Thanks for looking into this. |
I digged around on the napari-lattice and I think I found the gap: when napari-lattice 0.1.5 published, we correctly fetched it and created a profile for it in our system, however when we populate the metadata, we used the non-versioned pypi API to get the latest, which seems to have not updated to correctly report the metadata for 0.1.5, as a result, the 0.1.5 profile is corrupted with 0.1.4b0 data when it was indexed. I will submit another PR to fix this |
hey @potating-potato, let me know if there's anything I can do to facilitate here with npe2api. napari-lattice is looking good here: https://github.com/napari/npe2api/blob/main/public/manifest/napari-lattice.json |
thanks for the help! I think I have found the issue and going to submit a PR to close this. |
Just to give a quick update here: I think we may have pinpointed the cause here. Our hypothesis is that the lag may be due to PyPI's non-versioned endpoint sometimes taking time to catch up with the latest release data. For instance, https://pypi.org/pypi/mage-ai/json currently does not reflect latest release data available in https://pypi.org/pypi/mage-ai/0.2.1/json. #608 outlines a potential fix to this. |
I am going to close this since we have #608 to track work on a potential fix. We can re-open this in the future if needed or the situation worsens. |
Description
Hi all,
a new plugin napari-lattice does not show up on the napari-hub, even though the setup.cfg appears correct to us.
Can you give us guidance on how to publish this plugin on the hub?
Thanks!
CC @pr4deepr
See also
The text was updated successfully, but these errors were encountered: