Skip to content

Girder 5 Upgrade #241

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Girder 5 Upgrade #241

wants to merge 9 commits into from

Conversation

BryonLewis
Copy link
Collaborator

@BryonLewis BryonLewis commented Apr 9, 2025

resolves #239

TODO:

  • Figure out development building for both the web_client and the vue application
  • The vue application needs to be placed relative the main application something like dive_server/dive_client
  • Fix the girder worker and get it uploading and running again in the docker compose file
  • Update the project so it publishes a pypi package when complete
  • Make sure the web_client and the vue_app are built and included in the published version of the pypi package
  • Reconfigure back again the slicer web UI imports and installation, it has changed again.
  • Test installing directly from the pypi package.
  • Make sure the production version is utilizing gunicorn instead of girder-serve

Notes:

  • Previous references to girder_worker.girder_plugin have no become girder_plugin_worker for imports. Remember to install girder_plugin_worker and not girder_worker.
  • Similarly slicer_cli_web has been restructured so slicer_cli_web.models becomes girder_slicer_cli_web.models and the installation needs to be from girder-slicer-cli-web from pypi.
  • Having the default value for GIRDER_WORKER_BACKEND would be nice because the default was implicit before.
  • Maybe some better instructions involving adding a static vue application to a location other than the default directory.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Girder 5 Upgrade
1 participant