Exercice ShipDict : les fichiers de référence contiennent des données répliquées

[aollier]

L'exercice sur les bateaux utilisant des classes (merger.py, compare.py, kml.py, shipdict.py) a comme données de référence des données (positions) répliquées :

$ diff --unified ALL_SHIPS.kml.ref ALL_SHIPS.kml

--- ALL_SHIPS.kml.ref	2017-12-15 17:07:57.000000000 +0100
+++ ALL_SHIPS.kml	2020-12-05 09:34:57.854160153 +0100
@@ -14187,9 +14187,6 @@
 -3.278488,49.83147,0
 -3.326835,49.81415,0
 -3.408277,49.78483,0
--3.408277,49.78483,0
--3.408277,49.78483,0
--3.408277,49.78483,0
 -3.467693,49.76384,0
 -3.509062,49.74928,0
 -3.570023,49.72786,0
@@ -14212,9 +14209,6 @@
 -4.537915,49.38431,0
 -4.592753,49.37083,0
 -4.63896,49.35313,0
--4.63896,49.35313,0
--4.63896,49.35313,0
--4.63896,49.35313,0
 -4.740983,49.31512,0
 -4.791693,49.2964,0
 -4.8213,49.28574,0
@@ -14238,9 +14232,6 @@
 -5.784409,48.24205,0
 -5.742897,48.16587,0
 -5.718343,48.12303,0
--5.718343,48.12303,0
--5.718343,48.12303,0
--5.718343,48.12303,0
 -5.642142,47.99115,0
 -5.614835,47.943,0
 -5.58918,47.90204,0
@@ -33170,9 +33161,6 @@
 -5.379983,48.89437,0
 -5.204307,48.93785,0
 -5.149205,48.95719,0
--5.149205,48.95719,0
--5.149205,48.95719,0
--5.149205,48.95719,0
 -4.888073,49.05023,0
 -4.82133,49.0775,0
 -4.771345,49.09834,0

$ diff --unified ALL_SHIPS.txt.ref ALL_SHIPS.txt

--- ALL_SHIPS.txt.ref	2017-12-15 17:07:53.000000000 +0100
+++ ALL_SHIPS.txt	2020-12-05 09:34:57.578161081 +0100
@@ -156,7 +156,7 @@
 ENEZ EUSSA 3 (111 positions)
 ENEZ VIAN (19 positions)
 ENFANT DU VOYAGE (9 positions)
-ENFORCER (84 positions)
+ENFORCER (75 positions)
 ENTRE TERRE&MER (24 positions)
 ESBJERG (17 positions)
 ESCAPADE (13 positions)
@@ -459,7 +459,7 @@
 SAN ANTONIO (22 positions)
 SANAGA 2 (9 positions)
 SANTA ANNA (3 positions)
-SANTA CRUZ (31 positions)
+SANTA CRUZ (28 positions)
 SAPPHIRE ACE (25 positions)
 SARAH P (111 positions)
 SARK BELLE (110 positions)

$ diff --unified ALL_SHIPS-v.txt.ref ALL_SHIPS-v.txt

--- ALL_SHIPS-v.txt.ref	2017-12-15 17:07:57.000000000 +0100
+++ ALL_SHIPS-v.txt	2020-12-05 09:38:49.060471579 +0100
@@ -7890,7 +7890,7 @@
 <47.48'52'' N 03.55'36'' W @ 2013-10-01T12:08:00>
 <47.49'31'' N 03.55'38'' W @ 2013-10-01T12:13:00>
 <47.51'26'' N 03.55'39'' W @ 2013-10-01T12:26:00>
-ENFORCER (84 positions)
+ENFORCER (75 positions)
 <49.59'44'' N 02.46'13'' W @ 2013-10-01T03:38:00>
 <49.59'06'' N 02.49'19'' W @ 2013-10-01T03:48:00>
 <49.58'33'' N 02.52'13'' W @ 2013-10-01T03:58:00>
@@ -7903,9 +7903,6 @@
 <49.49'53'' N 03.16'42'' W @ 2013-10-01T05:28:00>
 <49.48'50'' N 03.19'36'' W @ 2013-10-01T05:39:00>
 <49.47'05'' N 03.24'29'' W @ 2013-10-01T05:56:00>
