biometricITC / cPP-toolboxes

This repository maintains the toolbox definitions for evaluations according to the biometric cPP
2 stars 1 forks source link

Face toolbox layout #35

Closed woodbe closed 4 years ago

woodbe commented 4 years ago

I think we need to re-think how the face toolbox should be laid out. My concern is in large part because in general it would seem that most sensors would need to do both the 2D and the 3D tests (since we are requiring that 2D be subjected to 3D artifacts), but laying them out in separate folders makes it seem like I would just use the one applicable to my sensor type.

I think we should create one "Face_Toolbox" and then under that in the Face_Overview we would list out the differences between them and how we expect them to work. I would also list out a new Verification list that would have 3 columns that would show the type of sensor and what specific attacks should be used for each one (i.e. in the 2D/Mixed/Unknown column all would be selected, but the 3D only would only have the 3D ones selected).

Its OK to have references split out and even separate overviews, as long as there is an overarching overview. My issue now is that to figure out the test I would have to do, I need to read both face toolboxes, but it isn't clear anywhere that this is what I would need to do until I dig further into the docs. I think this should be out front in its own FACE Overview, pointing to the 2D and 3D for more of the detailed info. Basically I would take the overview sections of the two docs and combine them into one, then leave that out of the 2D/3D docs.

I would also place this all under one single set of FACE, not 2D and 3D. I have renamed the future repository to Face and removed the references to 2D/3D for this. I think this will make things easier.

woodbe commented 4 years ago

I think we can work on the face as it currently is and then do this when we think the overall content is complete (as opposed to trying to do this at the same time).

woodbe commented 4 years ago

The Face toolbox has been moved to here https://github.com/biometricITC/Face-Toolbox