Load neighbours with the fss hash except dublicated neighours. #112
Conversation
Rewrite test for look-a-like functional. Current tests contain correlation columns and queries have more nodes and description features. Add aqo_k as custom parameter to define few number of features for prediction. Its default value is 3. Queries can contain a larger number of features than 3 especially generic queries. Also add predict_a_few_neibours parameter for switch avalable to predict a few neibors than 3. It is done for not to change the previous logic of the code
Alena0704
force-pushed
the
13-look-a-like
branch
from
913e727 to
ea8e2ef
Compare
aqo.c
Outdated
| PGC_USERSET, | ||
| 0, | ||
| NULL, | ||
| lc_assign_hook, |
There was a problem hiding this comment.
Зачем здесь эта ссылка на функцию?
There was a problem hiding this comment.
Поправила,ссылки нет.
| NULL, | ||
| NULL); | ||
|
|
||
| DefineCustomBoolVariable("aqo.predict_with_few_neighbors", |
There was a problem hiding this comment.
Этот параметр нужен только для тестирования или зачем? Стоит откомментировать его , иначе непонятна необходимость.
There was a problem hiding this comment.
Параметр нужен для того, чтобы не считались предсказания, если найдено меньшее количество соседей, чем задано в параметре min_neighbors_for_predicting. Без этой настройки, aqo сразу делает предсказание при одном найденном соседе, что может быть в некоторых случаях неверно.
Тут дается дополнительная возможность дождаться, когда aqo обучиться и запросов в базе знаний станет достаточно для получения средней оценки, например, на основе трех соседей.
| /* Machine learning parameters */ | ||
|
|
||
| /* The number of nearest neighbors which will be chosen for ML-operations */ | ||
| int aqo_k = 3; |
There was a problem hiding this comment.
Должно быть задано значение по-умолчанию, которое совпадает с умолчальным в точке объявления GUC'a - иначе тесты мастера не пройдет.
There was a problem hiding this comment.
Да, значение по умолчанию берется равным 3.
Rewrite test for look-a-like functional. Current tests contain correlation columns and queries have more nodes and description features.
Add aqo_k as custom parameter to define few number of features for prediction. Its default value is 3. Queries can contain a larger number of features than 3 especially generic queries.
Also add predict_a_few_neibours parameter for switch avalable to predict a few neibors than 3. It is done for not to change the previous logic of the code