search

ProteGO-Safe / specs

Opis, specyfikacja i zadania. Zacznij tutaj.
GNU General Public License v3.0
117 stars 29 forks source link

Wydajność Proxy i darmowego API Intermedica #166

Closed potiuk closed 4 years ago

potiuk commented 4 years ago

Describe the bug Zgodnie z dyskusją w #162 wydaje się że aplikacja nie jest przemyślana pod kątem wydajnościowym jeśli chodzi o moduł diagnostyczny.

Czekamy jeszcze na #155 żeby powiedzieć coś więcej, ale z pobieżnych wyliczeń (opartych na zgadywaniu) przedstawionych w https://github.com/ProteGO-Safe/specs/issues/162#issuecomment-626678774 wynika że przy sukcesie aplikacji, zarówno Darmowe API Intermedica jak i proxy (NGINX) ProteGO będą miały spory problem wydajnościowy.

Obliczenia 'na odwrocie koperty" wskazują że mówiimy o potecjalnym ruchu 40.000 żądań na sekundę w "peaku".

To Reproduce

Powtórzenie obliczeń z https://github.com/ProteGO-Safe/specs/issues/162#issuecomment-626678774 z mniej "zgadniętymi" wartościami.

Expected behavior Liczby które wyjdą powinne być zweryfikowane pod kątem wydolności zastosowanych rozwiązań.

MateuszRomanow commented 4 years ago

Informacyjnie - w poniedziałek dostaliśmy zgodę z MC na przepisanie modułu do samooceny tak, żeby funkcjonalność działała lokalnie. Jest to w procesie. Wspominam o tym tutaj, ponieważ bezpośrednio wpływa to na dezaktualizowanie wątków związanych z wydajnością API. Produkcyjne wdrożenie będzie w wersji 4.0.0.

kierepka commented 4 years ago

Ok - pytanie czy będzie dostępny moduł samooceny w formie kodów?

potiuk commented 4 years ago

Dzięki za informację @MateuszRomanow. Zamknę jak będzie wdrożone.

porz commented 4 years ago

Ja tylko informacyjnie powiem, że Infermedica nie jest zaangażowana w tworzenie wersji offline. O decyzji dowiaduję się z tego wątku, ale koncepcyjnie zgadzam się, że jest to właściwy kierunek w przypadku aplikacji ProteGO.

Dziękuję zatem wszystkim za dyskuję i będę trzymał kciuki za powodzenie projektu.

MateuszRomanow commented 4 years ago

@kierepka tak, na pewno trafi do repo w tej organizacji. @porz w wątku MC<>Infermedica ten temat czeka na ruch ze strony MC. Tak więc my też czekamy, niezależnie będziemy się kontaktować z Tobą/Wami

KoderFPV commented 4 years ago

Z powodu dłuższej nieaktywności w tym wątku, zostanie on automatycznie zamknięty w najbliższym czasie.

Jeżeli ktoś ma coś przeciwko to niech da znać, lub założy wątek zgodny z obecnymi założeniami. Ocena ryzyka została zaimplementowana lokalnie w 4.1.1

SeraMoon commented 4 years ago

@Tarvald skoro firma ta nie świadczy nam usług - temat jest chyba zamknięty? @potiuk miałeś coś jeszcze na myśli w tym temacie odnośnie Proxy w stanie z wersji 4.1.1?

KoderFPV commented 4 years ago

@SeraMoon Racja, zamykam.