Dodato jos frontova #2

2024-02-08T01:08:38Z

svitvojimilioni commented

2024-02-08 01:08:38 +00:00

Dodao sam jos frontova za aplikacije koje sam nasao ovde.

Izmenio sam malo mehaniku kako radi, pa se sada samo dopuni FRONTENDS dict i automatski ce poslati link ka libre verziji.

Zna da bude problema zbog www, na primer ako je www.imdb.com/film i uradi rewrite na www.fossimdb.com/film taj fossimdb mozda nema www verziju a mozda i ima. Ne pada mi napamet neko resenje trenutno, mislim i da se ne desava cesto.

Uglavnom probjate po kanalu, bacite review na kod pa ako je sve ok da ide na master

Dodao sam jos frontova za aplikacije koje sam nasao [ovde](https://farside.link). Izmenio sam malo mehaniku kako radi, pa se sada samo dopuni `FRONTENDS` dict i automatski ce poslati link ka libre verziji. Zna da bude problema zbog `www`, na primer ako je `www.imdb.com/film` i uradi rewrite na `www.fossimdb.com/film` taj fossimdb mozda nema www verziju a mozda i ima. Ne pada mi napamet neko resenje trenutno, mislim i da se ne desava cesto. Uglavnom probjate po kanalu, bacite review na kod pa ako je sve ok da ide na master

svitvojimilioni added 2 commits 2024-02-08 01:08:40 +00:00

Dodao frontende za vise servisa ea6ba7464c

- frontendi su u frontends.py fajlu
- ima vise instanci da ne bi stalno slao na jednu, da nema centralizacije ;)

Dodao frontende za vise servisa 305fc1405a

- frontendi su u frontends.py fajlu
- ima vise instanci da ne bi stalno slao na jednu, da nema centralizacije ;)

texhno commented

2024-02-08 11:51:03 +00:00

Bas mi je cudno kako python radi sa tuple-ovima.
Referenciraju se kao liste, ali su immutable. Bar mi tako delovalo kad sam pokusao nesto da implementiram ovde kao tuple.
Cisto iz radoznalosti, sto si ovde stavio tuple-ove umesto listi?

Bas mi je cudno kako python radi sa tuple-ovima. Referenciraju se kao liste, ali su immutable. Bar mi tako delovalo kad sam pokusao nesto da implementiram ovde kao tuple. Cisto iz radoznalosti, sto si ovde stavio tuple-ove umesto listi?

svitvojimilioni commented

2024-02-08 22:54:16 +00:00

Cisto iz radoznalosti, sto si ovde stavio tuple-ove umesto listi?

U praksi je ovde svejedno da li je lista ili tuple. Tuple ima konstantano vreme pristupa elementu, bas zbog toga sto je immutable, dok za listu mora da ide redom. I jednostavno je dobra praksa koristiti tuple za stvari koje se nece menjati(cesto) i koje ce biti potrebni samo kao podaci, a listu za stvari koje se cesto dinamicki menjaju, tipa da dodajemo elmente onda je bolje lista jer samo doda na kraj, a tuple bi pravio novi.

A ovde je svejedno jer su jako male liste od manje od 10 elemenata i ta razlika ne verujem da se vidi, da ih ima milion onda bi bilo vidljivo.

E da, da dodam. Jer je tuple immutable tuple se moze i koristiti za kjuc u dictu, kljuc u dictu mora biti immutable. Tako de eto mozda i tu ima neka primena tuple-a, van ovog primera ovde.

> Cisto iz radoznalosti, sto si ovde stavio tuple-ove umesto listi? U praksi je ovde svejedno da li je lista ili tuple. Tuple ima konstantano vreme pristupa elementu, bas zbog toga sto je immutable, dok za listu mora da ide redom. I jednostavno je dobra praksa koristiti tuple za stvari koje se nece menjati(cesto) i koje ce biti potrebni samo kao podaci, a listu za stvari koje se cesto dinamicki menjaju, tipa da dodajemo elmente onda je bolje lista jer samo doda na kraj, a tuple bi pravio novi. A ovde je svejedno jer su jako male liste od manje od 10 elemenata i ta razlika ne verujem da se vidi, da ih ima milion onda bi bilo vidljivo. E da, da dodam. Jer je tuple immutable tuple se moze i koristiti za kjuc u dictu, kljuc u dictu mora biti immutable. Tako de eto mozda i tu ima neka primena tuple-a, van ovog primera ovde.

svitvojimilioni referenced this issue from a commit

2024-02-09 11:16:05 +00:00

Merge pull request 'Dodato jos frontova' (#2) from svitvojimilioni/chatbot:libre-rewrite into dev

svitvojimilioni merged commit b4b4d9797d into dev

2024-02-09 11:16:06 +00:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Decentrala/chatbot#2

Allow edits from maintainers