Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
От:
Sinix
Дата: 28.04.16 17:39
Оценка:
Здравствуйте, AndrewVK, Вы писали:
AVK>Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.
Баг нашёлся в MaxBy/MinBy.
Тесты скинул в master, билд на тестах завалится. Если так делать не надо — fail-тесты буду отключать.
Здравствуйте, Sinix, Вы писали:
S>Скинул, в master, билд на тестах завалится. Если так делать не надо — fail-тесты буду отключать.
Отключать не надо, если по делу. А вот касательно того как правильно интерпретировать NaN есть большие вопросы.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
Здравствуйте, 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. Такие дела.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
От:
Sinix
Дата: 28.04.16 17:52
Оценка:
Здравствуйте, AndrewVK, Вы писали:
AVK>Отключать не надо, если по делу. А вот касательно того как правильно интерпретировать NaN есть большие вопросы.
Ну главное, чтоб одинаково было. Я туда сам не полезу — чувствуется, что код долго полировался
Не хочется портить исходную идею.
Но есть предложение: сделать по аналогии с перегрузками для nullable-значений, они пропускают все первые null.
От:
Sinix
Дата: 28.04.16 17:58
Оценка:
Здравствуйте, AndrewVK, Вы писали:
AVK>min1 = min2 = NaN. max1 = max2 = 6. Такие дела.
Ну да, min/max Comparer<T> используют. У генерик-версий Min/MaxBy будет такое же поведение.
Здравствуйте, Sinix, Вы писали:
S>Но есть предложение: сделать по аналогии с перегрузками для nullable-значений, они пропускают все первые null.
Согласен. Сейчас подпилю генератор
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
Здравствуйте, 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?
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Здравствуйте, Sinix, Вы писали:
S>Но есть предложение: сделать по аналогии с перегрузками для nullable-значений, они пропускают все первые null.
Поправил.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
От:
Sinix
Дата: 28.04.16 18:45
Оценка:
Здравствуйте, rameel, Вы писали:
R>Или речь о генерик-версии Enumerable.Min/Max?
О генерик, про негенерики благополучно забыл
Здравствуйте, AndrewVK, Вы писали:
AVK>Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.
Ошибки исправил в методах MinByOrDefault и MaxByOrDefault, если еще не поздно, в релизную ветку закиньте
... << RSDN@Home 1.0.0 alpha 5 rev. 0>>
Здравствуйте, rameel, Вы писали:
R>Ошибки исправил в методах MinByOrDefault и MaxByOrDefault, если еще не поздно, в релизную ветку закиньте
Не поздно.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
От:
Sinix
Дата: 29.04.16 05:56
Оценка:
Здравствуйте, AndrewVK, Вы писали:
AVK>Я так понимаю, что больше желания ни у кого править текущую релизную ветку нет? Тогда выпускаем стабильный релиз.
Ещё один вопрос по свежедобавленному ExpressionExtensions (
обсуждение тутАвтор: IT Дата: 22.04.16
)
У нас есть CodeJam.Reflection.ExpressionHelper, надо бы их объединить до релиза, иначе потом ломающее изменение будет.
Обсуждать лучше в той ветке, сразу скажу, что у меня конкретных предложений пока нет
Выпустил.
... << RSDN@Home 1.0.0 alpha 5 rev. 0 on Windows 8 6.2.9200.0>>
От:
Lexey
Дата: 01.05.16 21:03
Оценка:
"Будь достоин победы" (c) 8th Wizard's rule.
Пока на собственное сообщение не было ответов, его можно удалить.
Удалить