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

Рефакторинг /decl/observ/proc/raise_event #1953

Closed
Epicus7 opened this issue Apr 6, 2020 · 4 comments · Fixed by #7842
Closed

Рефакторинг /decl/observ/proc/raise_event #1953

Epicus7 opened this issue Apr 6, 2020 · 4 comments · Fixed by #7842
Assignees
Labels
:suspect: сложно Сложно, но можно справиться, если ты робастный 💰 награда За эту задачу выставлена награда 🔥 приоритет Билд неиграбелен из-за этого. 🔧 оптимизация Улучшения производительности 🔩 улучшение Новая фича или улучшение старой

Comments

@Epicus7
Copy link
Member

Epicus7 commented Apr 6, 2020

Вся система обсервов просто ебанутая, нужно рефакторить или заменять на сигналы с тг

Этот прок постоянно вылезает в топах профайлера:
profile.txt

Нужно оптимизировать работу прока, либо уменьшить количество его вызовов.

Чтобы закрыть данный иссуй нужно разобраться в тонкостях работы системы ивентов, разобраться почему она так аффектит производительность, придумать способ оптимизировать этот прок, не потеряв функциональность, или показать, что оптимизировать его без потери функционала невозможно.


За закрытие этой задачи выставлена награда размером в 1000 рублей!

Чтобы получить награду необходимо открыть пуллреквест с улучшениями описанными в задаче, который пройдет ревью двух разработчиков билда. После того как пуллреквест будет принят, а данная задача будет закрыта, пишите Epicus#1570 (Discord) для получения вознаграждения.

  • Если вы хотите заняться данной задачей, то вы можете её забронировать на какое-то время, но в случае отсутствия активности, бронь будет сниматься.
  • Обратите внимания на комментарии к задаче. Там могут быть пояснения и уточнения о том как её следует выполнять.
@Epicus7 Epicus7 added 🔩 улучшение Новая фича или улучшение старой 🔧 оптимизация Улучшения производительности 🔥 приоритет Билд неиграбелен из-за этого. labels Apr 6, 2020
@Epicus7 Epicus7 added the :suspect: сложно Сложно, но можно справиться, если ты робастный label May 16, 2020
@ChaoticTheBot ChaoticTheBot added the 💰 награда За эту задачу выставлена награда label May 19, 2020
@ChaoticTheBot
Copy link
Collaborator

За закрытие данной задачи выставлена награда размером в 500 рублей!

Подробности в ОП-посте.

@Epicus7
Copy link
Member Author

Epicus7 commented Jul 23, 2020

Награда за закрытие иссуя увеличена с 500 рублей до 1000!

@Pl3aseJust3ndMySuffering
Copy link
Contributor

Беру.

@igorsaux igorsaux self-assigned this Mar 28, 2022
@Epicus7
Copy link
Member Author

Epicus7 commented May 27, 2022

Награда выдана

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:suspect: сложно Сложно, но можно справиться, если ты робастный 💰 награда За эту задачу выставлена награда 🔥 приоритет Билд неиграбелен из-за этого. 🔧 оптимизация Улучшения производительности 🔩 улучшение Новая фича или улучшение старой
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants