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

Pull up Ticket names from their descriptions #5

Closed nikita-p closed 1 year ago

nikita-p commented 1 year ago

Возможно ли подтягивать имена тикетов из их описаний в классе TicketType? Это не особо важно наверное, но в принципе, было бы неплохо, чтобы уменьшить вероятность ошибки, т.к. сейчас часть имён тикетов берётся из TicketType, а часть из поля name описаний конкретных тикетов

https://github.com/caenHV/Setup/blob/665e26c95a9f8acb424c81ec6faf388713dc33ed/caen_setup/Tickets/TicketType.py#L7

AliensWithdrawPeople commented 1 year ago

Теперь (с коммита b361fca) имена подтягиваются из Ticket.type_description(). Возможно, есть смысл обратной связки (подтягивать в Ticket.type_description() из TicketType), но как это сделать непонятно. В тупом подходе там появляется круговой импорт. Подумаю как это реализовать.

nikita-p commented 1 year ago

Ага, ещё, как кажется, остаётся проблема с десериализацией, т.к. там имена тикетов берутся из Enum

https://github.com/caenHV/Setup/blob/b361fcaffc0d1dbd14f445f91092e07748f06b21/caen_setup/Tickets/TicketMaster.py#L16