Skip to content

PR: Implement and document "Pip", "Poetry" and "Docker" based installation.#1

Open
thomasmansencal wants to merge 2 commits into
wetadigital:mainfrom
colour-science:feature/docker
Open

PR: Implement and document "Pip", "Poetry" and "Docker" based installation.#1
thomasmansencal wants to merge 2 commits into
wetadigital:mainfrom
colour-science:feature/docker

Conversation

@thomasmansencal

@thomasmansencal thomasmansencal commented Aug 19, 2020

Copy link
Copy Markdown

Hi,

This PR implements and document installation via:

I removed the !pip install directives in the two notebooks so that to not trash people's system Python interpreters and updated the README.md file with installation and usage instructions, the three methods have been tested and work.

Cheers,

Thomas

Signed-off-by: Thomas Mansencal <thomas.mansencal@gmail.com>
@thomasmansencal
thomasmansencal force-pushed the feature/docker branch 2 times, most recently from d0d86e7 to a40eaea Compare August 19, 2020 09:00
@thomasmansencal thomasmansencal changed the title PR: Implement "Poetry" and "Docker" installation support. PR: Implement and document "Pip", "Poetry" and "Docker" installation. Aug 19, 2020
@thomasmansencal thomasmansencal changed the title PR: Implement and document "Pip", "Poetry" and "Docker" installation. PR: Implement and document "Pip", "Poetry" and "Docker" based installation. Aug 19, 2020
Signed-off-by: Thomas Mansencal <thomas.mansencal@gmail.com>
@antond-weta

Copy link
Copy Markdown
Contributor

@KelSolaar Hey Thomas, could you please resolve the conflicts, I'd like to get this merged. Thanks

@thomasmansencal

Copy link
Copy Markdown
Author

Let me look at this once back in New Zealand.

@thomasmansencal

Copy link
Copy Markdown
Author

So as per my new contractual requirements, I cannot touch this repo before asking perms to powers that be. Let me get back to you!

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