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

VerticaPy Development Instruction #224

Merged
merged 4 commits into from
Feb 28, 2024
Merged

VerticaPy Development Instruction #224

merged 4 commits into from
Feb 28, 2024

Conversation

mail4umar
Copy link
Collaborator

Added instructions on how to install VerticaPyLab without VerticaPy.

close #222

@mail4umar mail4umar added the documentation Improvements or additions to documentation label Jan 13, 2024
@mail4umar mail4umar requested a review from roypaulin January 13, 2024 11:16
@mail4umar mail4umar self-assigned this Jan 13, 2024
README.md Outdated
In order to use VerticaPyLab for development of VerticaPy, the following changes can be made to create a VerticaPyLab image that does not have VerticaPy installed:

1. ``cd`` into VerticaPyLab/docker-verticapy. Edit the ``requirements.txt`` file and remove "verticapy" from the list.
2. If you also want the installation to be light and remove unnecessary extensions, open the Dockerfile in the same directory and comment out from line 53 to 63 [This is optional]
Copy link
Collaborator

Choose a reason for hiding this comment

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

Would it be better to show those lines? Because if the Dockerfile is updated the line numbers might be different.

README.md Outdated

Then re-install it.

Note: For Jupyter Notebook you will need to refresh the kernel after the new installation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you can just say "For your changes to take effect, you must refresh the kernel after the new installation"

README.md Outdated Show resolved Hide resolved
@roypaulin roypaulin merged commit d041f26 into main Feb 28, 2024
1 check passed
@mail4umar mail4umar deleted the dev_instruction branch April 9, 2024 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
2 participants