Resolved two issues: added support for MKV files and clarified "error" printed output#461
Resolved two issues: added support for MKV files and clarified "error" printed output#461Cedric-Boucher wants to merge 12 commits into
Conversation
there is now a new type of result, duplicate files these are files that already exist in the destination folder and are not imported if not importing duplicates I have thoroughly verified that my changes do not cause any other part of the code to break or act in an unexpected manner. I have also removed some code that could never get used: elodie/localstorage.py: checksum() could never return None, code was not accessible due to earlier return because of this: elodie/filesystem.py: process_checksum() did not need to check if checksum was None after calculating it this is what allowed my new code to use None as a flag meaning that the file being imported and skipped is a duplicate file
|
|
|
The fail on ci/circleci is due to the printed output being changed in |
|
I am working on updating the tests for my code, I'll need to create another PR when it's done. I have noticed that my code does fail 4 tests, even after updating the tests. so I will figure that out now |
|
oh I don't need to make a new PR apparently. cool. Well as you can see I've updated the tests to the new formatting and fixed the minor issue that had caused 4 of them to fail |
|
On this note: We should really add some test cases to test the functionality of trying to import a file that has already been imported, to see if the duplicate count goes up |
|
the test that has now failed (for whatever reason) has nothing to do with any code I have changed. Odd |
|
I changed nothing and now the test passes. It seems that the failed test was a HTTPS call to that geolocation server. Annoying |
|
Another note. Is there any documentation that needs to be updated with this "UI" change? |
video file extensions
as they are supported by exiftool
jmathai
force-pushed
the
master
branch
3 times, most recently
from
bbdb460 to
ec11497
Compare
|
Were you able to get mkv files to work? |
|
it's been so long, I don't remember. to be honest I don't use Elodie anymore. I've moved completely over to Immich now |
|
Understood, thanks. I've been looking at Immich the past few weeks and it looks great. Any complaints? I'm planning to migrate away from Google Photos to Immich. Created gh-496. |
|
I don't have any complaints, really. Immich is now stable and I rarely have any issues at all with it. It's still in active development so they are constantly fixing minor issues and making improvements! It's very impressive. I'd say it might still be missing some features compared to Google photos but for the most part it works exactly like Google photos, with automatic background sync and being able to share albums with other users (who can access your server), as well as multi-user support of course. I would definitely recommend trying it out. If you already have a NAS such as truenas it's very easy to install there |
4 participants
resolved: #457
resolved: #460