Skip to content
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

Update Dockerfile #152

Merged
merged 1 commit into from
Jul 5, 2023
Merged

Update Dockerfile #152

merged 1 commit into from
Jul 5, 2023

Conversation

Dialdroid
Copy link
Contributor

@Dialdroid Dialdroid commented Jul 5, 2023

Summary

The proposed change seeks to update the EGI Notebook Dockerfile by adding Elyra, an AI-centric set of Jupyter notebook extensions. Elyra aims to help data scientists, machine learning engineers, and AI developers through the model development life cycle complexities.

Details

Elyra provides a visual interface for creating notebook-based AI pipelines, simplifying the conversion of multiple notebooks into batch jobs or workflows. Elyra also includes a set of visual editors for Jupyter and Python scripts, allowing them to be edited and managed just like notebooks.

Elyra extends this functionality further, enabling easier workflow and pipeline management for Jupyter notebooks in the environment.

Changes Made

To add Elyra to the Dockerfile, it was included in the list of packages for Mamba to install. No other changes were made. The line of code that installs Elyra is:

RUN mamba install -y --quiet \
    ...
    elyra \
    ...

Test Steps

To ensure that the Dockerfile correctly builds an image with Elyra, you can build the Docker image and run a container from it. Once the container is running, you can access a Jupyter Lab instance and check for the presence of Elyra extensions.

Expected Results

The Jupyter Lab instance should include the Elyra extensions, allowing for enhanced Jupyter notebook pipeline management and more.

Added Elyra to the Dockerfile
Copy link
Contributor

@enolfc enolfc left a comment

Choose a reason for hiding this comment

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

Thanks @Dialdroid, it looks good!

@enolfc enolfc merged commit 0ee49c2 into EGI-Federation:master Jul 5, 2023
@enolfc
Copy link
Contributor

enolfc commented Jul 5, 2023

A change request was created for putting this into production Will be deployed in the coming days.

@Dialdroid
Copy link
Contributor Author

Thank you for the review, @enolfc! I'm thrilled to know my pull request has been approved and will be deployed to production soon. I look forward to seeing it in action and continue contributing to the project. If there's anything else needed from my end, please let me know.

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.

2 participants