microsoft / microsoft/winml-cli

Analyze UI change: display patterns separately, pull pattern-contained nodes out of the node list

Open
#982 1 comment 0 reactions 1 assignee View on GitHub

Nobody has claimed this yet.

enhancement P2 static-analyzer triaged
Dominant language
Python
Stars
40
Forks
11
Avg merge
1d 8h
Merged PRs (30d)
50

Description

Background

As the pattern decision logic is cleaned up (see #981), the analyze output UI needs to be adjusted so that patterns are clearly visible in the results.

Goals

  • Display patterns separately: present each pattern as an independent unit in the analyze output.
  • Pull the pattern's constituent nodes out of the node list: nodes covered by a matched pattern are no longer shown mixed into the regular node list, but grouped under their corresponding pattern instead.

To be confirmed

The exact UI changes (layout, grouping, fields, whether a pattern is expandable to view its internal nodes, etc.) still need further confirmation before being finalized.

Related

  • Depends on #981 (clean up analyze subgraph pattern logic).

Relevant code

  • analyze/console_writer.py
  • analyze/models/output.py

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.