Image transfer demo

RE: Image transfer demo

ovrebekk — Tue, 21 May 2019 06:51:00 GMT

Hi guys

I am very happy to hear it works for both of you 🙂

I just pushed a new update with a modified fix.

I wasn't really happy with having to recompile the code when using the plus version of the sensor, so I realized I could just fix the issue on the app side rather than trying to fix it on the nRF side. If you pull the latest update to the app and the nRF code there is no define to set anymore, and the Android app will simply remove the extra byte if it is detected.

Best regards
Torbjørn

RE: Image transfer demo

Blaugranas23 — Tue, 21 May 2019 06:35:59 GMT

Hi! I checked your fixes too, the program also works fine! Thanks a lot!:)

RE: Image transfer demo

AlexW — Mon, 20 May 2019 13:57:48 GMT

Just did a quick test and it worked 👍

RE: Image transfer demo

ovrebekk — Mon, 20 May 2019 13:31:19 GMT

Hi Alexander

The problem was quite simple in the end ;)

Please let me know once you have had a chance to test it, so I know for sure that the problem is fixed 🙂

Best regards
Torbjørn

RE: Image transfer demo

AlexW — Mon, 20 May 2019 13:15:21 GMT

Hey Torbjørn,

Nice to hear that you alredy found a solution and updated the code!

Thank you a lot!

RE: Image transfer demo

ovrebekk — Mon, 20 May 2019 13:05:43 GMT

Hi Alexander

It seems the problem was quite simple. Removing the dummy read was indeed the solution, but some other minor changes were also needed.

Please pull the latest version of the repo and give it a try. You will need to set the ARDUCAM_MINI_2MP_PLUS_MODE define in main.c to 1 and recompile the project to have it work properly with the new modules.

Since we primarily use the older modules internally I don't want to change the default behavior of the project, but a longer term goal is to make a routine in the code that can detect the module type automatically so no recompile is needed.

Best regards
Torbjørn

RE: Image transfer demo

AlexW — Wed, 08 May 2019 14:11:34 GMT

Alright.
That sounds "good". 😃

I will focus on that as soon as I am back on that project as well.

Thanks.

RE: Image transfer demo

ovrebekk — Wed, 08 May 2019 13:10:10 GMT

Hi Alexander

I got the modules today, and did a quick test to see if they worked.

The "good" news is that they don't work properly for me either, so most likely this is a common issue with these modules. The previously suggested fix of removing the dummy read at the start of the sequence didn't work.

I will try to look into this in the coming weeks.

Best regards
Torbjørn

RE: Image transfer demo

AlexW — Wed, 08 May 2019 11:45:19 GMT

Sounds great!

Thank you so far ovrebekk!

RE: Image transfer demo

ovrebekk — Tue, 07 May 2019 13:30:49 GMT

Then it is most likely the same issue.

I am hoping to receive the module in the coming days, but because of other assignments it might be some weeks until I have time to test it properly.

Testing to see if I spot the same problem should be quick, but actually solving the issue might take more time ;)

Best regards
Torbjørn

RE: Image transfer demo

AlexW — Mon, 06 May 2019 07:20:35 GMT

Oh, I didn't realize that.

Actually, there is written "Mini-2MP-Plus" on my camera PCB as well.

RE: Image transfer demo

ovrebekk — Fri, 03 May 2019 12:07:47 GMT

Hi Alexander

The plus version I am referring to is the one Blaugranas23 linked to earlier in this thread:
https://www.amazon.com/Arducam-Module-Megapixels-Arduino-Mega2560/dp/B012UXNDOY

Based on your link it seems you are using the non-plus version, which is the one I am using.

You can confirm that the text on your module is "ArduCAM-Mini-2MP" and not "Mini-2MP-Plus" ?

Best regards
Torbjørn

RE: Image transfer demo

AlexW — Thu, 02 May 2019 09:47:46 GMT

Did you try starting a image stream and then observing if there are bluetooth packets >>26bytes?

Because the period of the capture is just half a second, right?

RE: Image transfer demo

AlexW — Thu, 02 May 2019 09:45:08 GMT

