Fix invalid pip syntax, use double quotes for portable version (zsh/windows)

[bmerkle]

pip install command with single quotes results in the following error:

Why are these changes needed?

because it results in a pip syntax error

pip install command with single quotes results in the following error:

(venv) C:\work\microsoft\autogen>pip install 'autogen-agentchat==0.4.0.dev8' 'autogen-ext[openai]==0.4.0.dev8'

ERROR: Invalid requirement: "'autogen-agentchat==0.4.0.dev8'": Expected package name at the start of dependency specifier
    'autogen-agentchat==0.4.0.dev8'
    ^

Related issue number

as this is a trivial change, i did not create a separate issue

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[ekzhu]

Ironically this will break usage in ZSH...

Let's use double quotes.

And while you at it, could you also help us fix the whole repo?

[bmerkle]

yes, I will do this.
Lets use double quotes

[ekzhu]

This file shouldn't have been changed.

[ekzhu]

This file shouldn't have been changed except for updating the installation instructions.

[ekzhu]

This file shouldn't have been changed except for updating the installation instructions.

[ekzhu]

This file shouldn't have been changed except for updating the installation instructions.

[ekzhu]

This file shouldn't have been changed except for updating the installation instructions.

[bmerkle]

I read your comments but IMO these are normal shell scripts, just inside a jupyter notebook cell, so we should update this also.
I have verified that the notebook runs, so there is no error, and it should be consistent between docs and code.

We should maybe consider even to use "%pip install ..." instead of "pip install ...." in the ipynb files (with percentage) as this is a good practice (to enforce installation into the venv envionment).

what do you think ?

Reference:
https://community.databricks.com/t5/data-engineering/whats-the-difference-between-magic-commands-pip-and-sh-pip/td-p/21618
https://stackoverflow.com/questions/65547021/difference-between-pip-and-pip-in-python-jupyter-notebook-and-colab

[ekzhu]

I have verified that the notebook runs, so there is no error, and it should be consistent between docs and code.

My comment was that I saw the outputs of the notebooks have been cleared -- we should keep them

We should maybe consider even to use "%pip install ..." instead of "pip install ...." in the ipynb files (with percentage) as this is a good practice (to enforce installation into the venv envionment).

I agree.

[bmerkle]

I have verified that the notebook runs, so there is no error, and it should be consistent between docs and code.

My comment was that I saw the outputs of the notebooks have been cleared -- we should keep them

ok, sorry i misunderstood you. I have a commit hook for jupyter file to clean the output :-)
should I add the output again and commit again ?

We should maybe consider even to use "%pip install ..." instead of "pip install ...." in the ipynb files (with percentage) as this is a good practice (to enforce installation into the venv envionment).

I agree.

cool, so i would change this to %pip install

[ekzhu]

No worries. Yes please reset changes on those files and apply double quotes again.