Bump is:pano threshold to 2.0.

Previous value of 1.6 was too small because iPhone 5's aspect ratio
is 1.78, meaning all landscape iPhone 5 photos were included in
is:pano. Same with Moto-X and probably other newer phones.

I tried 1.8, but it still "felt" too small. I think the correct
value is atleast 2.0, but perhaps larger. I don't have enough test
data in that range right now to get a good feel for it.

Change-Id: I6a3e054290d6f434afac8142a9abc68e53008229
This commit is contained in:
Aaron Boodman 2014-02-08 14:41:37 -08:00
parent b0775e03e9
commit 5a1e831a7b
2 changed files with 2 additions and 2 deletions

View File

@ -158,7 +158,7 @@ func parseExpression(ctx *context.Context, exp string) (*SearchQuery, error) {
continue
}
if word == "is:pano" {
andWHRatio(&FloatConstraint{Min: 1.6})
andWHRatio(&FloatConstraint{Min: 2.0})
continue
}
if word == "has:location" {

View File

@ -62,7 +62,7 @@ var parseExprTests = []struct {
File: &FileConstraint{
IsImage: true,
WHRatio: &FloatConstraint{
Min: 1.6,
Min: 2.0,
},
},
},