Неправильное формирование свечек, несколько ТФ, много стратегий

1. На счет объема не понял - он в лог не выводится... А в чем кривость свечек то?
2. А в чем разница между локальным компьютером и том, на котором лог писался?

1. свечка в 10:45 должна быть: открытие 139920 макс 139965 мин 139870 закрытие 139800 объём 6957 (никак не 474 по логу (объём дописал потом))

т.е. совсем другие данные

2. Никакой разницы нет,время синхронизировано с сервером и стоит одна и таже зона - Москва.

Если запрашивать через диапазон new Range(firstTime, marketTime), то можно захватить еще не сформированную текущую свечку. Случаем, не это ли случилось?

Нет, не это. Все остальные свечки после включения роботов (в 10:18) тоже кривые, всё они неверные - неправильно всё, от минимума и максимума до закрытия\открытия и объёма.

Так Вы же пишите, что включения робота свечки верные. Первые потому, что они полностью сформированы, и трюк к запросом всех свечек по текущее время прокатит. А у Вас наверное запрос свечек идет в OnProcess. И на каждой итерации будет захватываться еще не сформированная свечка... Откуда у Вас вызывается получение свечек?

Да, свечки получаются через var oldCandles = _candleManager.GetTimeFrameCandles(Security, TimeFrame, new Range<DateTime>(firstTime, Trader.MarketTime));

Но дальше я ведь пробегаю по всем этим свечкам в цикле и печатаю. Печатаются из цикла неверные значения для всех (!) свечек, не только для последней (последнюю я как раз и не смотрю) - для всех от 10:20 до 10:45. Печать, которую дал в первом посте была сделана в 10:50:01, просто убрал это из сообщения.

Если убрать всё ненужное, то вот печать:

                      var oldCandCount = oldCandles.Count();
                        while (it <= oldCandCount)
                        {
                            localCandle =

oldCandles.ElementAt(oldCandCount - it); if (localCandle != null) { AddLog(StrategyErrorStates.None, "localCandle: time({0}), open({1}), close({2}), high({3}), low({4}); " +

"TimeFrame: {5}, TotalVolume({6}), TotalPrice({7})", new object[] { localCandle.Time.TimeOfDay, localCandle.OpenPrice, localCandle.ClosePrice, localCandle.HighPrice, localCandle.LowPrice, localCandle.TimeFrame, localCandle.TotalVolume, localCandle.TotalPrice }); } it++; }

Ок... Еще по логу можете посмотреть, менялись ли старые свечки? Или на всех итерациях неправильное значение?

Просто поправка по коду. Как то сложно пишите. Может вот так?

foreach (var localCandle in oldCandles.OrderByDescending(c => c.Time))

Ух, так и правда проще, спасибо. Нет, не менялись - начиная с первой новой свечки (которая формируется по ходу стратегии, а не до запуска) - неправильные значения. На локальном компьютере с 1м квиком - всё нормально

Так, разница между тачками все-таки есть - 1 квик и несколько?

У меня просто там много дополнительных условий в цикле, наверное на foreach не получится так просто заменить. Попробую в любом случае =)

Чем проще цикл, тем меньше ошибок... Так что насчет разницы между тачками?

Да, на сервере 6 квиков, на локальной машине - 1 квик. И там и там идёт работа через MultiTrader, программа-робот одна и таже.

Ха, я кажется понял в чем фишка. Один Квик строит свечки последовательно, по сделкам. 6 квиков посылают сделки параллельно. Как следствие, неправильные свечки. Алго построения свечек основан на том, что как только появляется сделка из следующего ТФ, то текущая свечка помечается как сформированная и к ней более обращений не идет.

Понял, похоже на мои предположения. Буду ждать фикса тогда, сейчас с несколькими квиками не получается иначе работать :(

Хм, а вот тут не все так просто. Я пока даже предствить себе не могу, как это фиксить. Алгоритм свечной не поддерживает такое поведение... Может быть формировать свечки по конретным QuikTrader?

Я правильно понимаю что для каждого QuikTrader надо создать свой CandleManager и регистрировать необходимые ТФ? Если так, то не проблема, конечно сделаю.

Чтобы закрыть эту тему - реализовал именно таким образом сейчас, завтра протестирую

Михаил, задам вопрос тут, дабы не плодить множество тем - надо ли дополнительно проверять IsTimeFrameCandlesRegistered перед регистрацией таймфрейма или это уже запрятано внутрь RegisterTimeFrameCandles и повторной регистрации не произойдёт в любом случае?

Надо. Иначе будет исключение. Специально сделал чтобы ловили с своих роботах код, который норовит зарегистрировать лишнее.