Skip to content

Creating conversor from haarcascade to tracking.js array#164

Open
jayralencar wants to merge 1 commit into
eduardolundgren:masterfrom
jayralencar:haarcascade-conversor
Open

Creating conversor from haarcascade to tracking.js array#164
jayralencar wants to merge 1 commit into
eduardolundgren:masterfrom
jayralencar:haarcascade-conversor

Conversation

@jayralencar
Copy link
Copy Markdown

  • Converter from haarcascade to tracking.js array.
  • Can be called in browser or Node.js
var conversor = require('./haarcascade_conversor');

@@ -0,0 +1,62 @@
(function(){
'use strictc';
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small typo here "use strict"

* @param {Object} json as file opencv_haarcascade_eye.js
* @returns {Array}
*/
var conversor = function(json){
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i think it may be better to have a more explicit name, and in english. It would be easier to understand for the user

Maybe something like haarcascade_converter.js and function convertHaarCascase(){}

@jeromeetienne
Copy link
Copy Markdown
Collaborator

@eduardolundgren is there a difference with this code in assets ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants