Refactored cpu backend: #121

[DavidIkov]

Now its easier to add new types of synapses and neurons. Code reading should be easier.

[artiomn]

Это возможно сделать не ошибкой в рантайме, а ошибкой компиляции.
И последнее будет предпочтительней.
Используй static_assert, например.

[DavidIkov]

тогда оно вообще не будет работать, AllVariants содержит то чего нету в SupportedTypes, и это нормально.
но в твоем случае это выдаст ошибку.
такая ошибка должна вылетать в рантайме, нельзя это определить во время компиляции

[a-vartenkov]

Мне, честно говоря, очень не нравится это dispatch, как будто он не вычисляет проекцию, а занимается чем-то другим, а вычисление проекции надо отдельно.

[DavidIkov]

почему другим? тут весь код вычисления проекции, больше кода с проекциями нету, отделять нечего, только если еще больше разбивать функцию

[a-vartenkov]

Я конкретно про слово dispatch в названии функции. Когда пользователь её вызывает, он по идее ожидает что она что-то продиспетчеризует - направит, вышлет или распределит, нет? А она возьмёт и вычислит что-то вместо этого. У нас там есть типа dispatch messages, и оно раскидывает сообщения, а это по идее рассчитывает.

[DavidIkov]

согласен, если просто смотреть на название то может возникнуть недопонимание, но это не функция которую будет вызывать пользователь, это просто функция распределитель, она вызывается другими распределителями, пользователь ее впринципе не вызывает.
но можно было бы придумать как назвать получше, мб что то по типу "calculate_projection__dispatch" что бы разделить смысл?

[a-vartenkov]

Не знаю, я бы может быть убрал dispatch вообще, она вычисляет проекцию. На самом деле, я не знаю, calculate_general_projection или что-то подобное, может быть? Потому что да, по названию это вызов какой-то диспетчеризации, а вызывается вычисление, название обманчивое.