openspyrit / spas

Single-Pixel Acquisition Software
https://spas.readthedocs.io/en/latest/
GNU Lesser General Public License v3.0
8 stars 2 forks source link

Unable to use read_metadata()? #8

Closed nducros closed 1 year ago

nducros commented 1 year ago

Une idée ?

_, acquisition_parameters, _, _ = read_metadata(meta_path)

Gives

Traceback (most recent call last):

  Input In [27] in <cell line: 1>
    _, acquisition_parameters, _, _ = read_metadata(meta_path)

  File c:\users\ducros\openspyrit\spas\spas\metadata.py:823 in read_metadata
    saved_acquisition_params.undo_readable_pattern_order()

  File c:\users\ducros\openspyrit\spas\spas\metadata.py:244 in undo_readable_pattern_order
    self.patterns_wp = self.patterns_wp.strip('[').strip(']').split(', ')

AttributeError: 'NoneType' object has no attribute 'strip'
Mahieu-Williame commented 1 year ago

oui, j'ai ajouté cette fonction pour insérer les motif blanc (wp pour white pattern). Tu dois utiliser seulement le prog de reconstruction sans reconstruction et non le prog d'acquisition avec la reconstruction, tu confirmes ? tu as reconstruit avec quelles données, v1.2 - one arm ou v1.3 two arms ? il faut que je le mette à jour. je ferai ça demain.

nducros commented 1 year ago

Je reconstruis les données du papier OE (donc v1.2 ?). Dans un premier temps, j'avais juste besoin de read_metadata(), pas des fonctions de reconstruction. Donc je dirais que je confirme, si j'ai bien compris ta question.

Mahieu-Williame commented 1 year ago

voilà, j'ai mis une rustine, en fait je n'enregistre pas la variable patterns_wp, c'est pour ça qu'il n'a pas la méthode "strip". j'ai mis la fonction "Try" devant, donc ça passe maintenant.

nducros commented 1 year ago

Impec, c'est OK chez moi !