caenHV / Setup

Package that offers tools to communicate with new HV power supply of CMD-3 drift chamber (hope so).
0 stars 0 forks source link

Uniform output of Ticket.execute #11

Closed AliensWithdrawPeople closed 11 months ago

AliensWithdrawPeople commented 1 year ago

может сделаем все методы execute у тикетов возвращающими ответ в виде json в едином формате?

проблема, что сейчас тикеты ведут себя разным образом (т.е. GetParams_Ticket.execute возвращает ответ, а остальные тикеты ничего не возвращают)

можно например общую структуру ответа выстроить так:

{
    "status" : "ok",
    "body" : {},
}

и например для всех тикетов кроме GetParams_ticket возвращать пустой body, а для GetParams_ticket вписывать туда твой res dict, который ты сейчас отдаёшь

и нужно обсудить момент ещё, где перехватывать ошибки от CAEN'а

_Originally posted by @nikita-p in https://github.com/caenHV/Setup/pull/9#discussion_r1404002326_

AliensWithdrawPeople commented 11 months ago

Теперь (с коммита 7661523) Ticket.execute возвращает json строку в формате

{
    "status" : bool,
    "body" : {
         "error" : *error info*,<- in case if "status" == False
         ...
    }
}

Отразил эти изменения в Readme для Tickets.

AliensWithdrawPeople commented 11 months ago

Вроде, всё.