tricc-oo 1.6.14__py3-none-any.whl → 1.6.16__py3-none-any.whl

tricc_oo/converters/xml_to_tricc.py

@@ -483,7 +483,7 @@ def set_additional_attributes(attribute_names, elm, node):
             # input expression can add a condition to either relevance (display) or calculate expression
             if attributename == "expression_inputs":
                 attribute = [attribute]
-            elif attributename == "instance":
+            elif attributename in ["priority", "instance"]:
                 attribute = int(attribute)
             else:
                 attribute
@@ -612,7 +612,11 @@ def inject_bridge_path(node, nodes):
         calc = TriccNodeDisplayBridge(**data)
     else:
         calc = TriccNodeBridge(**data)
-
+    if node:
+        priority = getattr(node, 'priority', None)
+        if priority:
+            calc.priority = priority
+            
     for e in node.activity.edges:
         if e.target == node.id:
             if e.source in node.activity.nodes and len(node.activity.nodes[e.source].next_nodes):
@@ -632,23 +636,23 @@ def enrich_node(diagram, media_path, edge, node, activity, help_before=False):
         # get node and process type
         type, message = get_message(diagram, edge.source_external_id)
         if type is not None:
-            if type == "help":
-                help = TriccNodeMoreInfo(
-                    id=generate_id(),
-                    name=f"{node.name}.more_info",
-                    label=message,
-                    parent=node,
-                    group=node.group,
-                    activity=node.activity,
-                    required=None,
-                )
-                # node.help = message
-                if help_before:
-                    inject_node_before(help, node, activity)
-                else:
-                    set_prev_next_node(node, help, edge_only=True, activity=activity)
-                    activity.nodes[help.id] = help
-                return help, None
+            # if type == "help":
+            #     help = TriccNodeMoreInfo(
+            #         id=generate_id(),
+            #         name=f"{node.name}.more_info",
+            #         label=message,
+            #         parent=node,
+            #         group=node.group,
+            #         activity=node.activity,
+            #         required=None,
+            #     )
+            #     # node.help = message
+            #     if help_before:
+            #         inject_node_before(help, node, activity)
+            #     else:
+            #         set_prev_next_node(node, help, edge_only=True, activity=activity)
+            #         activity.nodes[help.id] = help
+            #     return help, None
 
             if type in (TriccNodeType.start, TriccNodeType.activity_start):
                 return True

tricc_oo/serializers/xls_form.py

@@ -432,7 +432,7 @@ def get_attr_if_exists(strategy, node, column, map_array):
         return ""
 
 
-def get_more_info_select(strategy, node):
+def get_more_info_select(strategy, base_name, relevance):
     values = []
     for column in SURVEY_MAP:
         if column == "type":
@@ -440,28 +440,30 @@ def get_more_info_select(strategy, node):
         elif column == "label":
             values.append(strategy.get_empty_label())
         elif column == "name":
-            values.append(get_export_name(node) + "_optin")
-        elif column == "hint":
-            print(get_xfrom_trad(strategy, node, column, SURVEY_MAP, clean_html=True))
-            values.append(get_xfrom_trad(strategy, node, column, SURVEY_MAP, clean_html=True))
+            values.append(base_name + "_optin")
         elif column == "relevance":
-            values.append(get_xfrom_trad(strategy, node.parent, column, SURVEY_MAP))
+            values.append(relevance)
         else:
             values.append(get_xfrom_trad(strategy, None, column, SURVEY_MAP))
     return values
 
 
-def get_more_info_message(strategy, node):
+def get_more_info_message(strategy, base_name, message):
     values = []
     for column in SURVEY_MAP:
         if column == "type":
             values.append("note")
+        elif column == "name":
+            values.append(base_name)
         elif column == "relevance":
-            values.append(f"${{{get_export_name(node)}_optin}} = 1")
-        elif column.startswith("hint"):
-            values.append(langs.get_trads("", trad=None))
+            values.append(f"${{{base_name}_optin}} = 1")
+        elif column.startswith("label"):
+            arr = column.split("::")
+            column = arr[0]
+            trad = arr[1] if len(arr) == 2 else None
+            values.append(langs.get_trads(message, trad=trad))
         else:
