added loading telemetry from recent TLM text files #3
Conversation
There was a problem hiding this comment.
решение требует генерализации. нет смысла иметь поддержку только одного типа файлов - нашему выходному файлу телеметрии.
потому:
- обобщить, читая файлы построчно
- совместить с уже имеющимся функционалом загрузки HEX значений из поля ввода
- соответствующе обозвать все функции, строки и тд
- кодстайл
- не забываем про документацию
| else: | ||
| self.stop() if self.sk else self._start() | ||
|
|
||
|
|
| @@ -579,13 +579,24 @@ def __init__(self, master, config, name): | |||
| self.dv_frame = DataViewFrame(self) | |||
| self.dv_frame.grid(column=1, row=0, rowspan=2, sticky=tk.NSEW, padx=2, pady=2) | |||
|
|
|||
| def openfiles(self): | |||
| path = HOMEDIR | |||
| files = filedialog.askopenfilenames(title='Choose packets', initialdir=path, filetypes=(("TLM file", "*.txt"), ("All files", "*.*"))) | ||
| for f in files: |
There was a problem hiding this comment.
тоже одноразовая files, можно прямо по результату функции итерироваться.
также askopenfilenames лучше заменить на askopenfiles, где открытие файла сделается само.
тут же - одна строка в одинарных кавычках, другие в двойных
| files = filedialog.askopenfilenames(title='Choose packets', initialdir=path, filetypes=(("TLM file", "*.txt"), ("All files", "*.*"))) | ||
| for f in files: | ||
| file = open(f, 'r') | ||
| line = file.readline().strip() |
There was a problem hiding this comment.
только одну строку? а если в файле их будет много?
в целом, итерация по строкам уже есть в _hex_values, надо лишь генерализовать, выведя в отдельную функцию для скармливания данных
There was a problem hiding this comment.
Предполагалось, что будут загружаться файлы телеметрии, уже созданные ранее декодером. А в них оригинальный пакет хранится в первой строке.
| @@ -579,13 +579,24 @@ def __init__(self, master, config, name): | |||
| self.dv_frame = DataViewFrame(self) | |||
| self.dv_frame.grid(column=1, row=0, rowspan=2, sticky=tk.NSEW, padx=2, pady=2) | |||
|
|
|||
| def openfiles(self): | |||
There was a problem hiding this comment.
функция внезапно оказалась выше всех остальных. лучше следовать порядку, в котором расположен список источника данных и разместить ее под _hex_values.
ну и название - мы таки открываем файлы или загружаем их?
| def named_conn_btn(self, _=None, **kw): | ||
| m = utils.ConnMode(self.conn_mode.current()) | ||
| d = { | ||
| utils.ConnMode.AGWPE_CLI: ('Connect', 'Disconnect'), | ||
| utils.ConnMode.TCP_CLI: ('Connect', 'Disconnect'), | ||
| utils.ConnMode.TCP_SRV: ('Start', 'Stop'), | ||
| utils.ConnMode.HEX: ('Run', 'Stop'), | ||
| utils.ConnMode.HEX_TXT: ('Load packets', 'Stop'), |
|
|
||
|
|
||
| con_mode_names = { | ||
| ConnMode.AGWPE_CLI: 'AGWPE Client', | ||
| ConnMode.TCP_CLI: 'TCP Client', | ||
| ConnMode.TCP_SRV: 'TCP Server', | ||
| ConnMode.HEX: 'HEX values', | ||
| ConnMode.HEX_TXT: 'HEX from TLM file', |
baskiton
force-pushed
the
dev
branch
7 times, most recently
from
7e664b3
to
d0262f4
Compare
baskiton
force-pushed
the
dev
branch
2 times, most recently
from
e3a040c
to
6558c00
Compare
No description provided.