Skip to content

Comment on How Do You Look When Merging Fails

Comments

I like how you can tweak the timing of the camera with the EMOTIONAL_SLUGGISHNESS_RATE

This was my first thought before I looked at the code. The constant name is perfect and intuitive. Great work.

The constant name is perfect and intuitive

I might have gone with DELAY_BETWEEN_DETECTION_AND_PICTURE, because it describes what the constant is used for and does not describe it as a "rate", which suggests some sort of x / t measurement where t is a unit of time.

Yes, but it wouldn't have made me laugh...

Sure, if you have no sense of humor...

Gotta ruin everyone's fun?

However correct you are! Maybe EMOTIONAL_DELAY?

(oh the implications!)

AboutSource Built by g1lg1l

Hackerly is an independent reader for Hacker News, built on the public HN API. Not affiliated with Y Combinator.