older camera module? I tought I am using the same you mentioned in the documantation and showed in the youtube video. (https://www.watterott.com/de/2MP-Mini-Kamera-Board-OV2640)

Which device you are referring to with the 'plus version'?
The smartphoe we used was a S8+, yes. But I tought that Blaugranas23 didn't try that one.

By the way, changing the resolution and the PHY works fine, so bluetooth communication in general should work.

RE: Image transfer demo

ovrebekk — Tue, 30 Apr 2019 13:20:14 GMT

So you don't see any image data being sent at all?

Normally the MTU exchange will happen 5-10 seconds after connection establishment, and should not be related to when you press the take picture button.

Best regards
Torbjørn

RE: Image transfer demo

ovrebekk — Tue, 30 Apr 2019 13:17:33 GMT

That is interesting, we have the Samsung Galaxy S8+ in the office as well, and I haven't seen any issues with the older camera module.

I assume you have the new plus version as well, similar to Blaugranas23?

Best regards
Torbjørn

RE: Image transfer demo

Blaugranas23 — Fri, 26 Apr 2019 07:42:03 GMT

Good day! I have caught some packets by sniffer during image transfer, it seems actata are transferred from peripheral device: Sequence of Empty Data PDU packets is sent constantly, and after "Take Picture" action only one ATT packet "Rcvd Exchange MTU Response" is displayed on WireShark screen. The length of this one seems to be 255 bytes, but in the lowest window we can see just 33. I'm sending you a screenshot of transferring packets during Image Transfer Exchange.

RE: Image transfer demo

ovrebekk — Fri, 26 Apr 2019 07:35:26 GMT

It will take some time unfortunately, as it takes a while to get the modules shipped to Norway ;)

Once you have some sniffer results please share them, it should be helpful in debugging the issue.

Best regards
Torbjørn

RE: Image transfer demo

AlexW — Thu, 25 Apr 2019 13:10:39 GMT

Hey.

I have exactly the same issue.

I use the nrf52840 DK and Arducam Mini 2MP on SideConnector.
(tested on nrf52832 as well - same issue)

Tested with Samsung Galaxy S8+ and Huawei Mediatab M2.

Data transfer seem to happen, but the picture won't be displayed in the APP.

RE: Image transfer demo

Blaugranas23 — Wed, 24 Apr 2019 15:22:39 GMT

Hi!

Thanks a lot, i'll be looking forward to your testing results.

Yes, i have a BLE sniffer PCA10031 dongle, i'll try to set up it on my PC and find out what particular data is sent in packets.

Best regards!

RE: Image transfer demo

ovrebekk — Tue, 23 Apr 2019 10:10:47 GMT

Your board and connections look good, that shouldn't be the problem.

I will order some of the modules and do some testing on my own, but unless I can reproduce the problem it is hard to find the solution.

Do you have some kind of Bluetooth sniffer so you can take a trace of the communications?

Best regards
Torbjørn

RE: Image transfer demo

Blaugranas23 — Mon, 15 Apr 2019 12:49:17 GMT

Hi!

Thanks for your response! Yes, i have just checked again my SDK verison (it is 15.2.0) and i cloned repository for image transfer project by next link "">github.com/.../nrf52-ble-image-transfer-demo.git", the the master branch is updated.

I'm sending you photos with board and module:

Have a good vacation time,

Best regards!

RE: Image transfer demo

ovrebekk — Fri, 12 Apr 2019 13:56:58 GMT

Then it is hard to say what the problem might be. Can you send me a picture of your board and module?

Did you make sure to pull the latest code from the repo?

The demo hasn't been tested on a Samsung Galaxy Note 8 specifically, but it works nicely on the Galaxy S8 and the Galaxy S8+. I have also tested it on various Nexus phones and a Huawei P20 Pro without any issues.

For the next week I will be out in Easter vacation unfortunately, but I will follow up the issue again when I'm back in office on the 23rd of April.

Best regards
Torbjørn

RE: Image transfer demo

Blaugranas23 — Wed, 10 Apr 2019 07:45:41 GMT

Hi!

Yes, i tried commented out the line "arducam_mini_2mp_fillBuffer(img_data_buffer, 1);", but during transfer i got new issue, it seems application receives not enough bytes of image. Screenshot is attached

Thanks!

RE: Image transfer demo

ovrebekk — Wed, 10 Apr 2019 06:56:17 GMT

Can you try to comment out line 796 of main.c and see if it works better?
https://github.com/NordicPlayground/nrf52-ble-image-transfer-demo/blob/master/main.c#L796

On some variants of the sensor this line is not needed, and having it in will cause problems.

Best regards
Torbjørn