[generator] Actualized and improved popularity generation. #13649
Conversation
maksimandrianov
force-pushed
the
popularity_improvements
branch
22 times, most recently
from
70721a0
to
e376fca
Compare
maksimandrianov
force-pushed
the
popularity_improvements
branch
from
e376fca
to
374cb8c
Compare
|
JTALL |
|
JTRGIT |
maksimandrianov
changed the title
| @@ -82,6 +86,12 @@ uint32_t GetMainType(FeatureParams::Types const & types) | |||
| return it != std::cend(types) ? *it : ftype::GetEmptyValue(); | |||
| } | |||
|
|
|||
| bool mainTypeIsBuildingPart(FeatureParams::Types const & types) | |||
There was a problem hiding this comment.
название с большой буквы?
| auto const & candidate = candidateNode->GetData(); | ||
| if (buildingPartChecker(place.GetTypes()) && | ||
| !(buildingChecker(candidate.GetTypes()) || buildingPartChecker(candidate.GetTypes()))) | ||
| if (mainTypeIsBuildingPart(place.GetTypes()) && |
There was a problem hiding this comment.
а есть вообще примеры когда building_part оказывался чем-то полезным?
из плохого -- мы начали выделять https://www.openstreetmap.org/relation/1834831 в отдельный объект
а примеры хорошего какие?
|
|
||
| auto hierarchyPlace = HierarchyPlace(fb); | ||
| if (!hierarchyPlace.IsPoint() && | ||
| base::AlmostEqualAbs(hierarchyPlace.GetArea(), 0.0, std::numeric_limits<double>::epsilon())) |
There was a problem hiding this comment.
это какой-то популярный вид мусорных объектов?
There was a problem hiding this comment.
Линейные объекты у нас могут быть HierarchyPlace? Вроде мы хотим сохранять условный арбат + никто не мешает повесить tourism=attraction на линейный.
Мы их тут случайно не выкинем?
В FilterComplexPopularity::IsAccepted вижу что линейные выкидываются, не для популярити тоже всегда будем выкидывать? Если да, то почему?
|
|
||
| bool FilterComplexPopularity::IsAccepted(feature::FeatureBuilder const & fb) const | ||
| { | ||
| if (!fb.IsArea() && !fb.IsPoint()) |
There was a problem hiding this comment.
а почему? бывают линейные объекты для которых мы хотим считать popularity, например
https://www.openstreetmap.org/way/385648322
такие объекты могут находиться внутри каких-то площадных
No description provided.