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

Config files - add frames per second per channel #27

Open
vstadnytskyi-FDA opened this issue Jul 25, 2022 · 4 comments
Open

Config files - add frames per second per channel #27

vstadnytskyi-FDA opened this issue Jul 25, 2022 · 4 comments
Labels
good first issue Good for newcomers help wanted Extra attention is needed

Comments

@vstadnytskyi-FDA
Copy link
Collaborator

vstadnytskyi-FDA commented Jul 25, 2022

Depending on Intel realsense depth camera we can pick difference FPS for different channels. I propose we redesign configuration file and make it nested. In addition, to standard fields, I propose to have: FPS, buffer length,

Here is an example of a section of config file:

channels:

  - #<-this is first entry in the channels list
    type: depth
    fps: 30
    buffer_length: 30

  - #<-this is second entry in the channels list
    type: color
    fps: 30
    buffer_length: 30

  - 
    type: infrared
    fps: 30
    buffer_length: 30

  - 
    type: accel
    fps: 250
    buffer_length: 30000

  - 
    type: gyro
    fps: 400
    buffer_length: 30000

Let us discuss.

I propose we split each stream(channel) in a different entry in the configuration file and assign different circular buffers and different FPS to each channel. It will provide more flexibility in future development. Though, it would add a little bit more coding.

in addition, it will allows to configure L515 camera differently from D435i camera based on config file. For example, there is no "infrared" channel in D435i or D455 cameras but it is present in L515. Adding channels to configuration file would provide needed flexibility.

What do you think? @AbdelRahmanNasser20

@vstadnytskyi-FDA vstadnytskyi-FDA added help wanted Extra attention is needed good first issue Good for newcomers labels Jul 25, 2022
@AbdelRahmanNasser20
Copy link
Contributor

I can start working on that but I have a question. How does the d435i not have a infrared channel but it can display a infrared image? is it mixed with another channel?

@vstadnytskyi-FDA
Copy link
Collaborator Author

I can start working on that but I have a question. How does the d435i not have a infrared channel but it can display a infrared image? is it mixed with another channel?

I did not know it has infrared image. I am curious to see how it looks. Last time I tried, I could not start device class for D455 camera because of gyroscope or accelerometer settings. I have hard-coded FPS into configuration file, but later I learned that two cameras have different framerate for gyro(accel).

@AbdelRahmanNasser20
Copy link
Contributor

AbdelRahmanNasser20 commented Jul 25, 2022

I can start working on that but I have a question. How does the d435i not have a infrared channel but it can display a infrared image? is it mixed with another channel?

I did not know it has infrared image. I am curious to see how it looks. Last time I tried, I could not start device class for D455 camera because of gyroscope or accelerometer settings. I have hard-coded FPS into configuration file, but later I learned that two cameras have different framerate for gyro(accel).

All you have to do connect the camera and change serial number in the config file and run the live stream test and you will be able to see it.

@AbdelRahmanNasser20
Copy link
Contributor

I don't have a D455 camera. Can you please confirm that it has a infrared channel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants