puhitaku/mtplvcap

Not working with D700

puhitaku opened this issue · 10 comments

puhitaku commented

From YasukiKishi:

  • D700を電源ON⇒mtplvcap.exe起動したらシャッターは開く。
  • localhost:42839でISO,F値,Focus制御はできるが、Width,Height,Frame rateの情報は取れていない、プレビューも絵が映らない
  • localhost:42839/viewも表示できない
[0000]  INFO usb: found: 04b0:0422
[0000]  INFO usb: found: 0cf3:e303
[0000]  WARN mtp: detected more than 1 device, opening the first device: 04b0:0422
[0000]  INFO main: started
[0002]  WARN lv: workerLV: failed to start live view: the camera is not ready
[0003]  WARN lv: frameCaptor: failed to decode header
puhitaku commented

Related branch: fix/d700

yanorei32 commented

Sorry it's my mistake.

Jokler commented

Removing the magic 8 here makes video work for me:

mtplvcap/mtp/server.go

Line 741 in 94f3b4c

err = binary.Read(bytes.NewReader(raw[8:hs]), binary.BigEndian, &lvr)

puhitaku commented

@Jokler Wow, great! I'll add a special path for D700 and release an experimental version later.

  • 👍1
puhitaku commented

@Jokler Sorry for keeping you waiting, here's an experimental build that includes the fix you proposed: v1.6.2-rc1.
Could you try this build and see if it works? Thank you!

Jokler commented

If I just try to run that I get this:

[0001] ERROR lv: failed to start live view (InvalidStatus). Investigating the reason...
[0001]  WARN lv: workerLV: failed to start live view, reason: recording destination is the card

I was testing on top of the fix/d700 branch when it was working for me. If I run my working version first and then run v1.6.2-rc1 keeping the camera connected it works too.

Since I wasn't super clear before only the video seems to work though, I'm not sure if the settings in the web interface are supposed to work for all cameras though.

puhitaku commented

Some older models need us to change the "record media" to the RAM. The message looks like it.
I pushed a commit to fix/d700-new and created a corresponding release v1.6.2-rc2.
@Jokler Could you try this again and see if it works without invoking the old fix/d700 build? Thanks for your cooperation!

Extra info of fix/d700: the branch is really old, and had some experimental commits that are also implemented in the master branch in very similar but different lines of code. I just gave it away to rebase and solve conflicts due to too many conflicting diffs.

Jokler commented

Now it works right away! :)

  • 🎉1
puhitaku commented

I've just released Version 1.6.2 🎉
Now I can close here, the longest issue I've ever had. Thank you very much! @Jokler

  • 🎉2