gemini-testing / looks-same

Node.js library for comparing images
MIT License
660 stars 55 forks source link

Add ability to ignore image antialiasing #19

Closed sipayRT closed 7 years ago

sipayRT commented 7 years ago

// cc @SevInf @j0tunn

close #12

SevInf commented 7 years ago

@sipayRT а не меряли, как оно на скорость влияет?

sipayRT commented 7 years ago

ну оно явно будет дольше из-за того, что дополнительные проверки проверяются. Но должно не сильно просесть. Я погоняю еще отдельно разные кейсы, чтобы замерить

j0tunn commented 7 years ago

В остальном /ok

@SevInf

а не меряли, как оно на скорость влияет?

по идее не должны заметить вообще, т.к. до этого дело доходит только если есть отличающиеся пиксели:

SevInf commented 7 years ago

Я бы еще ссылку на статью куда-нибудь в коменты добавил, чтоб понятно было откуда это все это взялось.

sipayRT commented 7 years ago

Я бы еще ссылку на статью куда-нибудь в коменты добавил, чтоб понятно было откуда это все это взялось.

добавил ссылку прям в доку. так норм? или прям в код вхерачить?