obdasystems / eddy

A graphical editor for the specification and visualization of Graphol ontologies
GNU General Public License v3.0
62 stars 7 forks source link

plugins: ontology-importer added #200

Closed MariaRosariaFraraccio closed 2 years ago

mnamici commented 2 years ago

Ho riordinato le modifiche.

Da un primo test veloce ho notato che se si esegue la seguente sequenza di azioni: 1) seleziona il pulsante di importazione 2) seleziona il nome del diagramma 3) inserisci un numero per lo spaziamento, poi cancella fino a che il campo è vuoto

si ottiene la seguente eccezione:

Traceback (most recent call last):
  File "/home/manuel/Workspace/eddy/eddy/plugins/ontology-importer/ontology_importer.py", line 5665, in onSpaceFieldChanged
    space = int(space.strip())
ValueError: invalid literal for int() with base 10: ''

P.S. per continuare sul tuo branch devi fare git pull --force prima.

mnamici commented 2 years ago

Il problema del segnale di salvataggio rimane, ma per il momento lasciamolo così.

Ad esempio che succede se l'applicazione crasha? Immagino l'importazione non rispecchia più lo stato del progetto.

MariaRosariaFraraccio commented 2 years ago

Il problema dello SpaceForm dovrebbe essere risolto.

Per quanto riguarda il segnale al momento sì, è così, quindi bisogna sicuramente trovare un'altra soluzione

mnamici commented 2 years ago

Ok il form è risolto ora comunque ci sono ancora un sacco di "rough edges" per considerarlo stabile, per esempio se cancello l'importazione questa viene fatta lo stesso, oppure se importo due volte la stessa ontologia l'importazione non viene effettuata ma viene lanciata un'eccezione per la violazione del vincolo UNIQUE (tenti comunque di fare l'inserimento).

Credi ci voglia molto per risolvere questi due bug in particolare? Sennò intanto rilasciamo così.

P.S. ho riorganizzato gli import e messo l'accesso alle classi Qt tramite il loro modulo per essere consistente col resto del progetto. P.S. 2: ho fatto force push per mantenere le modifiche ordinate e rendere la revisione più facile, per continuare da qui non fare merge ma reset:

$ git fetch origin
$ git reset --hard origin/ontology-importer

dovresti vedere due commit a partire dal tuo branch ql-restriction.

mnamici commented 2 years ago

Ok sono risolti ora, possiamo integrare.