-<49.47'05'' N 03.24'29'' W @ 2013-10-01T05:56:00>
-<49.47'05'' N 03.24'29'' W @ 2013-10-01T05:56:00>
-<49.47'05'' N 03.24'29'' W @ 2013-10-01T05:56:00>
 <49.45'49'' N 03.28'03'' W @ 2013-10-01T06:08:00>
 <49.44'57'' N 03.30'32'' W @ 2013-10-01T06:17:00>
 <49.43'40'' N 03.34'12'' W @ 2013-10-01T06:29:00>
@@ -7928,9 +7925,6 @@
 <49.23'03'' N 04.32'16'' W @ 2013-10-01T09:37:00>
 <49.22'14'' N 04.35'33'' W @ 2013-10-01T09:47:00>
 <49.21'11'' N 04.38'20'' W @ 2013-10-01T09:57:00>
-<49.21'11'' N 04.38'20'' W @ 2013-10-01T09:57:00>
-<49.21'11'' N 04.38'20'' W @ 2013-10-01T09:57:00>
-<49.21'11'' N 04.38'20'' W @ 2013-10-01T09:57:00>
 <49.18'54'' N 04.44'27'' W @ 2013-10-01T10:18:00>
 <49.17'47'' N 04.47'30'' W @ 2013-10-01T10:28:00>
 <49.17'08'' N 04.49'16'' W @ 2013-10-01T10:34:00>
@@ -7954,9 +7948,6 @@
 <48.14'31'' N 05.47'03'' W @ 2013-10-01T17:22:00>
 <48.09'57'' N 05.44'34'' W @ 2013-10-01T17:43:00>
 <48.07'22'' N 05.43'06'' W @ 2013-10-01T17:55:00>
-<48.07'22'' N 05.43'06'' W @ 2013-10-01T17:55:00>
-<48.07'22'' N 05.43'06'' W @ 2013-10-01T17:55:00>
-<48.07'22'' N 05.43'06'' W @ 2013-10-01T17:55:00>
 <47.59'28'' N 05.38'31'' W @ 2013-10-01T18:33:00>
 <47.56'34'' N 05.36'53'' W @ 2013-10-01T18:47:00>
 <47.54'07'' N 05.35'21'' W @ 2013-10-01T18:58:00>
@@ -23544,7 +23535,7 @@
 <48.48'40'' N 05.54'06'' W @ 2013-09-30T21:57:00>
 <48.46'52'' N 05.55'44'' W @ 2013-09-30T22:07:00>
 <48.43'18'' N 05.58'57'' W @ 2013-09-30T22:29:00>
-SANTA CRUZ (31 positions)
+SANTA CRUZ (28 positions)
 <48.27'52'' N 05.55'45'' W @ 2013-10-01T15:23:00>
 <48.31'44'' N 05.51'53'' W @ 2013-10-01T15:39:00>
 <48.33'41'' N 05.49'58'' W @ 2013-10-01T15:47:00>
@@ -23553,9 +23544,6 @@
 <48.53'39'' N 05.22'47'' W @ 2013-10-01T17:22:00>
 <48.56'16'' N 05.12'15'' W @ 2013-10-01T17:48:00>
 <48.57'25'' N 05.08'57'' W @ 2013-10-01T17:56:00>
-<48.57'25'' N 05.08'57'' W @ 2013-10-01T17:56:00>
-<48.57'25'' N 05.08'57'' W @ 2013-10-01T17:56:00>
-<48.57'25'' N 05.08'57'' W @ 2013-10-01T17:56:00>
 <49.03'00'' N 04.53'17'' W @ 2013-10-01T18:38:00>
 <49.04'39'' N 04.49'16'' W @ 2013-10-01T18:49:00>
 <49.05'54'' N 04.46'16'' W @ 2013-10-01T18:57:00>

[parmentelat]

les données de référence sont obtenues en exécutant le code (la partie 'template' + le corrigé de la partie à écrire par les élèves)

est-ce que tu sais dans laquelle de ces deux parties se trouve le bug qui provoque la duplication ?

si c'est dans le template, ça n'est pas trop grave car du coup les élèves arrivent facilement au même résultat que les fichiers de référence

ce qui n'empêche pas de corriger bien entendu

[aollier]

C'est dans la partie à écrire par les élèves (shipdict.py).

Je sais la corriger puisque c'est avec mon code que j'ai trouvé ce problème. Je dois fournir shipdict.py corrigé, les fichiers de référence au format tar, tar.gz et zip. Est-ce que je dois modifier quelque chose d'autre encore ?

[parmentelat]

honnêtement je ne me souviens plus de tous les détails, je n'ai pas l'impression qu'il y ait autre chose

il faudra re-publier les corrigés après coup mais c'est une autre histoire