search

otrecoding / OTrecod

R package for optimal transportation to recode variables
https://otrecoding.github.io/OTrecod
GNU General Public License v3.0
2 stars 1 forks source link

Update 0406 #2

Closed pnavaro closed 4 years ago

pnavaro commented 4 years ago

Voici les changements que m'a envoyé Valérie. C'est Gregory qui doit regarder que tout va bien. N'hésitez pas à poser des questions. On peut prendre le temps qu'il faut.

pnavaro commented 4 years ago

Pour créer cette proposition de changement, j'ai mis les modifications envoyés par @vagares dans une branche dev et maintenant je demande à ce que soit validé par @gguernec

pnavaro commented 4 years ago

Dans les onglets au dessus vous avez les différents commits et les fichiers modifiés. Il y a une erreur dans la fonction compare_lists. Je propose pour s'entrainer de faire ensemble une autre "pull request" une fois que l'erreur sera corrigée. En attendant si tout est OK pour Gregory il te suffit de cliquer sur le bouton vert "Merge pull request". Si des choses sont à corriger il faudra faire d'autres commits dans la branche "dev". La pull request sera mise à jour automatiquement.

vagares commented 4 years ago

Bonjour

Il faut que je reclone le dossier apres ma bétise. Quel est le lien déjà ?

Oui pour se faire une réunion. Mardi après midi ?

pnavaro commented 4 years ago

OK pour moi le 9 juin à 14:00

vagares commented 4 years ago

Gregory ne verra pas ses mails avant lundi après midi par contre !!

pnavaro commented 4 years ago

Gregory ne verra pas ses mails avant lundi après midi par contre !!

Je peux lui envoyer un texto

pnavaro commented 4 years ago

Tu peux recloner le dossier et changer de branche pour la branche dev

vagares commented 4 years ago

Peux tu me refiler le lien stp ?

vagares commented 4 years ago

il n'a pas de téléphone et d'internet chez lui.

J'ai son fixe. Il m'a dit que tu peux l'appeler, il pourra répondre à tes questions plus failement

pnavaro commented 4 years ago

Je conseille d’installer l’application GitHub sur ton téléphone c’est top!

pnavaro commented 4 years ago

Peux tu me refiler le lien stp ?

https://github.com/otrecoding/OTrecod

vagares commented 4 years ago

Merci !!!!

pnavaro commented 4 years ago

Dorénavant toutes modifications de la branche master devra être validée par les deux autres membres du projet 👮🏻👮🏻👩‍💻🧑‍💻

pnavaro commented 4 years ago

J'ai réglé le problème de data et supprimé les deux dépendances nnet et ordinal. Cependant j'ai toujours l'erreur avec compare_lists, j'ai ouvert l'issue #3 pour en discuter et résoudre le problème.

codecov-commenter commented 4 years ago

Codecov Report

Merging #2 into master will increase coverage by 0.14%. The diff coverage is 0.14%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #2      +/-   ##
=========================================
+ Coverage    0.00%   0.14%   +0.14%     
=========================================
  Files          19      19              
  Lines        1231    1423     +192     
=========================================
+ Hits            0       2       +2     
- Misses       1231    1421     +190
Impacted Files Coverage Δ
R/OT_outcome.r 0.00% <0.00%> (ø)
R/avg_dist_closest.r 0.00% <0.00%> (ø)
R/count_pos.r 0.00% <0.00%> (ø)
R/error_group.r 0.00% <0.00%> (ø)
R/find_coord.r 0.00% <0.00%> (ø)
R/ham.r 0.00% <0.00%> (ø)
R/imput_cov.r 0.00% <0.00%> (ø)
R/indiv_grp_closest.r 0.00% <0.00%> (ø)
R/indiv_grp_optimal.r 0.00% <0.00%> (ø)
R/merge_dbs.r 0.00% <0.00%> (ø)
... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update ff083b6...8d716a6. Read the comment docs.