Closed hbmartin closed 4 years ago
Merging #48 into master will increase coverage by
0.02%
. The diff coverage is80.95%
.
@@ Coverage Diff @@
## master #48 +/- ##
==========================================
+ Coverage 91.64% 91.67% +0.02%
==========================================
Files 14 14
Lines 922 949 +27
==========================================
+ Hits 845 870 +25
- Misses 77 79 +2
Impacted Files | Coverage Δ | |
---|---|---|
pytube/monostate.py | 86.66% <100%> (+2.05%) |
:arrow_up: |
pytube/query.py | 100% <100%> (ø) |
:arrow_up: |
pytube/streams.py | 93.49% <100%> (+0.45%) |
:arrow_up: |
pytube/__main__.py | 82.22% <42.1%> (-2.82%) |
:arrow_down: |
pytube/extract.py | 93.75% <96.15%> (+3.09%) |
:arrow_up: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update ea14b7b...8f9dd79. Read the comment docs.
@swiftyy-mage @movraxrsp I know y'all have looked into these issues before and I'm a bit nervous since taken together these are real changes to decrypting the streams. Can you take a look and try out with a few different videos?
@hbmartin Will do.
@jslay88 can you try this branch and verify that it addresses #47 ? After that I can cut a release with these changes.
is_otf
property toStream
for s in yt.streams.otf().all(): ...
to avoid 404sfilesize_approx
stream property accurate to <0.1MB to avoid HTTP callssts
if unavailable when loading age protected videosextract