Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.

Здравствуйте, AndrewVK, Вы писали:

AVK>Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.

Баг нашёлся в MaxBy/MinBy.
Тесты скинул в master, билд на тестах завалится. Если так делать не надо — fail-тесты буду отключать.

Здравствуйте, Sinix, Вы писали:

S>Скинул, в master, билд на тестах завалится. Если так делать не надо — fail-тесты буду отключать.

Отключать не надо, если по делу. А вот касательно того как правильно интерпретировать NaN есть большие вопросы.

Здравствуйте, Sinix, Вы писали:

var min1 = new[] { 1, double.NaN, 2, 3, 4, 5, 6 }.Min();
var min2 = new[] { double.NaN, 1, 2, 3, 4, 5, 6 }.Min();
var max1 = new[] { 1, double.NaN, 6, 5, 4, 3, 2, 1 }.Max();
var max2 = new[] { double.NaN, 6, 5, 4, 3, 2, 1 }.Max();

min1 = min2 = NaN. max1 = max2 = 6. Такие дела.

Здравствуйте, AndrewVK, Вы писали:


AVK>Отключать не надо, если по делу. А вот касательно того как правильно интерпретировать NaN есть большие вопросы.

Ну главное, чтоб одинаково было. Я туда сам не полезу — чувствуется, что код долго полировался Не хочется портить исходную идею.

Но есть предложение: сделать по аналогии с перегрузками для nullable-значений, они пропускают все первые null.

Здравствуйте, AndrewVK, Вы писали:

AVK>min1 = min2 = NaN. max1 = max2 = 6. Такие дела.

Ну да, min/max Comparer<T> используют. У генерик-версий Min/MaxBy будет такое же поведение.

Здравствуйте, Sinix, Вы писали:

S>Но есть предложение: сделать по аналогии с перегрузками для nullable-значений, они пропускают все первые null.

Согласен. Сейчас подпилю генератор

Здравствуйте, Sinix, Вы писали:

S>Ну да, min/max Comparer<T> используют.

Разве? http://referencesource.microsoft.com/#System.Core/System/Linq/Enumerable.cs,1499

public static double Min(this IEnumerable<double> source) {
        if (source == null) throw Error.ArgumentNull("source");
        double value = 0;
        bool hasValue = false;
        foreach (double x in source) {
                if (hasValue) {
                        if (x < value || System.Double.IsNaN(x)) value = x;
                }
                else {
                        value = x;
                        hasValue = true;
                }
        }
        if (hasValue) return value;
        throw Error.NoElements();
}

Или речь о генерик-версии Enumerable.Min/Max?

Здравствуйте, Sinix, Вы писали:

S>Но есть предложение: сделать по аналогии с перегрузками для nullable-значений, они пропускают все первые null.

Поправил.

Здравствуйте, rameel, Вы писали:


R>Или речь о генерик-версии Enumerable.Min/Max?
О генерик, про негенерики благополучно забыл

Здравствуйте, AndrewVK, Вы писали:

AVK>Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.

Ошибки исправил в методах MinByOrDefault и MaxByOrDefault, если еще не поздно, в релизную ветку закиньте

Здравствуйте, rameel, Вы писали:

R>Ошибки исправил в методах MinByOrDefault и MaxByOrDefault, если еще не поздно, в релизную ветку закиньте

Не поздно.

Здравствуйте, AndrewVK, Вы писали:

AVK>Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.

Ещё один вопрос по свежедобавленному ExpressionExtensions (обсуждение тут

Автор: IT
Дата: 22.04.16

)
У нас есть CodeJam.Reflection.ExpressionHelper, надо бы их объединить до релиза, иначе потом ломающее изменение будет.

Обсуждать лучше в той ветке, сразу скажу, что у меня конкретных предложений пока нет

Выпустил.