Как правильно сделать запрос gPRC Tinkoff.InvestApi методу GetTechAnalysis?

Question

[Bodden]

Здравствуйте.
Пробую отправить запрос к методу GetTechAnalysis, но получаю ошибку

Unhandled exception. System.ArgumentException: Conversion from DateTime to Timestamp requires the DateTime kind to be Utc (Parameter 'dateTime')
   at Google.Protobuf.WellKnownTypes.Timestamp.FromDateTime(DateTime dateTime)
   at Program.Main(String[] args) in C:\C#_DEV\Test\TinkofTest\Program.cs:line 14

Что не так с Timestamp, не ка не пойму,
Сам код:

using Google.Protobuf.WellKnownTypes;

internal class Program
{
    private static void Main(string[] args)
    {
        var settings = new Tinkoff.InvestApi.InvestApiSettings()
        {
            AccessToken = "token"
        };

        var client = Tinkoff.InvestApi.InvestApiClientFactory.Create(settings);

        var getTechAnalysis = client.MarketData.GetTechAnalysis(new Tinkoff.InvestApi.V1.GetTechAnalysisRequest
        {
            IndicatorType = Tinkoff.InvestApi.V1.GetTechAnalysisRequest.Types.IndicatorType.Rsi,
            InstrumentUid = "a22a1263-8e1b-4546-a1aa-416463f104d3",
            From = Timestamp.FromDateTimeOffset(new DateTime(2023,12,31)),
            To = Timestamp.FromDateTime(DateTime.Now),
            Interval = Tinkoff.InvestApi.V1.GetTechAnalysisRequest.Types.IndicatorInterval.OneDay,
            TypeOfPrice = Tinkoff.InvestApi.V1.GetTechAnalysisRequest.Types.TypeOfPrice.High,
            Length = 96,
            //Deviation =
            //{
            //    DeviationMultiplier =
            //    {
            //        Units = 266,
            //        Nano= 905
            //    }
            //}


        }); ;

        foreach (var techAnalysis in getTechAnalysis.TechnicalIndicators)
        {
            Console.WriteLine(
                $"{techAnalysis.Timestamp} {techAnalysis.MiddleBand}");
        }
    }

}

Deviation
закомментировал, по причине возврата NULL.

Подскажите пожалуйста.

Answer 1

[#]

а так?

From = Timestamp.FromDateTimeOffset(new DateTime(2023, 12, 31).ToUniversalTime()),
To = Timestamp.FromDateTime(DateTime.UtcNow),

у меня ваш код компилируется и исполняется с отказом авторизации, что логично..
.. а по сигнатуре метода из нугет пакета (текущий релиз) то что у вас закомментировано, вроде как и не к месту..
.. если что пишите, обсудим (в профиле у меня телега..

ps что то мне сдается, что по сигнатуре скорее так

From = Timestamp.FromDateTime(new DateTime(2023, 12, 31).ToUniversalTime()),
To = Timestamp.FromDateTime(DateTime.UtcNow),

проDeviation пока не понял..

pps offset все таки смещение/разница, а методы api содержат вменяемые описания на нашем родном. оба параметра предполагают точечное указание даты. формат локального и универсального времени один, разница +3 часа для Мск. это не могло быть причиной ошибки. думаю все таки не удачный выбор метода (смещение/разница).. типа вы запросили 2023 года, ни то в прошлое, ни то в будущее..

Comments

[Bodden]

Я тоже чуток стал догадываться

From = Timestamp.FromDateTime(DateTime.UtcNow.Date.AddDays(-700)),

Но по мне это костыль, ваш изящней выглядит.

[#]

Bodden, в общем случае, думаю "костыль" сгодится.. это смотря какую задачу, и с какой логикой решаете ))
.. я лишь искал что не так, могло быть в вашем коде ))

[#]

Bodden, "костыль" очень даже логичен из принципа "за последние 2 года".. ну разаве что 365*2 ))
ну и как вариант (просто по моему ходу мыслей лучше читается) можно так

