Skip to content

GitLab

Merged
Created by Juan Jose Lopez@jjlopezMaintainer2 of 13 tasks completed2/13 tasks

Resolver fallos en la detección de hologramas y la demo comercial

🎯 Summary

Resolver los fallos genéricos que hemos ido viendo estos días:

  • El texto para hacer la validación de hologramas en la demo comercial se mostraba en inglés aún teniendo configurado el idioma español.
  • Si la validación de hologramas falla (aka el backend devuelve un code distinto de "OK"), se devuelve un error MobbScanAPIErrorCode_CONNECTION_ERROR, que aparte de ser poco descriptivo, es incompatible con MOBBSCAN-5927.
  • Si la validación de hologramas falla, se avisa al backend con el fallo genérico MSUnattendedVideoCancelReason_UNKNOWN, que no ayuda a saber la causa del fallo.
  • La vista de resultados de la demo comercial no mostraba correctamente los campos "Comparativa facial" y "Detección de vida": como estos valores antes no estaban en el proceso desatendido, en MOBBSCAN-5630 se sustituyeron por un icono de un reloj, pero ahora ya si se reciben estos valores.
  • El proceso desatendido falla indefinidamente con un CAPTURE_FACE_ERROR si anteriormente se ha abortado un proceso antes de que se envíe y borre el video del liveness.

🤔 Type

  • ✨ New feature
  • 🐛 Bug fix
  • 👷 Technical debt
  • ☢️ Breaking change (it breaks backward compatibility)
  • 📚 Requires a change in the docs

⚙️ Configuration

Workspace branch: [feature/MOBBSCAN-6012]

Related MRs:

⚗️ Solution

  • Corregido el texto en español para la validación de hologramas.
  • Añadido MobbScanAPIErrorCode_STAMP_DETECTION_ERROR y MSUnattendedVideoCancelReason_VERIFICATION_SERVER_CONNECTION_ERROR.
  • Añadido livenessScore a la API de MobbScan. Mirar MR adicional MOBBSCAN-6027.
  • Corregida vista de resultados de la demo comercial para que coja correctamente el score y el nuevo livenessScore.

🔗 JIRA: MOBBSCAN-6012

🔬 Tests

Nota: no hay forma que se me ocurra de forzar un fallo en el backend en la detección de hologramas, así que eso no se puede probar más que provocando un error artificialmente en el código.

  • Iniciar un proceso desatendido con una licencia que tenga detección de hologramas, algún tipo de liveness, y un dispositivo en español.
  • Comprobar que la detección de hologramas muestra el texto en español.
  • Comprobar que en la vista de resultados aparece correctamente si el liveness y el reconocimiento facial han sido correctos o no (score >= 0.5), en vez de la anterior vista con un reloj (el resultado no estaba disponible para cuando terminaba el proceso).

🧐 Checklist:

  • The code follow the project guidelines.
  • The changes have been tested locally.
  • The code has internal docs.
  • Docs are updated (README, public docs, etc.)
  • Some tests have been added to check that the changes work as expected (unit, integration or e2e tests ...)
  • Unit tests are running without errors.
  • Other dependant changes have already been merged.
  • Changelog updated (if it applies)
Edited by Juan Jose Lopez