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.

Sign up for GitHub

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

Jump to bottom

[2022] Simulacije muve #9

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from
Draft

[2022] Simulacije muve #9

wants to merge 8 commits into from

Conversation

basicskill
Copy link

No description provided.

@basicskill basicskill requested a review from a team as a code owner October 15, 2022 11:10
basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md Outdated Show resolved Hide resolved
content/2022/simulacija-muve.md Outdated Show resolved Hide resolved
content/2022/simulacija-muve.md Outdated Show resolved Hide resolved
content/2022/simulacija-muve.md Show resolved Hide resolved
content/2022/simulacija-muve.md Outdated Show resolved Hide resolved
content/2022/simulacija-muve.md Outdated Show resolved Hide resolved
content/2022/simulacija-muve.md
Implementirana je neuralna mreža i genetski algoratam tako da se jednostavnim interfejsom može primeniti za razne ML probleme. Takođe je implementiran i kod koji koristi ove biblioteke za treniranje muve da leti. Više o interfejsu između muve i genetskog algoritma malo kasnije.

Kod je podeljen u dva dela
- network.py - ovde je implementirana mreža
Copy link
Author

Choose a reason for hiding this comment

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

  • Konkretne implementacione stvari idu u segment eksperimenti/rezultati, ne u metod

Copy link
Member

Choose a reason for hiding this comment

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

Možeš malo da pojasniš ovo @basicskill
Nije mi najjasnije šta si ovime hteo da kažeš.

Copy link
Author

Choose a reason for hiding this comment

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

Ispravka, nema potrebe za naglašavanjem u koje fajlove je kod podeljen.

content/2022/simulacija-muve.md Outdated Show resolved Hide resolved
basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md Outdated
Napravljene su dve verzije mreže, jedna vektorizovana i druga ne vektorizovana. Ovde ću opisati implementaciju vektorizovane mreže i algoritama koji su prilagođeni takvoj implementaciji.

#### Opis interfejsa za treniranje
Algorigam funkcioniše tako što traži da korisnik inicijalizuje populaciju tipa Population nad kojom će posle pozvati funkciju run. Pri pravljenju populacije treba da prosledi broj jedinki u populaciji i arhitekturu mreže. Pri pozivu funkcije run nad populacijom treba da se proslede fitnes funkcija i ostali parametri kao što su šansa za mutaciju gena, nazivi fajlova u kojima će se čuvati geni najboljih mreža, način na koji će se čuvati itd.
Copy link
Author

Choose a reason for hiding this comment

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

  • Ponovo, hajde da pišemo o metodu a ne o konkretnoj implementaciji
  • Izbegavajte da rečenice počinjete sa "Pri"

basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md

Označimo sa $X_{j}^{i}$ j-ti ulazni podatak (input feature) od i-tog seta ulaznih podataka. Onda format matrice ulaznih podataka za funkciju Activate izgleda ovako:

| Set 1 | Set 2 | ... | Set m |
Copy link
Author

Choose a reason for hiding this comment

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

Šta je poenta ove tabele?

basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md

Ovde imamo n ulaznih podataka za svaki set. Imamo m setova. Ovo znači da naša mreža u prvom sloju ima n neurona i da će u ovom slučaju obrađivati m setova ulaznih podataka paralelno. Vrednost m ne mora da bude ista za svaki poziv Activate funkcije dok n mora jer je već definisana arhitekturom mreže.

**bitno je voditi računa o orijentacijama vektora u implementaciji!**
Copy link
Author

Choose a reason for hiding this comment

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

Izveštaj nije uputstvo za korišćenje koda, već objašnjenje metodologije i pregled rezultata. Hajde da ne pišemo ove boldovane komentare

basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md

| | | | |
| ----- | ----- | ----- | ----- |
| $W[k]_{00}$ | $W[k]_{01}$ | ... | $W[k]_{0m}$ |
Copy link
Author

Choose a reason for hiding this comment

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

Čemu služi ova tabela?

basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md
| $b[k]_{0}$ |
| $b[k]_{1}$ |
| ... |
| $b[k]_{n}$ |
Copy link
Author

Choose a reason for hiding this comment

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

Čemu služi ova tabela?

basicskill
basicskill commented Oct 15, 2022
View reviewed changes
content/2022/simulacija-muve.md
A[k] = tanh(Z[k])
$$

**Imati na umu da je $\times$ operacija množenja matrica**
Copy link
Author

Choose a reason for hiding this comment

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

Znamo, nema potrebe za ovim boldovanim tekstom

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 15, 2022 •
edited
Loading

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 720598a
Status: ✅  Deploy successful!
Preview URL: https://2c6f5680.pfe-izvestaji.pages.dev
Branch Preview URL: https://2022-simulacija-muve.pfe-izvestaji.pages.dev

View logs

@Aleksandar204 Aleksandar204 requested a review from a team October 15, 2022 16:59
@djokjulapfe djokjulapfe changed the title Izvestaj simulacije muve [2022] Simulacije muve Oct 16, 2022
basicskill
basicskill commented Oct 16, 2022
View reviewed changes
content/2022/simulacija-muve.md Outdated
**Na svim graficima (osim ako drugačije nije naznačeno) će se nalaziti zavisnost visine bube od vremena proteklog od početka simulacije.**

![slika](/images/2022/simulacija-muve/Figure_3.png)
Copy link
Author

Choose a reason for hiding this comment

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

Dodajte naslov na grafik i obeležite ose na srpskom jeziku

basicskill
basicskill commented Oct 16, 2022
View reviewed changes
content/2022/simulacija-muve.md
Kada smo pokrenuli kod, program je često pucao i na grafiku bube se videlo kako se u nekim delovima veoma brzo menja visina bube, kao što se može videti na sledećem grafiku:

**Na svim graficima (osim ako drugačije nije naznačeno) će se nalaziti zavisnost visine bube od vremena proteklog od početka simulacije.**
Copy link
Author

Choose a reason for hiding this comment

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

Ne koristiti boldovane rečenice

@filiparag filiparag added the 2022 Projekti starih polaznika rađeni 2022. godine label Oct 16, 2022
@filiparag filiparag marked this pull request as draft October 16, 2022 20:25
basicskill
basicskill commented Nov 20, 2022
View reviewed changes
content/2022/simulacija-muve.md
### Apstrakt na engleskom


### Uvod
Copy link
Author

Choose a reason for hiding this comment

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

  • Izbegavajte korišćenje skraćenica koje niste prethodno napisali šta označavaju (npr. ML ne piše na šta se odnosi)
  • Dodajte "mašinsko učenje (engl. machine learning, ML)" i u nastavku je okej da koristite ML kao skraćenicu
  • Nerualnim mrežama i genetskim algoritmom ne treninramo simulaciju već nekog agenta unutar same simulacije
  • Koristite pasivan govor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@natasa-jovanovic natasa-jovanovic natasa-jovanovic left review comments

@djokjulapfe djokjulapfe djokjulapfe left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
2022 Projekti starih polaznika rađeni 2022. godine
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@basicskill @djokjulapfe @natasa-jovanovic @filiparag @Aleksandar204