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
SlicerDMRI extension is not updating for 5.6.1 #7531
Comments
The "published" date is when the extension package was uploaded to Girder. That is a new date for each Slicer version. It is different from last update time. See details of the macOS Slicer-5.6.1 SlicerDMRI package here: https://slicer-packages.kitware.com/#item/657844c683a3201b44d52d8b Interestingly, the last update is December 30, which is neither the last SlicerDMRI update date (Dec 15) nor the latest extension package date (yesterday).
The yellow icon on the dashboard is there for all builds, but recent ones cannot be downloaded (file not found on girder): Since the latest available package corresponds to the latest SlicerDMRI update (Dec 15) I'm not sure if there is anything wrong. It may be a valid behavior not to update packages if there was no change in the source repository. Maybe you could update something in the SlicerDMRI extension and see if the changes show up in the extensions catalog the next day? |
Small change to see if a new package becomes available for the 5.6.1 release of Slicer. See: Slicer/Slicer#7531
Tried this but it didn't result in a new DMRI being offered. After doing an update to SlicerDMRI yesterday, a new package was built last night, but the older December 12 version of the DMRI extension is still being offered in the Extension manager for SlicerStable (5.6.1). The package appears to be in girder but it's not being detected somehow. |
As of https://discourse.slicer.org/t/many-extensions-are-not-getting-updates-for-slicer-stable-release/34343/4, the macOS delivery of SlicerDMRI should be working for Slicer Stable (5.6.1). See https://slicer.cdash.org/builds/3308587. @pieper can confirm prior to closing this issue. |
It looks good to me but it would be great if @jhlegarreta or @ljod could doublecheck. |
Hi I am getting a Dec 30 version of SlicerDMRI on Mac when I install it in Slicer 5.6.1. Is that what we expect? Or do we expect something from Jan 11? |
Thanks for the report, while the extension was successfully build1 against the latest Stable build tree, there is any issue uploading the package. We are investigating and will report back. Footnotes |
When I go to https://extensions.slicer.org/catalog/All/32438/macosx?q=slicerdm, SlicerDMRI for macOS is downloading
|
After further analysis, I confirm @jamesobutler conclusion. |
It appears that the database entry was updated on December 30th instead of December 15th/16th when the latest commit was pushed because there were some build errors that ultimately got resolved. Whether the extensions manager web page should say the date of the commit that was last built successfully instead of when the database was updated with the new uploaded package is another thing. Maybe it would reduce confusion @jcfr ? ^ If after install on macOS it is indicating a commit other than SlicerDMRI/SlicerDMRI@b4357b9, that would still be an open issue to look into. On Windows it is weird in that it states the commit is SlicerDMRI/SlicerDMRI@a23fcbb from SlicerDMRI/SlicerDMRI#234, however that commit was later force-pushed off of the |
In the meantime, this is intended to display the revision: |
Summary
Even though the repository has been updated Dec 15, 2023, the DMRI extension offered for 5.6.1 is still from Dec 12, 2023. The s4ext file points to the master branch of SlicerDMRI for both 5.6 and the master branch of the extension index. The SlicerPreview versions correctly offer a newer build with the most recent SlicerDMRI.
Steps to reproduce
Try to install dmri in 5.6.1 and you are offered the older version.
5.6.1:
Nightly:
Environment
For reference: #7530
Possibly related to: AIM-Harvard/SlicerRadiomics#80, but different because there is no superbuild for SlicerDMRI so the same fix does not apply.
@lassoan do you have any suggestions?
The text was updated successfully, but these errors were encountered: