Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update additional dependencies install command #689

Merged
merged 1 commit into from
Jun 26, 2023
Merged

Update additional dependencies install command #689

merged 1 commit into from
Jun 26, 2023

Conversation

tjburch
Copy link
Contributor

@tjburch tjburch commented Jun 24, 2023

Was starting to do some development and ran into the following:

pip install .[dev]
zsh: no matches found: .[dev]

Found that adding the single quote around it resolved it, so updated the instructions accordingly:

pip install '.[dev]'

And similar for the jax dependencies.

I assume this is ubiquitous, but if it's just a zsh for some reason, feel free to disregard.

Was starting to do some development and ran into the following:

```bash
pip install .[dev]
zsh: no matches found: .[dev]
```

Found that adding the single quote around it resolved it, so updated the instructions accordingly:

```bash
pip install '.[dev]'
```

And similar for the jax dependencies.

I assume this is ubiquitous, but if it's just a zsh for some reason, feel free to disregard.
@codecov-commenter
Copy link

codecov-commenter commented Jun 24, 2023

Codecov Report

Merging #689 (120b591) into main (259c474) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #689   +/-   ##
=======================================
  Coverage   88.32%   88.32%           
=======================================
  Files          40       40           
  Lines        2862     2862           
=======================================
  Hits         2528     2528           
  Misses        334      334           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tomicapretto
Copy link
Collaborator

I was not aware of this potential issue! Thanks for fixing it. I confirm it still works on my side.

@tomicapretto tomicapretto merged commit ea96a73 into bambinos:main Jun 26, 2023
@tjburch tjburch deleted the patch-1 branch June 26, 2023 10:15
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants