-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Calibration #321
Calibration #321
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gloriajjl Я бы хотела обсудить способ хранения "calibration_params", чтобы в дальнейшем было понятнее, что это за числа. В комментариях к коду написала свои мысли.
Bumps [django](https://github.com/django/django) from 1.11.23 to 1.11.28. - [Release notes](https://github.com/django/django/releases) - [Commits](django/django@1.11.23...1.11.28) Signed-off-by: dependabot[bot] <[email protected]>
@gloriajjl after merge 'variant_init_ver' to 'calibration' Potential data loading is broken |
@elpiankova , здається я знаю в чому причина. Ти додала два нових каналаґи в init_data.json, line 992: один з lables: "pd/ekp/00", а другий "pd/nkp/0/". Перед цим ці шляхи для вичитки файлів даних проекта Потенціал здається не використовувались, а використовувались "ez/lf/0/" та "ez/hf/00". В ez папках обов'язково лежало два файли, які закінчувалися на *mv.csv та *mv.set. А по нових шляхах таких файлів нема, тому код класа Потенціал і падає з помилкою
Остання строчка якраз показує цю перевірку. |
@gloriajjl Предлагаю добавить в код исключение для этих каналов, чтобы не потерять уже эти данные в init. И загрузка данных для этих новых каналов делать в рамках другого issue #313 (другой ветки). Другой вариант - попробовать убрать коммит a8e96b3, но я не очень знаю как это делать (но наверное это можно сделать). Какой думаешь вариант лучше? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gloriajjl Я не смогла разгадать проблему. Но примерно, когда я скачиваю текстовые большие файлы, у меня иногда имя файла нормально в виде "1109784190_1109784457_E.txt", а иногда "undefined.txt".
Остальное - это мои недоработки в init файле. Готова апрувить
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
На файлы заведу отдельный issue.
Please ignore landswellsong commits in this branch...