Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

Conversation

@kaushikb11
Copy link
Contributor

@kaushikb11 kaushikb11 commented May 16, 2022

Before submitting

  • Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Fixes # (issue)

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

@codecov
Copy link

codecov bot commented May 16, 2022

Codecov Report

Merging #172 (93d358d) into main (c478057) will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff         @@
##           main   #172   +/-   ##
===================================
  Coverage    75%    75%           
===================================
  Files         2      2           
  Lines       370    370           
===================================
  Hits        276    276           
  Misses       94     94           

Comment on lines 5 to 8
! pip uninstall --quiet "torchaudio" "torchvision" "torchtext" "fastai" "datascience" "kapre" --yes
! pip install --quiet "rl-warp-drive>=1.6.5" "pytorch-lightning>=1.4"
else:
! pip install --quiet "ipython[notebook]" "rl-warp-drive>=1.6.5" "ffmpeg-python" "setuptools==59.5.0" "torch>=1.8" "torchmetrics>=0.7" "pytorch-lightning>=1.4"
Copy link
Contributor

Choose a reason for hiding this comment

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

this is not valid python, if you have some additional comment on how to install it on different env such as colab, lest add it as text celll

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We could use subprocess.run(..)

Copy link
Contributor

Choose a reason for hiding this comment

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

this is not the point, you try to band the system for this specific notebook and specific env of Colab...

Comment on lines 5 to 8
! pip uninstall --quiet "torchaudio" "torchvision" "torchtext" "fastai" "datascience" "kapre" --yes
! pip install --quiet "rl-warp-drive>=1.6.5" "pytorch-lightning>=1.4"
else:
! pip install --quiet "ipython[notebook]" "rl-warp-drive>=1.6.5" "ffmpeg-python" "setuptools==59.5.0" "torch>=1.8" "torchmetrics>=0.7" "pytorch-lightning>=1.4"
Copy link
Contributor

Choose a reason for hiding this comment

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

this is not the point, you try to band the system for this specific notebook and specific env of Colab...

@Borda Borda mentioned this pull request May 17, 2022
3 tasks
@Borda Borda self-requested a review May 17, 2022 04:57
@Borda Borda added the enhancement New feature or request label May 17, 2022
@Borda Borda merged commit 3fda6eb into main May 17, 2022
@Borda Borda deleted the install/warpdrive branch May 17, 2022 23:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants