Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Display dataflow analysis (if attached) in IR graph #183

Merged
merged 5 commits into from
Nov 10, 2023

Conversation

joscao
Copy link
Contributor

@joscao joscao commented Oct 27, 2023

Enhances the IR graph representation by also showing lives,uses and defines when dataflow_analysis is attached

uses_defines_live

  • Provide a reasonable test case

@codecov
Copy link

codecov bot commented Oct 27, 2023

Codecov Report

Merging #183 (9016906) into main (af8ec9f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #183   +/-   ##
=======================================
  Coverage   92.14%   92.14%           
=======================================
  Files          90       90           
  Lines       16690    16697    +7     
=======================================
+ Hits        15379    15386    +7     
  Misses       1311     1311           
Flag Coverage Δ
lint_rules 96.22% <ø> (ø)
loki 92.11% <100.00%> (+<0.01%) ⬆️
transformations 91.40% <ø> (ø)

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

Files Coverage Δ
loki/visitors/ir_graph.py 95.16% <100.00%> (+0.28%) ⬆️

@joscao
Copy link
Contributor Author

joscao commented Oct 31, 2023

Provided a test case. e34b98f

@joscao joscao changed the title Draft: Enable ir_graph representation while dataflow analysis Enable ir_graph representation while dataflow analysis Nov 2, 2023
Copy link
Collaborator

@reuterbal reuterbal left a comment

Choose a reason for hiding this comment

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

Useful addition, many thanks!. Implementation looks good to me

uses_symbols,
"]",
)
except (RuntimeError, KeyError, AttributeError) as _:
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you can simply skip the as _ if you're not planning to use the exception

@reuterbal reuterbal added the ready for merge This PR has been approved and is ready to be merged label Nov 9, 2023
@reuterbal reuterbal changed the title Enable ir_graph representation while dataflow analysis Display dataflow analysis (if attached) in IR graph Nov 9, 2023
@reuterbal reuterbal merged commit a2f2428 into ecmwf-ifs:main Nov 10, 2023
8 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for merge This PR has been approved and is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants