Update app.py

app.py

@@ -350,15 +350,11 @@ class DicomAnalyzer:
         except Exception as e:
             logger.error(f"Error adding formulas: {str(e)}")
 
-    ########################################################################
-    # نفس الدالة الأصلية، لكن نهاية الدالة تضع الجدول في الصف 35 بالشكل المطلوب
-    ########################################################################
     def save_formatted_results(self, output_path):
         try:
             if not self.results:
                 return None, "No results to save"
 
-            # هنا كل شيء مثل كودك: تنشئ Workbook جديد، تصنع الشيت، تضع الهيدرز...
             wb = openpyxl.Workbook()
             ws = wb.active
             red_font = openpyxl.styles.Font(color="FF0000")
@@ -391,13 +387,11 @@ class DicomAnalyzer:
                 '(4.5mm)', '(4mm)', '(3.5mm)', '(3mm)', '(2.5mm)'
             ]
             
-            # كتابة أسماء الفانتوم باللون الأحمر في العمود A (قبل الصف الأول)
             for i, size in enumerate(phantom_sizes):
                 header_cell = ws.cell(row=row_pairs[i][0]-1, column=1, value=size)
                 header_cell.font = red_font
                 header_cell.alignment = center_alignment
 
-            # توزيع نتائج ROI في هذه الصفوف والأعمدة كما في كودك
             result_idx = 0
             current_col_group = 0
             current_row_pair = 0
@@ -426,7 +420,6 @@ class DicomAnalyzer:
                     current_col_group = 0
                     current_row_pair += 1
 
-            # ضبط المحاذاة للخلايا التي كُتبت بالفعل
             for cols in column_groups:
                 for col in cols:
                     for row in range(2, 31):
@@ -434,24 +427,15 @@ class DicomAnalyzer:
                         if cell.value is not None:
                             cell.alignment = center_alignment
 
-            ################################################################
-            # وهنا بدلًا من وضع جداول STDDEV/Mean/CNR منفصلة،
-            # نضع الجدول المدمج "1-AVG" من الصف 35 مثل الصورة التي أرسلتها
-            ################################################################
-            
-            # سنستخدم نفس فكرة الجدول بالدمج من D35:E35, F35:G35, H35:I35
+            # الآن نضع جدول 1-AVG من الصف 35 إلى 45
             start_row = 35
-            
-            # ضع عنوان "1-AVG" في C35
             ws['C35'] = "1-AVG"
             ws['C35'].alignment = center_alignment
             
-            # ادمج الخلايا الخاصة بالعناوين
             ws.merge_cells('D35:E35')
             ws.merge_cells('F35:G35')
             ws.merge_cells('H35:I35')
             
-            # ضع العناوين باللون الأحمر
             avg_headers = {
                 'D35': 'AVG MEAN',
                 'F35': 'AVG STDDEV',
@@ -462,81 +446,60 @@ class DicomAnalyzer:
                 ws[c_ref].font = red_font
                 ws[c_ref].alignment = center_alignment
             
-            # نفس أسماء الفانتوم
             phantom_sizes2 = [
                 '(7.0mm)', '(6.5mm)', '(6.0mm)', '(5.5mm)', '(5.0mm)',
                 '(4.5mm)', '(4.0mm)', '(3.5mm)', '(3.0mm)', '(2.5mm)'
             ]
             
             for i, p_size in enumerate(phantom_sizes2):
-                row = start_row + i + 1  # 36..45
+                row = start_row + i + 1  # من 36..45
                 
-                # ادمج لكل صف: D-E ، F-G ، H-I
                 ws.merge_cells(f'D{row}:E{row}')
                 ws.merge_cells(f'F{row}:G{row}')
                 ws.merge_cells(f'H{row}:I{row}')
                 
-                # اكتب اسم الفانتوم باللون الأحمر في العمود C
                 size_cell = ws[f'C{row}']
                 size_cell.value = p_size
                 size_cell.font = red_font
                 size_cell.alignment = center_alignment
 
-                # لكي نحسب الـAVG الحقيقي من الصفوف 2..3 إلخ،
-                # نستخدم row_pairs[i] إن أردت. أو كما في المثال الأصلي:
                 if i < len(row_pairs):
                     (raw_row1, raw_row2) = row_pairs[i]
                 else:
-                    continue  # لو عندك 10 فقط
-                
-                # لنفترض أننا نقرأ من نفس الأعمدة اللي فيها mean & std
-                # نفترض عندنا على الأقل عمود mean_col = cols[1], stddev_col = cols[2]
-                # لكن عندك عدة col groups؛ إذا تريد تجمع من الأول فقط:
-                # أو خذ فكرة snippetك السابق:
+                    continue
                 
                 mean_values = []
                 stddev_values = []
                 cnr_values = []
                 
-                # Example: لو تحب تجمع من نفس الـ column_groups الأولى بس
                 for group in column_groups:
-                    mean_col = group[1]  # 'C','I','O'...الخ
-                    std_col  = group[2]  # 'D','J','P'...الخ
+                    mean_col = group[1]  # ex. 'C'
+                    std_col  = group[2]  # ex. 'D'
                     
-                    # raw_row1 = الصف الأول
-                    # raw_row2 = الصف الثاني
                     m1 = ws[f"{mean_col}{raw_row1}"].value
                     m2 = ws[f"{mean_col}{raw_row2}"].value
-                    std1 = ws[f"{std_col}{raw_row1}"].value
+
+                    # في السابق كانت:
+                    # std2 = ws[f"{std_col}{raw_row2}"].value
+                    # لكنك تريد أخذ stddev من الصف الأول (raw_row1):
+                    std2 = ws[f"{std_col}{raw_row1}"].value  # <-- التعديل هنا
                     
                     try:
                         m1   = float(m1)   if m1   not in [None,''] else None
                         m2   = float(m2)   if m2   not in [None,''] else None
-                        std1 = float(std1) if std1 not in [None,''] else None
+                        std2 = float(std2) if std2 not in [None,''] else None
                     except:
-                        m1,m2,std1 = None,None,None
-                    if (m1 is not None) and (m2 is not None) and (std1 is not None) and (std1!=0):
+                        m1,m2,std2 = None,None,None
+                    if (m1 is not None) and (m2 is not None) and (std2 is not None) and (std2!=0):
                         mean_values.append(m1)
-                        stddev_values.append(std1)
-                        cnr_values.append((m1 - m2)/std1)
+                        stddev_values.append(std2)
+                        cnr_values.append((m1 - m2)/std2)
                 
-                # الآن نحسب المتوسط
-                if len(mean_values) > 0:
-                    final_mean = sum(mean_values)/len(mean_values)
-                else:
-                    final_mean = None
-                
-                if len(stddev_values) > 0:
-                    final_std = sum(stddev_values)/len(stddev_values)
-                else:
-                    final_std = None
-                
-                if len(cnr_values) > 0:
-                    final_cnr = sum(cnr_values)/len(cnr_values)
-                else:
-                    final_cnr = None
+                # نحسب المتوسط
+                final_mean = (sum(mean_values)/len(mean_values)) if mean_values else None
+                final_std  = (sum(stddev_values)/len(stddev_values)) if stddev_values else None
+                final_cnr  = (sum(cnr_values)/len(cnr_values)) if cnr_values else None
                 
-                # نكتبها في الخلايا المدمجة
                 if final_mean is not None:
                     ws[f'D{row}'].value = final_mean
                     ws[f'D{row}'].alignment = center_alignment
@@ -552,7 +515,6 @@ class DicomAnalyzer:
                     ws[f'H{row}'].alignment = center_alignment
                     ws[f'H{row}'].number_format = '0.0000'
 
-            # أخيراً، ضع الحدود حول C35..I45
             thin_side = openpyxl.styles.Side(style='thin')
             border = openpyxl.styles.Border(
                 left=thin_side, right=thin_side, top=thin_side, bottom=thin_side
@@ -566,7 +528,6 @@ class DicomAnalyzer:
         except Exception as e:
             logger.error(f"Error saving formatted results: {str(e)}")
             return None, f"Error saving results: {str(e)}"
-    ########################################################################
 
     def _write_result_to_cells(self, ws, result, cols, row):
         center_alignment = openpyxl.styles.Alignment(horizontal='center')