Image transfer demo

[Blaugranas23]

Hi guys! I have had an issue with Image Transfer Demo project. I use nRF52832 with OV2640 camera. I used installation project process described on the page https://github.com/NordicPlayground/nrf52-ble-image-transfer-demo#softdevice-version. The project was successfully compiled an launched on Nordic. Then i downloaded application for Android "ImageTransferDemo". Connection between Android app and the nRF is successfully established and when i try "Take Picture" from the app transmission of Packets normally executes, but picture doesn't appear into app after receiving packets. What the reason could be? Thanks...

[AlexW]

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.

[ovrebekk]

Hi 

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

[ovrebekk]

Hi

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

[AlexW]

Sounds great!

Thank you so far ovrebekk!

[ovrebekk]

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

[AlexW]

Alright.
That sounds "good".


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

Thanks.

[ovrebekk]

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

[AlexW]

Hey Torbjørn,

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


Thank you a lot!

[ovrebekk]

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

[AlexW]

Just did a quick test and it worked 

[Blaugranas23]

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

[ovrebekk]

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