Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pictures taken by the SelfieWidget are rotated incorrectly on some devices #1552

Closed
grzesiek2010 opened this issue Oct 23, 2017 · 6 comments
Closed
Labels
bug help wanted Issues that are well-specified and don't require too much context.
Milestone

Comments

@grzesiek2010
Copy link
Member

Software and hardware versions

Collect v1.10.2

Problem description

Pictures taken by the SelfieWidget are rotated incorrectly on some devices.

Other information

This table is from #1541 so it might look a little different on v1.10.2 but the problem is visible there too and once we have merged #1541 it will look the same.

case/device Samsung S7 huawei sony z3 samsung J1 infinix
image widget, front camera, vertical vertical vertical vertical vertical vertical
image widget, front camera, horizontal horizontal horizontal horizontal horizontal horizontal
selfie widget, front camera, vertical vertical vertical vertical horizontal horizontal
selfie widget, front camera, horizontal crash upside down upsiede down horizontal horizontal
@lognaturel lognaturel added this to the Next milestone Oct 23, 2017
@lognaturel lognaturel added help wanted Issues that are well-specified and don't require too much context. bug labels Jan 18, 2018
@ShenJack
Copy link
Contributor

Excuse me,I've been searching on this problem for 2days,and i also found the same problem appears in Google's Camera2 example.
If you don't mind that I‘m just an Android greenhand with 1 year experience and 4 months for intern , please asign it to me, I'll try to fix it up with pleasure.
May I claim it ?

@ShenJack
Copy link
Contributor

@grzesiek2010 @lognaturel

@grzesiek2010
Copy link
Member Author

@ShenJack yeah sure, sounds good! You can claim any issue by yourself, here you can find a way how to do that https://github.com/opendatakit/collect/blob/master/CONTRIBUTING.md

i also found the same problem appears in Google's Camera2 example

You are right.

@ShenJack
Copy link
Contributor

@opendatakit-bot claim

@getodk-bot
Copy link
Member

Welcome to Open Data Kit, @ShenJack! We just sent you an invite to collaborate on this repository at https://github.com/opendatakit/collect/invitations. Please accept this invite in order to claim this issue and begin a fun and rewarding experience contributing to Open Data Kit!

Here are some tips to get you off to a good start:

  • Please read the README.md and CONTRIBUTING.md in this repo. Those two documents have much of what you need to get started.
  • Join the ODK developer Slack to get help, chat about this issue, and meet the other developers.
  • Sign up and introduce yourself on the ODK community forum to meet the broader ODK community.

See you on the other side (that is, the pull request side)!

@ShenJack
Copy link
Contributor

@grzesiek2010
Things got much more clearer.
I guess the cause is the incorrect use of Google's example, the example is only used for back camera.
When switching the code from back into front camera, some code hasn's been changed thoroghly, thus making some device seem allright while some produce error.

After fix, Test cases has passed on
samsung s6 7.1.1 front camera 270°
oppo a53 5.1.1 270°
meizu m1 metal 5.1 270°

I still wonder the stability on other devices whose front is not 270°, but 270° is mainstream.
Maybe more tests are needed.

I'll open a PR soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted Issues that are well-specified and don't require too much context.
Projects
None yet
Development

No branches or pull requests

4 participants