-            values.append(get_xfrom_trad(strategy, node, column, SURVEY_MAP, clean_html=True))
+            values.append(get_xfrom_trad(strategy, None, column, SURVEY_MAP, clean_html=True))
     return values
 
 
@@ -482,6 +484,13 @@ def get_more_info_choice(strategy):
     return values
 
 
+def inject_more_info(strategy, base_name, relevance, message, df_survey, df_choice):
+    df_survey.loc[len(df_survey)] = get_more_info_select(strategy, base_name, relevance)
+    df_survey.loc[len(df_survey)] = get_more_info_message(strategy, base_name, message)
+    if len(df_choice[(df_choice["list_name"] == "more_info")]) == 0:
+        df_choice.loc[len(df_choice)] = get_more_info_choice(strategy)
+
+
 def generate_xls_form_export(
     strategy,
     node,
@@ -492,6 +501,7 @@ def generate_xls_form_export(
     df_calculate,
     cur_group,
     calculates,
+    help_before=False,
     **kwargs,
 ):
     # check that all prev nodes were processed
@@ -503,6 +513,15 @@ def generate_xls_form_export(
         codesystems=kwargs.get("codesystems", None),
     ):
         if node not in processed_nodes:
+            if help_before and getattr(node, 'help', None):
+                base_name=f"{get_export_name(node)}_more_info"
+                inject_more_info(
+                    strategy,
+                    base_name,
+                    get_xfrom_trad(strategy, node, "relevance", SURVEY_MAP),
+                    node.help,
+                    df_survey,
+                    df_choice)
             add_calculate(calculates, node)
             if node.group != cur_group and not isinstance(node, TriccNodeSelectOption):
                 return False
@@ -530,10 +549,14 @@ def generate_xls_form_export(
                     ):
                         df_choice.loc[len(df_choice)] = values
                 elif isinstance(node, TriccNodeMoreInfo):
-                    df_survey.loc[len(df_survey)] = get_more_info_select(strategy, node)
-                    df_survey.loc[len(df_survey)] = get_more_info_message(strategy, node)
-                    if len(df_choice[(df_choice["list_name"] == "more_info")]) == 0:
-                        df_choice.loc[len(df_choice)] = get_more_info_choice(strategy)
+                    inject_more_info(
+                        strategy,
+                        get_export_name(node),
+                        get_xfrom_trad(strategy, node.parent, "relevance", SURVEY_MAP),
+                        node.label,
+                        df_survey,
+                        df_choice
+                    )
                 elif node.tricc_type in ODK_TRICC_TYPE_MAP and ODK_TRICC_TYPE_MAP[node.tricc_type] is not None:
                     if ODK_TRICC_TYPE_MAP[node.tricc_type] == "calculate":
                         values = []
@@ -560,6 +583,16 @@ def generate_xls_form_export(
                         logger.warning("node {} have an unmapped type {}".format(node.get_name(), node.tricc_type))
                 else:
                     logger.warning("node {} have an unsupported type {}".format(node.get_name(), node.tricc_type))
+            if not help_before and getattr(node, 'help', None):
+                base_name=f"{get_export_name(node)}_more_info"
+                inject_more_info(
+                    strategy,
+                    base_name,
+                    get_xfrom_trad(strategy, node, "relevance", SURVEY_MAP),
+                    node.help,
+                    df_survey,
+                    df_choice
+                )
             # continue walk °
             return True
     return False

tricc_oo/strategies/output/xlsform_cht.py

@@ -640,6 +640,7 @@ class XLSFormCHTStrategy(XLSFormCDSSStrategy):
         df_settings.to_excel(writer, sheet_name="settings", index=False)
         writer.close()
         # pause
+        logger.info("generating the task and after pause questionnaires")
         ends = []
         for p in self.project.pages.values():
             p_ends = list(

tricc_oo/visitors/tricc.py

@@ -56,6 +56,9 @@ from tricc_oo.converters.tricc_to_xls_form import get_list_names, get_export_nam
 logger = logging.getLogger("default")
 ONE_QUESTION_AT_A_TIME = False
 
+# Track the last group that was reordered to avoid unnecessary reordering
+_last_reordered_group = None
+
 
 def merge_node(from_node, to_node):
     if from_node.activity != to_node.activity:
@@ -405,6 +408,11 @@ def get_bridge_path(prev_nodes, node=None, edge_only=False):
         calc = TriccNodeDisplayBridge(**data)
     else:
         calc = TriccNodeBridge(**data)
+    if node:
+        priority = getattr(node, 'priority', None)
+        if priority:
+            calc.priority = priority
+    
     return calc
 
 
@@ -968,6 +976,7 @@ def walktrhough_tricc_node_processed_stached(
     warn=False,
     node_path=[],
     process=None,
+    loop_count=0,
     **kwargs,
 ):
     ended_activity = False
@@ -1133,7 +1142,10 @@ def walktrhough_tricc_node_processed_stached(
                     if nn not in stashed_nodes:
                         stashed_nodes.insert_at_top(nn)
         if not recursive:
-            reorder_node_list(stashed_nodes, node.group, processed_nodes)
+            global _last_reordered_group
+            if _last_reordered_group != node.group:
+                reorder_node_list(stashed_nodes, node.group, processed_nodes)
+                _last_reordered_group = node.group
 
     else:
         if prev_process and process and prev_process != process[0]:
@@ -1280,7 +1292,7 @@ def stashed_node_func(node, callback, recursive=False, **kwargs):
 
 
 # check if the all the prev nodes are processed
-def is_ready_to_process(in_node, processed_nodes, strict=True, local=False):
+def is_ready_to_process(in_node, processed_nodes, strict=True, local=False, loop_count=0):
     if isinstance(in_node, TriccNodeSelectOption):
         node = in_node.select
     elif isinstance(in_node, (TriccNodeActivityStart, TriccNodeMainStart)):
@@ -1291,36 +1303,38 @@ def is_ready_to_process(in_node, processed_nodes, strict=True, local=False):
     if hasattr(node, "prev_nodes"):
         # ensure the  previous node of the select are processed, not the option prev nodes
         for prev_node in node.prev_nodes:
-            if is_prev_processed(prev_node, node, processed_nodes, local) is False:
+            if is_prev_processed(prev_node, node, processed_nodes, local, loop_count) is False:
                 return False
     return True
 
 
-def is_prev_processed(prev_node, node, processed_nodes, local):
+def is_prev_processed(prev_node, node, processed_nodes, local, loop_count=0):
     if hasattr(prev_node, "select"):
-        return is_prev_processed(prev_node.select, node, processed_nodes, local)
+        return is_prev_processed(prev_node.select, node, processed_nodes, local, loop_count)
     if prev_node not in processed_nodes and (not local):
-        if isinstance(prev_node, TriccNodeExclusive):
-            iterator = iter(prev_node.prev_nodes)
-            p_n_node = next(iterator)
-            logger.debug(
-                "is_ready_to_process:failed:via_excl: {} - {} > {} {}:{}".format(
-                    get_data_for_log(p_n_node), prev_node.get_name(), node.__class__, node.get_name(), node.instance
+        # Only log detailed failures when we suspect dependency loops (loop_count > 5)
+        if loop_count > 5:
+            if isinstance(prev_node, TriccNodeExclusive):
+                iterator = iter(prev_node.prev_nodes)
+                p_n_node = next(iterator)
+                logger.debug(
+                    "is_ready_to_process:failed:via_excl: {} - {} > {} {}:{}".format(
+                        get_data_for_log(p_n_node), prev_node.get_name(), node.__class__, node.get_name(), node.instance
+                    )
                 )
-            )
 
-        else:
-            logger.debug(
-                "is_ready_to_process:failed: {} -> {} {}:{}".format(
-                    get_data_for_log(prev_node), node.__class__, node.get_name(), node.instance
+            else:
+                logger.debug(
+                    "is_ready_to_process:failed: {} -> {} {}:{}".format(
+                        get_data_for_log(prev_node), node.__class__, node.get_name(), node.instance
+                    )
                 )
-            )
 
-        logger.debug(
-            "prev node node {}:{} for node {} not in processed".format(
-                prev_node.__class__, prev_node.get_name(), node.get_name()
+            logger.debug(
+                "prev node node {}:{} for node {} not in processed".format(
+                    prev_node.__class__, prev_node.get_name(), node.get_name()
+                )
             )
-        )
         return False
     return True
 
@@ -1753,9 +1767,12 @@ def reorder_node_list(node_list, group, processed_nodes):
             return MAP_PRIORITIES[node.id]
         if isinstance(node, (TriccNodeActivityStart, TriccNodeMainStart)):
             return get_priority(node.activity)
+        if isinstance(node, (TriccNodeSelectOption)):
+            return get_priority(node.select)
 
         # Cache attributes to avoid repeated getattr calls
-        priority = int(getattr(node, "priority", 0) or 0) 
+        explicit_priority = getattr(node, "priority", None)
+        priority = int(explicit_priority or 0)
         node_group = getattr(node, "group", None)
         activity = getattr(node, "activity", None)
 
@@ -1785,7 +1802,7 @@ def reorder_node_list(node_list, group, processed_nodes):
         elif issubclass(node.__class__, TriccRhombusMixIn):
             priority += RHOMBUS_PRIORITY_TO_UP
 
-        if node.prev_nodes:
+        if node.prev_nodes and not explicit_priority and not isinstance(node, TriccNodeMainStart):
             priority = max(priority, *[get_priority(p) for p in node.prev_nodes])
         
         MAP_PRIORITIES[node.id] = priority

{tricc_oo-1.6.14.dist-info → tricc_oo-1.6.16.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tricc-oo
-Version: 1.6.14
+Version: 1.6.16
 Summary: Python library that converts CDSS L2 in L3
 Project-URL: Homepage, https://github.com/SwissTPH/tricc
 Project-URL: Issues, https://github.com/SwissTPH/tricc/issues

{tricc_oo-1.6.14.dist-info → tricc_oo-1.6.16.dist-info}/RECORD

@@ -9,7 +9,7 @@ tricc_oo/converters/datadictionnary.py,sha256=JasqlLKiZzKdidsA1xc2SJ_Af1Xr6A3sKf
 tricc_oo/converters/drawio_type_map.py,sha256=Zp8J9iHNSJkIVrmRSM0_d4vA1X8wFPLKb8nCMPUMXKU,9114
 tricc_oo/converters/tricc_to_xls_form.py,sha256=39hwWgYNitGE-AuKtjUwNLz39tEpwc7nd9gT_gw5wjc,3842
 tricc_oo/converters/utils.py,sha256=JZrtrvvOfXwdkw49pKauzinOcauWwsy-CVcw36TjyLo,1684
-tricc_oo/converters/xml_to_tricc.py,sha256=ea8LNEPDe32q74AJCbEjxaLt_Po47oH45K_G8fo7TzE,40388
+tricc_oo/converters/xml_to_tricc.py,sha256=6cl0X2cHACVq56cCd9ZHj1ixpsBYSu0ruSKJSQH6ZZI,40570
 tricc_oo/converters/cql/cqlLexer.py,sha256=8HArbRphcrpnAG4uogJ2rHv4tc1WLzjN0B1uFeYILAc,49141
 tricc_oo/converters/cql/cqlListener.py,sha256=fA7-8DcS2Q69ckwjdg57-OfFHBxjTZFdoSKrtw7Hffc,57538
 tricc_oo/converters/cql/cqlParser.py,sha256=x3KdrwX9nwENSEJ5Ex7_l5NMnu3kWBO0uLdYu4moTq0,414745
@@ -25,7 +25,7 @@ tricc_oo/parsers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,
 tricc_oo/parsers/xml.py,sha256=uzkb1y18MHfqVFmZqVh0sKT4cx6u0-NcAT_lV_gHBt8,4208
 tricc_oo/serializers/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tricc_oo/serializers/planuml.py,sha256=t57587-6L3aDncpHh58lS77Zft8yxDE9DPtXx2BeUSU,132
-tricc_oo/serializers/xls_form.py,sha256=ydaD5l_CsMEhJlSO0XvXPdxrwsDw4HiRjTOPHogTQdw,23071
+tricc_oo/serializers/xls_form.py,sha256=E0VKcAXoVvGfinMZXOmo-90uypjsGAKS2S3-mtvAT7I,24215
 tricc_oo/strategies/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tricc_oo/strategies/input/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 tricc_oo/strategies/input/base_input_strategy.py,sha256=BEODXS74na1QRRcJVQ4cxiD8F7uRqaLyhE3QzKpGVvk,3891
@@ -38,14 +38,14 @@ tricc_oo/strategies/output/openmrs_form.py,sha256=ne6TwAyhafR-WDs27QTKKFl85VD5si
 tricc_oo/strategies/output/spice.py,sha256=QMeoismVC3PdbvwTK0PtUjWX9jl9780fbQIXn76fMXw,10761
 tricc_oo/strategies/output/xls_form.py,sha256=_pNTND7n-55EjRphJ1hSVtRYa-UkXlmwpam2OKQ8o_w,30860
 tricc_oo/strategies/output/xlsform_cdss.py,sha256=X00Lt5MzV8TX14dR4dFI1MqllI5S1e13bKbeysWM9uA,17435
-tricc_oo/strategies/output/xlsform_cht.py,sha256=Zy8aggR1sTBP0b33RGbfUpk8pRppI1LGQEif4E1l49A,28523
+tricc_oo/strategies/output/xlsform_cht.py,sha256=0wrZ_Ez41D-SciXoBq4Pn4o1bK1ofUxXqbTjDoQuCGE,28597
 tricc_oo/strategies/output/xlsform_cht_hf.py,sha256=xm6SKirV3nMZvM2w54_zJcXAeAgAkq-EEqGEjnOWv6c,988
 tricc_oo/visitors/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-tricc_oo/visitors/tricc.py,sha256=QkVZpmDvSrMbMQeCxVn15zSk3iH-3-e-udMys96JkXE,110923
+tricc_oo/visitors/tricc.py,sha256=ArvDNcuzIQNJpYyT1vB_B9Gxc2m5YAxg9ss_orpMqK0,111780
 tricc_oo/visitors/utils.py,sha256=j83aAq5s5atXi3OC0jc_uJd54a8XrHHmizeeEbWZQJg,421
 tricc_oo/visitors/xform_pd.py,sha256=ryAnI3V9x3eTmJ2LNsUZfvl0_yfCqo6oBgeSu-WPqaE,9613
-tricc_oo-1.6.14.dist-info/licenses/LICENSE,sha256=Pz2eACSxkhsGfW9_iN60pgy-enjnbGTj8df8O3ebnQQ,16726
-tricc_oo-1.6.14.dist-info/METADATA,sha256=_WkaLTsN1Yj8koCLVrOmdY9egk06du90gsMqRCcQ0Js,8600
-tricc_oo-1.6.14.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-tricc_oo-1.6.14.dist-info/top_level.txt,sha256=NvbfMNAiy9m4b1unBsqpeOQWh4IgA1Xa33BtKA4abxk,15
-tricc_oo-1.6.14.dist-info/RECORD,,
+tricc_oo-1.6.16.dist-info/licenses/LICENSE,sha256=Pz2eACSxkhsGfW9_iN60pgy-enjnbGTj8df8O3ebnQQ,16726
+tricc_oo-1.6.16.dist-info/METADATA,sha256=9eOqHUBcjhaviC0zU-JT-NYWTY-yc6kq3r8iv6qbGoE,8600
+tricc_oo-1.6.16.dist-info/WHEEL,sha256=qELbo2s1Yzl39ZmrAibXA2jjPLUYfnVhUNTlyF1rq0Y,92
+tricc_oo-1.6.16.dist-info/top_level.txt,sha256=NvbfMNAiy9m4b1unBsqpeOQWh4IgA1Xa33BtKA4abxk,15
+tricc_oo-1.6.16.dist-info/RECORD,,

{tricc_oo-1.6.14.dist-info → tricc_oo-1.6.16.dist-info}/WHEEL

@@ -1,5 +1,5 @@
 Wheel-Version: 1.0
-Generator: setuptools (80.9.0)
+Generator: setuptools (80.10.1)
 Root-Is-Purelib: true
 Tag: py3-none-any