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

[SR] Week 1 translations #720

Open
wants to merge 50 commits into
base: master
Choose a base branch
from
Open

Conversation

EmaPajic
Copy link
Contributor

No description provided.

@EmaPajic
Copy link
Contributor Author

It's really good. I've fixed the language specification lang: sr (otherwise the website builder won't work).

Can you please reintroduce the English text back, as comment? See translation guideline wiki. Then I'll merge.

Great! :)
Done adding.

Atcold
Atcold previously approved these changes Dec 15, 2020
Copy link
Owner

@Atcold Atcold left a comment

Choose a reason for hiding this comment

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

This looks good to me. All technical stuff has been taken care of.
@PetarV-, please review the content, provide feedback, and merge when good.

docs/sr/README-SR.md Outdated Show resolved Hide resolved
docs/sr/week01/01-2.md Outdated Show resolved Hide resolved
Copy link

@dsvilarkovic dsvilarkovic left a comment

Choose a reason for hiding this comment

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

We should synchronize on do we use transcription or not - there is McCullough, LeCun and then there is Fukušima and Križevski (not Fukushima and Krizhevsky). Same thing for model names. I suggest we keep them as original names, since I guess someone might want to look them up on Google Scholar or something. Even better, add \emph{} on names, so in case we later decide on cyrillic alphabet, we can easily avoid their names being converted to cyrillic.

EmaPajic and others added 6 commits December 17, 2020 13:01
Co-authored-by: Dusan Svilarkovic <dusansvilarkovic@gmail.com>
Co-authored-by: Dusan Svilarkovic <dusansvilarkovic@gmail.com>
Co-authored-by: Dusan Svilarkovic <dusansvilarkovic@gmail.com>
Co-authored-by: Dusan Svilarkovic <dusansvilarkovic@gmail.com>
Co-authored-by: Dusan Svilarkovic <dusansvilarkovic@gmail.com>
Co-authored-by: Dusan Svilarkovic <dusansvilarkovic@gmail.com>
@EmaPajic
Copy link
Contributor Author

We should synchronize on do we use transcription or not - there is McCullough, LeCun and then there is Fukušima and Križevski (not Fukushima and Krizhevsky). Same thing for model names. I suggest we keep them as original names, since I guess someone might want to look them up on Google Scholar or something. Even better, add \emph{} on names, so in case we later decide on cyrillic alphabet, we can easily avoid their names being converted to cyrillic.

I agree, maybe we should check what others think and then update?

Copy link
Contributor

@PetarV- PetarV- left a comment

Choose a reason for hiding this comment

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

I have made a few comments (still unfinished) from a previous pass of the work. I'm leaving them here and will start a new review to pick up :)

docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
docs/sr/week01/01-1.md Outdated Show resolved Hide resolved
@Atcold Atcold dismissed their stale review December 18, 2020 23:34

This was a technical review and not based on the content.


<!--Fukushima (1982) built a neural net (NN) that worked the same way as the brain, based on two concepts. First, neurons are replicated across the visual field. Second, there are complex cells that pool the information from simple cells (orientation-selective units). As a result, the shift of the picture will change the activation of simple cells, but will not influence the integrated activation of the complex cell (convolutional pooling).
-->
Fukušima je 1982. godine napravio neuronsku mrežu koja je radila na isti način kao i mozak, bazirano na 2 koncepta. Prvo, neuroni su postavljeni po celom vidnom polju. Drugo, postoje kompleksne ćelije koje agregiraju informacije iz jednostavnih ćelija (jedinica koje reaguju na orijentaciju). Kao rezultat, pomeraj slike će uticati na aktivacije jednostavnih ćelija, ali neće uticati na agregiranu aktivaciju komplikovane ćelije (agregiranje konvolucijom).

Choose a reason for hiding this comment

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

bazirano -> i bila je bazirana


<!--LeCun (1990) used backprop to train a CNN to recognize handwritten digits. There is a demo from 1992 where the algorithm recognizes the digits of any style. Doing character/pattern recognition using a model that is trained end-to-end was new at that time. Previously, people had used feature extractors with a supervised model on top.
-->
LeCun je 1990. godine iskoristio propagaciju unazad da obuči konvolucionu neuronsku mrežu da prepozna rukom pisane cifre. Postoji video iz 1992. gde algoritam prepoznaje cifre napisane različitim stilovima. Prepoznavanje karaktera / oblika koristeći model koji rešava problem od početka do kraja je bilo novo u to vreme. Ranije je bilo neophodno izvlačenje obeležja pre modela nadgledanog učenja.

Choose a reason for hiding this comment

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

karaktera -> slova
je bilo novo u to vreme -> predstavljala je inovaciju u to vreme
izvlačenje obeležja: not sure if this is immediately understandable as feature extractor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants