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

[ELEC1350] exam Électromagnétisme janvier 2020 #857

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

Conversation

MiguelDLC
Copy link
Contributor

Tout est dans le titre

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.

Quelques petites modifs :

  • pour les unités, c'est plus propre d'utiliser siunitx ;
  • il manquait un \nosolution à la fin ;
  • renommage de fichiers pour éviter des bugs potentielles.

Techniquement il y a aussi des $...$ manquants à plusieurs endroits pour encadrer les noms de variables, mais bon, ça passe encore.
Si le fichier vient d'un Overleaf, il faudrait mettre un message/note sur l'Overleaf pour demander de ne plus le modifier, et de rediriger les modifs vers le repo GitHub.

@Jimvy
Copy link
Collaborator

Jimvy commented Aug 9, 2020

Ca peut sembler être beaucoup de changements, mais c'est juste les \SI{}{} qui sont nombreux ^^

MiguelDLC and others added 13 commits August 9, 2020 14:22
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
…xam-2020-Janvier-Majeure.tex

Co-authored-by: Jean-Martin Vlaeminck <[email protected]>
@MiguelDLC
Copy link
Contributor Author

MiguelDLC commented Aug 9, 2020

Corrections effectuées mais je n'ai pas su utiliser \imagj, erreur undefined control sequence. Tu sais où la commande \imagj est censée être définie? Du coup à la place j'ai mis \sisetup{output-complex-root=\ensuremath{\mathrm{j}}} et ça affiche le j droit correctement.

L'exam vient de la boite mail (et l'auteur l'a fait avec le template overleaf) donc je ne pense pas que ce soit nécessaire d'aller modifier l'overleaf source puisque personne d'autre que lui n'y a accès.

@Jimvy
Copy link
Collaborator

Jimvy commented Aug 9, 2020

Normalement \imagj vient de eplcommon.sty, donc c'est assez surprenant. Ou alors j'ai fait une typo. Sinon le truc avec \sisetup est impressionnant 😮

Ok pour l'overleaf ;)

@MiguelDLC
Copy link
Contributor Author

Je crois que le problème venait du fait que dans \SI{valeur}{unité} la valeur doit être numérique et donc il ne devait pas comprendre un \mathrm{j}

@MiguelDLC MiguelDLC requested a review from Jimvy August 9, 2020 20:06
@Jimvy Jimvy changed the title exam Électromagnétisme janvier 2020 [ELEC1350] exam Électromagnétisme janvier 2020 Feb 8, 2021
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