Maarten Vandersteegen

Results 10 comments of Maarten Vandersteegen

I had the same error with TF2.4.0 and TF2.5.0. Downgrading to TF2.3.0 fixed the problem. I guess this repo needs an upgrade tot support the later TF releases.

Just a small note: code coverage is only measured on unittests, cannot be fixed by writing compliance tests (this is meant to be like this)

Nice, be carefull though if you generate HTML reports. I see the default html report of gcovr only shows line coverage. The tool can handle branch coverage so you should...

@frederikvs We should! But it's very hard to garantee that since we experienced warnings can pop up all of a sudden even when you change configuration (module configuration), not to...

Thanks for your contribution. I will merge your code after my holidays.

The lib does not support it (yet). I updated the readme with this info. Any contributions are welcome.

Did you try this from a linux virtual machine or native linux? What camera type did you use?

Nope, sorry it is known not to work stable via a virtual machine. I'm currently not planning to provide support for VM's.

Hi @chinhuang007. Thanks for the detailed response. I actually don't mind that ONNX is demanding NCHW. It actually think this is a good thing because it standardizes the exchange format....

+1 for branch coverage