Created UserManager #46
Conversation
src/unit/unit_wrapper.py
Outdated
|
|
||
|
|
||
| UnitWrapper = UnitWrapper() | ||
| BaseUnitWrapper = UnitWrapper() |
There was a problem hiding this comment.
а был смысл его переименовывать? думаю нет
There was a problem hiding this comment.
Такая штука была, что импортился класс UnitManager, а не его экземпляр. Поэтому переименовал
src/unit/UnitManager.py
Outdated
| self.collection = "unitData" | ||
| self.database = BaseMongoDbWrapper |
There was a problem hiding this comment.
лучше сделать как аттрибут класса
class UnitManager:
collection = "..."
wrapper = "..."
src/unit/UnitManager.py
Outdated
| @lru_cache(maxsize=4) | ||
| def _get_cur_unit(self) -> Unit: |
There was a problem hiding this comment.
я уверен что эта штука мега жидко обосрется, если и делать кеш - то его надо как-то инвалидировать когда юнит снимается с воркбенча например, а тут этого нет
There was a problem hiding this comment.
вообще можно поиграться и сделать просто current_unit как property, будет ярко
There was a problem hiding this comment.
Кстати, я читал, что запросы в монгу по дефолту кешируются. Что скажешь по этому поводу?
src/unit/UnitManager.py
Outdated
| translation("Component") | ||
| + " " | ||
| + component.model_name | ||
| + " " | ||
| + translation("NotPartOfUnit") | ||
| + " " | ||
| + self.model_name |
There was a problem hiding this comment.
раз уж переделываем сделай тут через f-strings пожалуйста)))
хотя в этом случае хорошо будет смотреться .format()
src/unit/UnitManager.py
Outdated
| ) | ||
| metrics.register_complete_unit(None, self) | ||
|
|
||
| self.employee = None |
There was a problem hiding this comment.
почему-то в конце нигде нет пустых строк) а они должны быть в конце любого .py файла, гитхаб не просто так этот значок ставит
…aemon into ref/unit-mongo
No description provided.