Liste des bugs et feature requests pour OImaging

Cette liste fait doublon avec les tickets track / issue github et ne servira que temporairement le temps de définir les priorités de developpement 2021

In progress

Voir board https://github.com/orgs/JMMC-OpenDev/projects/3

Items en cours:

  • champs ServiceResult => keyword Outputparams (alagorithm, comment, rating, timestamps ...). Première proposition sur branche perso buthanoid.
Items intégrés:
  • PR #3: table dynamique (output, input keywords) + table editor. la gestion des colonnes est globale et non spécialisée à la liste courante de serviceResults. Cela permet de garder le code des colonnes assez simple.
  • PR#4: dev mode :chargement auto de fichiers de résultats (prépare la sauvegarde de session)
  • PR#5: supprimer liste results à gauche + problèmes d'affichage avec panneaux + problèmes d'édition GUI Netbeans
  • PR#6: supprimer le dialogue demandant l'autorisation de renommer une image dont le nom fait doublon. le renommage est à présent automatiquement fait sans dialog.
  • ScrollBar du panneau gauche (des choix de l'algo, paramètres etc) très lente. Réduire la fenêtre en hauteur pour s'en apercevoir, comparer au tableau de données dans Oifits->data par exemple.
  • Beta release 0.5.4 beta 4: http://jmmc.fr/~betaswmgr/OImaging/
  • tickets #946 #1065 fixed

Bugs

GUI OImaging

  • Bugs lié à la règle :
    • clic doit Copy ou Save as > on perd la règle
    • parfois la regle diparait en passant sur plusieurs résultats
  • les keywords suivants sont vides sauf pour WISARD #1052
    LAST_IMG Identifier of the final image
    NITER Total iterations done in the current program run
    CHISQ Reduced chi-squared
    FLUX Total image flux
  • #818 North / East RA/Dec still confusing
  • OImaging exporte de l'OIFITSv1 meme quand l'input est OIFITSv2 (voir #824)
  • l'export OImage ne contient pas l'image initiale (au moins avec MiRA) (voir # 826 et un peu #879)
  • Apres reconstruction Mira puis wisard
    • -> COMMENT Image reconstructed by MiRA algorithm <https://github.com/emmt/MiRA>COMMENT Arguments: --oi-imaging --save_visibilities --recenter --bootstrap=2 --vCOMMENT erb=1 /usr/local/tomcat/temp/uws/UPLOAD_1631010593982_inputfile /usr/locCOMMENT al/tomcat/temp/uws/UPLOAD_1631010593982_inputfile.outCOMMENT Regularization: "hyperbolic" with MU=100, TAU=0.0001, ETA=1END

  • #933 BSMEM: RGL_ALPH and RGL_BETA are not required.
  • check mandatory KEYWORD #936 see spreadsheet https://docs.google.com/spreadsheets/d/1JwgiJLGVmE9YrQLsywpV7lEhHCSs13t63TTIzRr_vTQ/edit#gid=1224797885
  • Non-affichage de l'onglet OIFits : "missing target name in response". Scénario : ouvrir OImaging. charger PI_GRU_forImage.fits. Run. Aller à l'onglet OIFits.
  • Étrange valeur pour INIT_IMG: utilise un label d'image de départ qui a changé depuis. Scénario : Ouvrir OImaging avec zéro run. Lancer WISARD sur PI_GRU_forImage.fits. Noter que le résultat s'est placé en image de départ, et s'appelle "PI_GRU_32x32_50". Relancer à nouveau WISARD en cliquant sur Run. Noter que le keyword INIT_IMG contient "PI_GRU_32x32_50". Seulement, observer dans la liste d'images en haut, que c'est le nouveau nom de la NOUVELLE image d'arrivée. L'image de départ a été renommée en "PI_GRU_32x32_50_2021-10-04-etc-date-du-jour". Pour se faire peur, sélectionner maintenant "PI_GRU_32x32_50" dans la liste d'image d'input à gauche. Ensuite, essayer de changer l'image dans la liste d'image en haut. Observer que la liste du haut ne modifie plus l'image affichée en viewer panel (on reste bloqué en mode INPUT).

Softs:

  • BSMEM semble etre plus précis dans les retours
  • BSMEM -> le menu RGL_PRIO pour selectionner l'image a priori n'est pas dans le panel de gauche (Ticket #811)
  • WISARD: aligner THRESHOLD sur les valeurs de FLUXERR (d'ailleurs, remplacer carrément le mot-clé THRESHOLD par FLUXERR dans l'interface, tout en passant toujours THRESHOLD à wisard-ci). Et rajouter FLUX comme pour les autres (il vaut 1 pour wisard, tout le temps).
  • WISARD n'utilise pas le mot cle RGL_WGT mais SCALE
  • MIRA : bug dans la partie IDL qui remet en forme les données (utilisé pour mira)?

Améliorations

  • Supprimer de l'action Panel " Ready to spawn process"
  • manque le keyword algorithm
  • difficulté d'identifier les images
  • pas de possibilité de régler la dynamique générale
  • améliorer la présentation de l'affichage des longueurs d'ondes pour la table; afficher plutot la bande ?
  • boutons Save image / save OIfits: que sauvent ils et quand?
  • revoir les panneaux affichés lorsque plusieurs images sont selectionnées => vue speciale GRID
  • Ergonomie modification de l'image d'entrée / ne pas modifier un résultat
  • Export fits / png /pdf
    • inclusion ou pas du réticule axe X/Y en gris foncé
    • inclusion de la regle
  • Notebook?
  • question de sauvegarde de la session
    • avec par ex la sortie des executions logs (cf dev mode)
    • possibilité de stockage arbitraire des resultas intermediaires
    • sauvegarde automatique de la session
    • nettoyage de la session par le bouton delete
  • Mots clefs pour un mapping avec le standard phase 3 de l'ESO
  • #823 allow user to change / add keyword
  • fix hyper-parameters default values
  • Proposer un moyen d'avoir une image de départ #878 (cause de #879?)
    • bibliothèque d'image de départ
    • module importé de LITpro?
    • ajout de fonctionnalité rescale / rotate / padd #935
    • #957 filtre les images remplies de zero
    • #973 bien afficher les unités (et faire les conversions)
  • renseigner l'utilisateur sur la taille du champ et du pixel minimales
  • #1052 qualité des images
  • ajouter les logs dans l'image?
  • Taille des colonnes dans l'onglet Paramètres : KeyWord sera sûrement toujours de taille 8, pourtant la colonne fait la même taille que la colonne Description, cette dernière manque donc de place sur petite résolution.
  • Clic droit -> Copier, dans l'onglet Oifits->data. (Ctrl+c est déjà implémenté donc il s'agit d'ajouter un évènement souris lié à la même action)
  • Tooltip dans tous les tableaux, dès que le contenu de la case est coupé par des petits points (...), faire apparaître le texte complet dans un tooltip
  • Retirer le dialog de mise à jour de OImaging -> remplacer par un message en bas à droite ? Non, c'est le standard du JMMC (jmcs)

Outils développement

  • plan de test
  • integration continue? peut être plus de travail que nécéssaire.

Mots cles phase 3 ESO donné par Claudia Palidini sur slack fin aout 2021:

A preliminary list with some extra info I received from Laura Mascetti and the ASG group:
Please find below a preliminary list of the missing keywords required for Phase 3, they are defined in details in the Phase3 Science Data Product Standard. The keywords are needed to characterize the products so that they can be integrated in the ESO archive and users have all the information to be able to exploit the data further:PRODCATG = ‘SCIENCE.IMAGE’ or maybe PRODCATG = ‘SCIENCE.CUBE.IFS’, still to be discussed.CHECKSUM ‘…’
DATASUM =‘….’
ORIGIN
‘ESO’ / European Southern Observatory
OBSTECH = ‘INTERFEROMETRY’ / -> To be taken from the raw file mandatory keyword MJD-OBS is undefined. -> to be tacked from the raw file.
mandatory keyword MJD-END is undefined. -> MJD-OBS of the raw file + its exposuremandatory keyword PROCSOFT is undefined: name of the reduction software usedREFERENC = ‘2018Natur.553..310P’TELESCOP= ‘ESO-VLTI-A1234’ / ESO <TEL> -> To be taken from the raw fileINSTRUME= ‘PIONIER’ / Instrument usedmandatory keyword PROG_ID is undefined. -> To be taken from the raw file
mandatory keyword OBID1 is undefined. -> To be taken from the raw filemandatory keyword FILTER is undefined.
mandatory keyword OBJECT is undefined.
FLUXCAL = ‘ABSOLUTE’mandatory keyword EXPTIME is undefined.
mandatory keyword TEXPTIME is undefined. -> these two keywords have the same value in your case. Their values might differ only in case of mosaics made of original files covering not the same area.mandatory keyword BUNIT is undefined.
mandatory keyword PHOTZP is undefined.
mandatory keyword PHOTSYS is undefined.
mandatory keyword ABMAGLIM is undefined. -> estimated sensitivity
mandatory keyword ABMAGSAT is undefined.
mandatory keyword PSF_FWHM is undefinedAnd the ones related to the astrometric calibration:mandatory keyword RA is undefined.
mandatory keyword DEC is undefined.mandatory keyword CRVAL1 is undefined.
mandatory keyword CRVAL2 is undefined.
mandatory keyword CRPIX1 is undefined.
mandatory keyword CRPIX2 is undefined.
mandatory keyword CD1_1 is undefined.
mandatory keyword CD1_2 is undefined.
mandatory keyword CD2_1 is undefined.
mandatory keyword CD2_2 is undefined.
mandatory keyword CTYPE1 is undefined.
mandatory keyword CTYPE2 is undefined.
mandatory keyword CUNIT1 is undefined.
mandatory keyword CUNIT2 is undefined.

Edit | Attach | Watch | Print version | History: r17 < r16 < r15 < r14 < r13 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r17 - 2021-10-04 - AntoineKaszczyc
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2024 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback