Skip to content

Conversation

@murphp15
Copy link
Contributor

@murphp15 murphp15 commented Jul 12, 2023

Why

We need to publish the image to a pip registry so others can consume it and setup is easy and doesn't require cloning the source code

What

Most plugins are installed using the same command. which is declared in a common gitlab file.
For building this plugin I didn't think it made sense to reuse that code as building jupyter is a little more complicated.
So the release is totally new code.

How was this tested

I published to the registry and installed locally.

Commands:

pip install -U vdk_jupyterlab_extension
jupyter lab

Make sure the UI has the correct panels
Screenshot 2023-07-13 at 12 32 42

Signed-off-by: murphp15 <murphp15@tcd.ie>
paulm2 added 12 commits July 12, 2023 16:17
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
@murphp15 murphp15 changed the title [DRAFT] juypter build vdk-jupyter: publish image to pip registry Jul 13, 2023
paulm2 added 3 commits July 13, 2023 12:48
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Signed-off-by: murphp15 <murphp15@tcd.ie>
Copy link
Contributor

@duyguHsnHsn duyguHsnHsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

Signed-off-by: murphp15 <murphp15@tcd.ie>
@murphp15 murphp15 merged commit 4d07d8b into main Jul 14, 2023
@murphp15 murphp15 deleted the person/murphp15/build_juypter_python_package branch July 14, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants