Skip to content

Add method reference for ‎add_table_row in document#528

Merged
zinwang merged 1 commit into
ev-flow:masterfrom
LiangPPP:master
Jul 19, 2023
Merged

Add method reference for ‎add_table_row in document#528
zinwang merged 1 commit into
ev-flow:masterfrom
LiangPPP:master

Conversation

@LiangPPP

Copy link
Copy Markdown
Contributor

add_table_row

The algorithm of add_table_row

The function add_table_row adds a list to the table.

Here is the process of add_table_row.


    1. The method add_row is then called with a list of parameters. This list includes:
        name: filename of rule in rule_obj
        rule_obj.crime: description of quark_analysis's crime in rule_obj
        confidence: the number of True values in check_item and multiplying by 20
        score: the score in rule_obj
        weight: the weight in rule_obj

    2. The add_row method takes these parameters and adds them as a new row in the summary_report_table.
 

Here is the flowchart of add_table_row.

The code of add_table_row

    def add_table_row(self, name, rule_obj, confidence, score, weight):

        self.quark_analysis.summary_report_table.add_row(
            [
                name,
                green(rule_obj.crime),
                yellow(confidence),
                score,
                red(weight),
            ]
        )

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (c6be110) 79.19% compared to head (dbfe664) 79.19%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #528   +/-   ##
=======================================
  Coverage   79.19%   79.19%           
=======================================
  Files          63       63           
  Lines        4960     4960           
=======================================
  Hits         3928     3928           
  Misses       1032     1032           
Flag Coverage Δ
unittests 79.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@zinwang zinwang self-assigned this Jul 19, 2023
@zinwang zinwang requested review from pulorsok and zinwang July 19, 2023 07:33
@zinwang zinwang added documentation Improvements or additions to documentation pr-processing-state-05 labels Jul 19, 2023

@zinwang zinwang left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation pr-processing-state-05

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants