In the rules report report I want to know whether a rule is giving an issue, or whether it is skipped or whether it passed. Therefore we need to have an updated outcome:
- Skipped: one of the prerequisites was not available so the rule was not executed
- Success: the rule did not provide any output
- Issue reported: the rule encountered and issue in the data and the issue can be found in the issue summary and issue detail
report-template-update.xlsx
From #788
This ticket will be to two-fold:
-
the first AC would be to enhance logging for the is_suitable_for_validation functionality.
|
if self.rule_processor.is_suitable_for_validation( |
For each test in the suitable_for_validation--adding logs would be beneficial to troubleshoot why rules are skipped.
-
the second, more involved AC would be to create an object when a rule is skipped that is carried over to the report. The report doc should be updated to include a column to the right column of 'Status' in the Rule Report Tab, This would include the reason it is skipped whether it is rule structure or class / domain not applicable to the rule. Ideally, it would eventually include errors during execution as well.
In the rules report report I want to know whether a rule is giving an issue, or whether it is skipped or whether it passed. Therefore we need to have an updated outcome:
report-template-update.xlsx
From #788
This ticket will be to two-fold:
the first AC would be to enhance logging for the is_suitable_for_validation functionality.
cdisc-rules-engine/cdisc_rules_engine/rules_engine.py
Line 158 in 85f33e6
For each test in the suitable_for_validation--adding logs would be beneficial to troubleshoot why rules are skipped.
the second, more involved AC would be to create an object when a rule is skipped that is carried over to the report. The report doc should be updated to include a column to the right column of 'Status' in the Rule Report Tab, This would include the reason it is skipped whether it is rule structure or class / domain not applicable to the rule. Ideally, it would eventually include errors during execution as well.