Skip to content
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

Merged
merged 18 commits into from
Nov 20, 2020
Merged

Calibration #321

merged 18 commits into from
Nov 20, 2020

Conversation

gloriajjl
Copy link
Contributor

Please ignore landswellsong commits in this branch...

Copy link
Member

@elpiankova elpiankova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gloriajjl Я бы хотела обсудить способ хранения "calibration_params", чтобы в дальнейшем было понятнее, что это за числа. В комментариях к коду написала свои мысли.

backend/promis/backend_api/fixtures/init_data.json Outdated Show resolved Hide resolved
backend/promis/backend_api/fixtures/init_data.json Outdated Show resolved Hide resolved
backend/promis/classes/base_calibration.py Outdated Show resolved Hide resolved
@elpiankova
Copy link
Member

@gloriajjl after merge 'variant_init_ver' to 'calibration' Potential data loading is broken

@gloriajjl
Copy link
Contributor Author

gloriajjl commented Jul 7, 2020

@elpiankova , здається я знаю в чому причина. Ти додала два нових каналаґи в init_data.json, line 992: один з lables: "pd/ekp/00", а другий "pd/nkp/0/". Перед цим ці шляхи для вичитки файлів даних проекта Потенціал здається не використовувались, а використовувались "ez/lf/0/" та "ez/hf/00". В ez папках обов'язково лежало два файли, які закінчувалися на *mv.csv та *mv.set. А по нових шляхах таких файлів нема, тому код класа Потенціал і падає з помилкою

  File "promis/manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/usr/local/lib/python3.6/site-packages/django/core/management/__init__.py", line 364, in execute_from_command_line
    utility.execute()
  File "/usr/local/lib/python3.6/site-packages/django/core/management/__init__.py", line 356, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/usr/local/lib/python3.6/site-packages/django/core/management/base.py", line 283, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/usr/local/lib/python3.6/site-packages/django/core/management/base.py", line 330, in execute
    output = self.handle(*args, **options)
  File "/usr/src/app/promis/backend_api/management/commands/check_data_updates.py", line 49, in handle
    sat_obj.update()
  File "/usr/src/app/promis/classes/base_project.py", line 39, in update
    self.fetch(data_id)
  File "/usr/src/app/promis/classes/potential.py", line 104, in fetch
    assert(len(mvfile) == 1 and len(csvfile) == 1)

Остання строчка якраз показує цю перевірку.

@elpiankova
Copy link
Member

@gloriajjl Предлагаю добавить в код исключение для этих каналов, чтобы не потерять уже эти данные в init. И загрузка данных для этих новых каналов делать в рамках другого issue #313 (другой ветки). Другой вариант - попробовать убрать коммит a8e96b3, но я не очень знаю как это делать (но наверное это можно сделать). Какой думаешь вариант лучше?

Copy link
Member

@elpiankova elpiankova left a 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 файле. Готова апрувить

backend/promis/classes/base_calibration.py Outdated Show resolved Hide resolved
Copy link
Member

@elpiankova elpiankova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

На файлы заведу отдельный issue.

@gloriajjl gloriajjl merged commit 43d9116 into dev Nov 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants