troussil / convergents

GNU General Public License v3.0
1 stars 1 forks source link

Nouvelle Structure de l'algorithme #30

Closed homeostasie closed 11 years ago

homeostasie commented 11 years ago

Je te joins la nouvelle structure de l'algorithme dans cette pull request. Elle passe les tests sans problèmes. Je n'ai pas encore re-commenté le code comme il faut. Je fais ça maintenant.

J'ai laissé une structure (selon moi) un peu plus facilement lisible en :

if (){
  return (...)
}
else
{
...
}

On pourrai facilement la remplacer par la suivante pour gagner quelques lignes et un peu d'indentation :

if (){
  return (...)
}
...
homeostasie commented 11 years ago

J'ai déplacé la mise à jour des convergents vers le bas de la boucle pour qu'elle soit appelé quand pConv est à l'intérieur et que le lancer de rayon est convaincant et quand pConv est à l'intérieur sans pour autant correspondre au critère pour être un sommet de l'alpha-shape.

Du coup, il n'est plus nécessaire de tester si le lancer de rayon est effectif lorsque pConv est à l'intérieur car il passe tout de suite après dans le test de la boucle while.

Je vais quand même essayer de lancer une grande série de test pour voir si l'algo est vérifié.

troussil commented 11 years ago

J'ai fusionné avec le master. Deux remarques pour la prochaine fois:

homeostasie commented 11 years ago

J'ai également fusionné ton master avec le mien. Je prends note des remarques sur les commentaires.