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

Question re IC2.5 WDR settings? #17

Open
g40 opened this issue Jul 30, 2024 · 12 comments
Open

Question re IC2.5 WDR settings? #17

g40 opened this issue Jul 30, 2024 · 12 comments
Labels
bug Something isn't working

Comments

@g40
Copy link

g40 commented Jul 30, 2024

image

The TIS IC Capture 2.5 application has exposes the property page shown above. Are these settings renamed in IC4 or have they been removed/not yet implemented?

Thanks.

@TIS-Edgar
Copy link
Member

Hi,
The feature should still be available. It should under "Tone Mapping Control". The ic4-demoapp property dialog should allow you to search for "Tone Mapping" to verify this.

With IC4 all camera properties are now GenICam Standard Features Naming Convention (SFNC) compatible. This means that some properties might have been renamed to be compliant. The majority of properties should remain unchanged between 3.5 and 4.

@TIS-Edgar TIS-Edgar added the question Request for information/clarifications. label Jul 30, 2024
@g40
Copy link
Author

g40 commented Jul 30, 2024

@TIS-Edgar Thank you. I'll check it out.

@g40
Copy link
Author

g40 commented Jul 30, 2024

@TIS-Edgar Using the 1.0.0.2416 build, no Tone Mapping property can be found -

image

@TIS-Edgar
Copy link
Member

ok, that should list the tone mapping properties. So something is fishy.

What version of the gige gentl provider is installed?

@g40
Copy link
Author

g40 commented Jul 30, 2024

1.1.0.674 dated 18/03/24. See below.

image

@TIS-Edgar
Copy link
Member

The IC4 and gentl provider versions both support tone mapping.
Can you save the genicam document and give it to us?
Open IC4 Demo App and click:
Device -> Driver Properties -> Guru -> 'StoreRemoteDeviceGenICamDocument'

The xml file will contain a complete description of what the camera properties are.

@g40
Copy link
Author

g40 commented Jul 30, 2024

@TIS-Edgar: Please find attached. Simply remove .txt extension to get original XML (which is an unsupported upload on Github)

DFK 33GX545_RemoteDevice.xml.txt

@TIS-Edgar
Copy link
Member

Thank you.
The xml contains ToneMappingEnable and other properties, meaning camera and driver are exporting the property correctly.

Two possibilities come to mind that might hide tone mapping:

  1. The property SoftwareTransformEnable has to be On. It should have that value per default, but better to check.
  2. What PixelFormat are you using? ToneMapping should be available for all PixelFormat, but it might very well be a bug for one. Does switching the format make a difference?

@g40
Copy link
Author

g40 commented Jul 31, 2024

@TIS-Edgar It looks like the pixel format affects the properties. How do we gt tone properties using BGR8?

Thanks.

image

Using BGR8

image

@TIS-Edgar
Copy link
Member

You found a bug in our gige provider!
BGR8 should also have Tone Mapping.
We will fix that in our next driver release. Not certain about the time frame, so cannot comment on that yet.

As a potential workaround:
Set PixelFormat to BayerX and the sink PixelFormat to BGR8.
You can do that in the sink constructor.
You will have higher cpu usage due to the conversion but lower network load when using this.

@TIS-Edgar TIS-Edgar added bug Something isn't working and removed question Request for information/clarifications. labels Jul 31, 2024
@g40
Copy link
Author

g40 commented Sep 12, 2024

@TIS-Edgar Any news on this one? Thanks.

@TIS-Edgar
Copy link
Member

No not yet, sorry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants