moviemaker.rb fails on GoogleDrive.login

[danrademacher]

I just tried to make a movie from the Rim Fire images at request of USFS, and it looks like the changes @kueda made to fireslurp.rb for oauth maybe need to happen in moviemaker.rb too.

Error looks like this:

moviemaker.rb:91:in `download_images': undefined method `login' for GoogleDrive:Module (NoMethodError)
    from moviemaker.rb:200:in `<main>'

Line 91:

  session = GoogleDrive.login(GOOGLE_EMAIL, GOOGLE_PASSWORD)

I suspect the changes to fireslurp.rb in this commit need to be done in moviemaker.rb as well (or some variation of same)

[danrademacher]

I think I've got this working locally. Will verify and commit if so

[kueda]

Did it work? Sorry I didn't fix this when I fixed the other script.

[danrademacher]

I have the auth working -- but it doesn't seem to download the same number of images each time, and then I was getting an error after it "finished" downloading. Ran out of time and energy over the weekend but didn't want to commit only partially working code. I'll get more info and post before the night is out

[danrademacher]

OK, pretty sure auth fixed in this branch: https://github.com/nerdsfornature/changebrackets/tree/moviemaker-fix

@kueda could you give it a try and verify? I am getting other errors after auth and download of images but I think that's more evidence that I never ran this script before and maybe I'm missing dependencies or something

[kueda]

Yeah, running into an issue with @pglotov's alignment script, which means it would need to be rebuilt with a recent version of opencv... which might not work. I've since discovered http://wiki.panotools.org/Align_image_stack, which might be a better solution to the alignment problem, but it's a bit fussy and will need some tweaking before it will work (right now it just segfaults on me when run with default settings).

[jasper2xF]

I also had problems with the GoogleDrive access. Using the up-to-date authentication as in fireslurp.rb fixed this for me. Looks like exactly what @danrademacher did.

Also note that I raised issue #9 in relation to the alignment script being deprecated.