vyper icon indicating copy to clipboard operation
vyper copied to clipboard

feat[venom]: scheduler optimization

Open harkal opened this issue 1 year ago • 1 comments

What I did

How I did it

How to verify it

Commit message

Commit message for the final, squashed PR. (Optional, but reviewers will appreciate it! Please see our commit message style guide for what we would ideally like to see in a commit message.)

Description for the changelog

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

harkal avatar Jun 20 '24 15:06 harkal

Codecov Report

Attention: Patch coverage is 15.21739% with 195 lines in your changes missing coverage. Please review.

Project coverage is 45.79%. Comparing base (e9db8d9) to head (21e20c7).

:exclamation: Current head 21e20c7 differs from pull request most recent head 78b0aac

Please upload reports for the commit 78b0aac to get more accurate results.

Files Patch % Lines
vyper/venom/passes/dft.py 12.71% 151 Missing :warning:
vyper/utils.py 4.34% 22 Missing :warning:
vyper/venom/basicblock.py 33.33% 18 Missing :warning:
vyper/venom/__init__.py 60.00% 2 Missing :warning:
vyper/venom/analysis/analysis.py 0.00% 1 Missing :warning:
vyper/venom/analysis/cfg.py 0.00% 1 Missing :warning:

:exclamation: There is a different number of reports uploaded between BASE (e9db8d9) and HEAD (21e20c7). Click for more details.

HEAD has 201 uploads less than BASE | Flag | BASE (e9db8d9) | HEAD (21e20c7) | |------|------|------| ||202|1|
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #4161       +/-   ##
===========================================
- Coverage   91.37%   45.79%   -45.59%     
===========================================
  Files         109      109               
  Lines       15606    15799      +193     
  Branches     3432     3486       +54     
===========================================
- Hits        14260     7235     -7025     
- Misses        916     8007     +7091     
- Partials      430      557      +127     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jun 20 '24 19:06 codecov[bot]

@harkal is this superseded by #4255? if so i guess we can close this

charles-cooper avatar Nov 21 '24 07:11 charles-cooper

@harkal is this superseded by #4255? if so i guess we can close this

yes! I am closing it

harkal avatar Dec 20 '24 17:12 harkal