From = Timestamp.FromDateTime(DateTime.Now.Date.AddDays(-365*2).ToUniversalTime()),

... даже собсвенный код, допустим через полгодика, не всегда понятен с полувзгляда ;))

[#]

Bodden, а так логика еще прозрачнее .. и точнее (мало ли там високосные, пусть рантайм умничает ;)

From = Timestamp.FromDateTime(DateTime.Now.Date.AddYears(-2).ToUniversalTime()),

[#]

Bodden, и еще, приведение типа к самому себе излишне ;)))

From = Timestamp.FromDateTime(DateTime.Now.AddYears(-2).ToUniversalTime()),

(.Date)

[#]

Bodden, ну а если это фильтрация даты, то самое читаемое

From = Timestamp.FromDateTime(DateTime.Now.AddYears(-2).ToUniversalTime().Date),
To = Timestamp.FromDateTime(DateTime.Now.ToUniversalTime().Date),

это прочтете и через годы ;))
... хотя тут уже еще напрашивается оптимизация.. но это уже перфекционизм махровый начнется ..вынос за пределы вызова ))

var now = DateTime.Now.ToUniversalTime().Date;
...
From = Timestamp.FromDateTime(now.AddYears(-2)),
To = Timestamp.FromDateTime(now),
...

.. зато читаемо.. и меньше места для ошибок ;))

[#]

var now = DateTime.UtcNow.Date;
...
From = Timestamp.FromDateTime(now.AddYears(-2)),
To = Timestamp.FromDateTime(now),
...

.. равнозначно... чисто вопрос читаемости...

[#]

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

[#]

Bodden, сорян.. просто увлекся вопросом.. и еще мысль, если интересуют транзакции именно по РФ, то все таки логичнееvar now = DateTime.Now.Date.ToUniversalTime();... utc лишь формат (часовой пояс точнее, формат тот же) удобный для линукс/юникс систем/бд... а если интересуют торги в РФ, то логично все таки сначала дату, и только потом приведение к Гринвичу ..
... но думаю увидели, что в рамках одного и того же формата, расширениями можно играть как удобно..

[#]

Bodden, спасибо за отметку )).. снова вспомнил.. и опять перфекционизм..

var now = DateTime.Now.Date;
        // далее с этим работаете по логике задачи. но это именно Ваш часовой пояс
        // можно добавить переменную для "глубины истории" )))
        var getTechAnalysis = client.MarketData.GetTechAnalysis(new Tinkoff.InvestApi.V1.GetTechAnalysisRequest
        {
            IndicatorType = Tinkoff.InvestApi.V1.GetTechAnalysisRequest.Types.IndicatorType.Rsi,
            InstrumentUid = "a22a1263-8e1b-4546-a1aa-416463f104d3",
            // а к UTC логичнее приводить только там где это критично )))
            From = Timestamp.FromDateTime(now.AddYears(-2).ToUniversalTime()),
            To = Timestamp.FromDateTime(now.ToUniversalTime()),
            Interval = Tinkoff.InvestApi.V1.GetTechAnalysisRequest.Types.IndicatorInterval.OneDay,
            TypeOfPrice = Tinkoff.InvestApi.V1.GetTechAnalysisRequest.Types.TypeOfPrice.High,
            Length = 96
            // Deviation вот с этим не копал, сорян
        });

Answer 2

[Василий Банников]

Вроде же чёрным по белому написно:

Conversion from DateTime to Timestamp requires the DateTime kind to be Utc (Parameter 'dateTime')

Timestamp.FromDateTime(DateTime.Now)
Вот тут надо UtcNow

Timestamp.FromDateTimeOffset(new DateTime(2023,12,31)),

и вот тут явно указать kind

Comments

[Bodden]

Пробовал так:

From = Timestamp.FromDateTime(DateTimeKind.Utc(2023)),

Сейчас ругается на параметр
From

Unhandled exception. Grpc.Core.RpcException: Status(StatusCode="Unauthenticated", Detail="40003")