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

Suggestion/Bugs from EDPR #3

Open
15 of 21 tasks
triccyx opened this issue May 6, 2021 · 9 comments
Open
15 of 21 tasks

Suggestion/Bugs from EDPR #3

triccyx opened this issue May 6, 2021 · 9 comments
Assignees
Labels
domain-software Related to Software

Comments

@triccyx
Copy link
Member

triccyx commented May 6, 2021

Some points came from EDPR @Iaxama:

  • Keep Exposition range between 15msec to 50msec
  • Bug on white balance, Red should be switched to Blue.
  • Check for artifacts in HiRes again
  • Log absolute exposition info
  • Add the third channel for white balance via rpc API
  • Check for latency
  • Left cam on setup is not perfectly on focus
  • preview is missing in UI
  • subsampling info is missing in UI
  • test honorfps from UI is working as expected?
  • application ico is missing
  • add contrast control
  • Reconnect button in UI
  • Add example for a client
  • Split README.md
  • Allow set also with absolute value from API
  • Log temperature on UI
  • UI window should not be resizable
  • Fill help
  • Default value button on UI
  • Led on UI for signal if the board is working
@pattacini
Copy link
Member

cc @Iaxama

@triccyx
Copy link
Member Author

triccyx commented May 6, 2021

@Iaxama feel free to post comments/bugs/requests here.
PS. Not all requests will be fulfilled 😄

@triccyx
Copy link
Member Author

triccyx commented May 6, 2021

Log absolute exposition info, for now, could be done only on the yarpdev log together with the statistics I have added. See above,

[INFO] |yarp.device.usbCamera| frames read by YARP  frame number: 66  fps: 13.2  interval: 5.02771  sec.  exposition: 15  msec.
[INFO] |yarp.dev.Drivers| device active in background...
[INFO] |yarp.device.usbCamera| frames read by YARP  frame number: 66  fps: 13.2  interval: 5.02664  sec.  exposition: 15  msec.
[INFO] |yarp.dev.Drivers| device active in background...

@triccyx
Copy link
Member Author

triccyx commented May 7, 2021

@salvi-mattia "Left cam on setup is not perfectly on focus." If you are at CRIS on the 12th or 13th could you check it with me?

@triccyx
Copy link
Member Author

triccyx commented May 7, 2021

Relating to latency:
I have reduced the number of buffers submitted to the low-level driver and the latency is now ok.
Note: Some other tests needed
Thanks to @Iaxama for having spotted the problem.

@triccyx
Copy link
Member Author

triccyx commented May 13, 2021

Relating to "Bug on white balance, Red should be switched to Blue.":
channels switched

@triccyx
Copy link
Member Author

triccyx commented May 28, 2021

Update some missing points.

@triccyx
Copy link
Member Author

triccyx commented May 31, 2021

Honorfps and resolution

commit
UI001

@triccyx
Copy link
Member Author

triccyx commented Jun 4, 2021

Thanks to @mfussi66 now we have the contrast.
@pattacini

@triccyx triccyx added the domain-software Related to Software label Jun 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain-software Related to Software
Projects
None yet
Development

No branches or pull requests

2 participants