Gestione classe per la raccolta di metadati

[ZompaSenior]

Creare una classe base per la raccolta dei metadati, partendo dal modulo placeholder:
https://github.com/ConoscereLinux/DeskApp/blob/b32fcf51f7a201b116b4074a81e7d5f7815579c5/src/meta/metadata_manager.py

[ZompaSenior]

@metster, aggiungiamo una nuova funzione, per predisporci ad essere integrato con le altri parti appena saranno pronte.

Dovresti aggiungere alla funzione, un nuovo metodo denominato scanfile, che prede come parametri origin_file, elaborated_file, move (booleano per decidere se spostare o no il file). I metadati raccolti andrebbero restituiti in un dizionario con il return della funzione.

Per ogni esigenza, commenta qui sotto!

[metster]

OK, provo ad aggiungere metodo con specifiche indicate poi faccio pull request. Volevo anche provare a scaricare tutto per vedere se funzionava, come da te richiesto ma, nonostante veda file correttamente aggiornati su mio github, facendo upgrade project da pycharm, non mi aggiorna nulla! Tipo cartella DIG e INDEX con relativi .py all'interno, non me li salva in locale. qualcosa avrò sbagliato... ora intanto faccio modifica poi vedo di risolvere problema aggiornamento!

[ZompaSenior]

Ciao, dopo aver aggiornato su GitHub, devi fare pull dal tuo PC, in modo da scaricare l'aggiornamento. Probabilmente c'è un menu anche da PyCharm.

[metster]

SI trovato! in basso a dx si deve indicare il branch e, nel mio caso, indicata feature/metadati e non develop quindi aggionavo solo il mio pezzettino che era già aggiornato. ora ho risolto, scaricato tutto il programma con tue modifiche e funziona tutto. :-) ora procedo con modifica del nuovo metodo e pull request

[metster]

Ciao, chiedevo un chiarimento.... il metodo scan non fa' già quello che chiedi? L'unica modifica che sarebbe da fare è quella di togliere la parte che salva tutto su file txt e inserire la creazione di un dizionario vuoto che poi viene popolato da coppie chiave:valore dei metadati estrapolati dalla foto. la parte del move invece non l'ho capita... perchè dovrei voler non spostare il file? se non li togli dalla cartella di origine, tutte le volte che fai partire applicazione, torna a processare sempre la stessa foto, no? attendo tue... :-)

[ZompaSenior]

Ciao,
allora chiarisco meglio:

1. La cosa principale che dovrebbe cambiare è che scanfile (che poi l'ho chiamato male, sarebbe meglio scan_file) analizza 1 solo file e poi si ferma, e il file gli viene detto da fuori, non si occupa lui di scorrere i file della cartella.
2. La seconda cosa, è che il dizionario di metadati deve essere restituito dalla funzione tramite return e non salvato.

Ora ti spiego i ragionamenti dietro le scelte:

1. in questa classe digger_manager.py, verrà implementato un processo che si occupa di scansionare le cartelle, chiedere per ogni file alla tua classe di raccogliere i metadati, che poi passerà a indexer_manager.py perché si occupi di salvarli in un indice. In metadata_manager.py verranno delegate solo le attività di raccolta dei metadati, in tutte le loro forme (AI, Machine Learnig, OCR, ecc ...)
2. Il fatto di rendere opzionale lo spostamento, è perchè non tutti vogliono toccare l'origine dei dati, e per non analizzare una seconda volta lo stesso file, ci appoggeremo ad un sistema basato sull'indice e data/ora di modifica o l'hash del file, in modo da poter analizzare il file se cambia.

Spero di averti chiarito un po' le cose, ma se hai altri dubbi, scrivimi pure.

P.S.
Secondo me nella pull-request, hai spostato per sbaglio la cartella doc sotto a src.