[REDMINE1D-349] [RM-8454] Segmentation fault when launching amazed with an existing empty calibration dir Created: 01/Nov/23  Updated: 22/Nov/23  Resolved: 22/Nov/23

Status: Done
Project: 1D Redmine
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Task Priority: Normal
Reporter: Redmine-Jira Migtation Assignee: Redmine-Jira Migtation
Resolution: Done Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified


 Description   

Created on 2023-10-31 16:37:39 by Fanny Dufresne. % Done: 100

When launching amazed with :

  • an non-existent calibration_dir in config.json: an error is raised.
  • an existing calibration_dir directory but which doesn't contain the expected calibration files: a segmentation fault error with no more comments

=> We would like a more explicit error if calibration_dir does not contain the expected files

MR: https://gitlab.lam.fr/CPF/cpf-redshift/-/merge_requests/562

 Comments   
Comment by Redmine-Jira Migtation [ 22/Nov/23 ]

Comment by Didier Vibert on 2023-11-08 09:27:42:
@fdufresne est-ce que tu as un liens vers un rep de sortie sur le cluster qui produit ça (ou un parameter.json) ?

suite à la discussion d'hier:

  • il faut border ça dans calibration.py
  • il faut aussi debugger le C++ pour comprendre où il segfault et ajouter une exception avant....
Comment by Redmine-Jira Migtation [ 22/Nov/23 ]

Comment by Didier Vibert on 2023-11-09 08:31:28:
hello, le problème venait de la lecture des courbes d'extinctions igm.
Pour que les tests unitaires python passent avec un seul bin en z pour alléger le test, quelqu'un avait laisser passer le fait qu'un fichier soit absent, et du coup on laissait aussi passer tous les fichiers d'extinctions absents. J'ai corrigé ça dans calibration.py au load_meiksin, j'ai aussi corrigé le c++ pour éviter la segfault avec une exception.

J'en ai profité pour ajouter quelques exceptions manquantes lors des différents load.

@aallaoui tu fais la revue ?

Comment by Redmine-Jira Migtation [ 22/Nov/23 ]

Comment by Pierre-yves Chabaud on 2023-11-15 11:02:50:
Merged into @develop@ (@1891e735@)

Generated at Sat Feb 10 15:31:49 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.