error in lld.py in line 12 #1
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…a variable reference in one place. - Regex pattern now allows for better file saving without as many errors as before on Windows file systems. - replacement of invalid characters is now a space instead of nothing to help delineate words better - Fixing typo in readme file.
Attempt to help fix errors within file paths on Windows with revised regex pattern, additional minor revisions
…r seems to works better. - In the future you could consider something like this below: invalid_file_chars = re.compile(r'[^/]') if os.name == "nt": invalid_file_chars = re.compile(r'[^\\/:*?"<>|]')
Updating regex pattern.
Subtitle Support, Release Date, Chapter & File numbering
Added Subtitle-Downloads, Release-Date and Chapter/Video-Numbering, German Umlaut
1. The course release date is now added to the course folder's name (because Lynda often updated the courses and you didn't know of what date your downloaded version was). In the future, we should differentiate between the release-date element and the update-date element (which's name I don't know yet) and use the newer one of both for the folder name. 2. I added subtitle support. That was a bit of a mess, but I tried my best, seems to be working. .srt files have to be the same name of the .mp4 files so a player like MPC HC will load it automatically. 3. I took over the chapter/video numbering from mclmza/linkedin-learning-downloader#3 (comment), so thanks for that! 4. Merged pull requests mclmza/linkedin-learning-downloader#36 and mclmza/linkedin-learning-downloader#33 5. Corrected bug: re.sub(invalid_file_chars, " ", string) is leading to an IO error when a video has an unallowed char at the end, e.g. out\What Is the Google Cloud Platform \ but Windows will remove space at the end of an folder, so the next download attempt will fail because there isn't such a path anymore. Strip() does the thing. Just saw that mclmza/linkedin-learning-downloader#33 also had that idea. 6. Replaced German Umlaut with their non-crictial equivalents
1. Added download for exercise files 2. Checking now for full course access 3. Now using configurable download path 4. Some code clean-up
Now catching the 'elements' error when trying to loop through an empty list because the request gave an 404 response caused by mostly user error when inserting a wrong course slug in the config file.
1. Easier adding of courses to the downloader. Just add them via LinkedIn (app or browser) to your bookmarks. Tool will now parse your LinkedIn bookmarks (https://www.linkedin.com/learning/me/saved). The previously alternative of adding course slugs manually by pasting them into the config file is still there. The app will do a check for duplicates before automatically adding a course-slug to the list. 2. After successfully downloading a course, the course will be disabled in the config file (Download list) by putting a "#" in front of the course. If you want to check the files for integrity again, just the "#" and the next time when running the app it will check the enabled courses (if already).
Fixed Bug: Built a workaround for getting all bookmarks. LinkedIn is only providing 10 bookmarks at a time, so API has to be triggered (bookmarks/10) times
…ame old folders and files, use extractName to get alphabetic names. Backwards compatible with previous folder structure and naming convention. Allows to complete previous downloads (with old naming of folders and files) and change the naming of folder and files
…ame old folders and files, use extractName to get alphabetic names. Backwards compatible with previous folder structure and naming convention. Allows to complete previous downloads (with old naming of folders and files) and change the naming of folder and files. mclmza/linkedin-learning-downloader#40.
…om/dpkkdn/linkedin-learning-downloader into backwards-compatible-naming-resume
Various additions: subtitle & exercise downloads, file numbering, bookmark parsing etc
Backwards compatible naming resume
|
File "llcd.py", line 55 I get this error Everytime |
|
I'm getting an error: Please help. |
|
Please help me, this errors: |
I think it works only on up to Python 2.7 ! No support for Python 3. |
A minor changes to make it works with the current LinkedIn form.
Added 1 second sleep between requests to server to prevent 429 response (Too many requests).
Skip existing videos
This should fix the download issue
|
Line 50 is throwing an error:
|
You are geting this error becuase you are using python3 but this code it designed to run on python2. I think you have installed python2 and 3, so you will need to mentioned all the command as python2 and start if from the beginning |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Traceback (most recent call last):
File "lld.py", line 12, in
reload(sys)
NameError: name 'reload' is not defined