Skip to content

fusion des reseaux : gerer les conflits #127

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

Closed
qchasserieau opened this issue Oct 13, 2022 · 4 comments · Fixed by #483 or #499
Closed

fusion des reseaux : gerer les conflits #127

qchasserieau opened this issue Oct 13, 2022 · 4 comments · Fixed by #483 or #499
Assignees
Labels

Comments

@qchasserieau
Copy link
Contributor

on pourrait avoir un bouton d'upload toujours accessible qui permet d'ajouter un zip / des geojsons et les fusionner à ceux qu'on a déjà.

1 vérifier s'il y a un conflit sur les indexs des liens et des nœuds
2 ouvrir une boite de dialogue

  • qui dit s'il y a des conflits et
  • proposer (imposer le cas échéant) de choisir des suffixes pour les indexs des deux tables
  • proposer l'ajout d'une colonne pour les lignes (et les nœuds ?) avec des valeurs
    La colonne pourra être utilisée pour des tris dans l'affichage ensuite

Les liens et les nœuds sont concaténés en conservant toutes les colonnes des deux tables.

@qchasserieau qchasserieau added this to the v2.6 milestone Oct 13, 2022
@siforf564 siforf564 modified the milestone: v2.6 Oct 17, 2022
@sboivinsystra sboivinsystra changed the title Possibilité d'uploader un réseau et de le fusionner au réseau déjà chargé fusion des reseaux : gerer les conflits Oct 25, 2022
@sboivinsystra sboivinsystra modified the milestones: v2.6, v2.7 Nov 4, 2022
@sboivinsystra sboivinsystra modified the milestones: v2.7, 3.0 Nov 21, 2022
@sboivinsystra sboivinsystra removed this from the 3.0 milestone Feb 7, 2023
@sboivinsystra
Copy link
Contributor

dans import.vue. wrappé autour de la function loadNetwork() me semble l'endroit ideal.

tout fichiers loadé dans l'app passe par cette fonction. on peut regarder si les indices sont dupliqué des reseaux.
if not duplicated : go on and import
if duplicated: give choices (create new index from 0. create new random indexes for new files, etc).

Then change the files. then recheck.

@qchasserieau
Copy link
Contributor Author

lancer action quand un réseau link + nodes est chargé alors qu'un réseau est déjà présent dans l'app

action :

  1. test index noeuds -> no conflict ou conflict
  2. si conflit : tester correspondence avec les hash-> perfect_match ou no_match
  3. si no_match

Dans l'import de PT network :
Si aucun index dupliqué : GOOD

Lignes :
key : default no
if yes ask for two keys for app and uploaded network
add a column with the keys in the links

Nodes :
perfect match only
Duplicate distant dones

@sboivinsystra sboivinsystra linked a pull request Feb 15, 2024 that will close this issue
@sboivinsystra
Copy link
Contributor

nodes:

  1. drop perfect matches
  2. create new index for non perfect match (but same index)

links:

  1. drop duplicated trip_id
  2. drop perfect matches
  3. create new index for non perfect match (but same index).

not done:
nodes:

  • perfect match if dist<1m : simple method really slow to implement. is it necessary?

links:
perfect match if idx,a,b are the same. : not done as they could have different trip_id and we dont want to drop those. ex: montreal orange line have a trip with 1 more link. we dont want to drop every other links .

@sboivinsystra
Copy link
Contributor

perfect match for dist <10 (1meters was really nothing)

@sboivinsystra sboivinsystra linked a pull request Mar 21, 2024 that will close this issue
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants