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

[ELEC2870] big update summary machine learning elec #867

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

Conversation

gauone
Copy link
Contributor

@gauone gauone commented Jan 15, 2021

No description provided.

@Jimvy Jimvy changed the title big update summary machine learning elec [ELEC2870] big update summary machine learning elec Feb 8, 2021
Copy link
Collaborator

@Jimvy Jimvy left a comment

Choose a reason for hiding this comment

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

Bon, trois mois plus tard je repasse sur les PR en attente...

Je vois quelques problèmes de type "nitpick" (\dots au lieu de ..., ``...'', des blancs en fin de ligne, des $$..$$ au lieu de \[..\], des doubles backslashs, des incohérences espaces/tabulations au niveau de l'indentation).
Est-ce que ça vaut la peine de les corriger, ou est-ce qu'on laisse ça à une correction ultérieure ?

src/q7/ml-ELEC2870/summary/ml-ELEC2870-summary.tex Outdated Show resolved Hide resolved
Comment on lines +1281 to +1285
\begin{enumerate}
\setcounter{enumi}{1943-1}
\item McCulloch \& Pitts : artificial neuron as descriptive model, no training nor learning, fixed weights
\setcounter{enumi}{1949-1}
\item Hebb's law on neuronal activity
Copy link
Collaborator

Choose a reason for hiding this comment

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

Je suis pas méga fan de cette technique pour réaliser une timeline en LaTeX, mais bon, ça passe.

src/q7/ml-ELEC2870/summary/ml-ELEC2870-summary.tex Outdated Show resolved Hide resolved
src/q7/ml-ELEC2870/summary/ml-ELEC2870-summary.tex Outdated Show resolved Hide resolved
src/q7/ml-ELEC2870/summary/ml-ELEC2870-summary.tex Outdated Show resolved Hide resolved
src/q7/ml-ELEC2870/summary/ml-ELEC2870-summary.tex Outdated Show resolved Hide resolved
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.

2 participants