Skip to content

Change _const prefix to _val | feat(torchlib) #886

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

Merged
merged 2 commits into from
Jul 22, 2023

Conversation

justinchuby
Copy link
Collaborator

@justinchuby justinchuby commented Jul 17, 2023

Use _val for all intermediate values to avoid ambiguity based on comment: #881 (review)

@justinchuby justinchuby added the module: torchlib Related to the torch/aten function lib in development label Jul 17, 2023
@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Merging #886 (1f30e26) into main (b2c19a7) will increase coverage by 0.00%.
The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             main     #886   +/-   ##
=======================================
  Coverage   76.73%   76.74%           
=======================================
  Files         112      112           
  Lines       13490    13486    -4     
  Branches     1364     1363    -1     
=======================================
- Hits        10352    10350    -2     
+ Misses       2797     2796    -1     
+ Partials      341      340    -1     
Impacted Files Coverage Δ
...nxscript/function_libs/torch_lib/graph_building.py 84.43% <66.66%> (+0.40%) ⬆️

@justinchuby justinchuby merged commit 457383d into main Jul 22, 2023
@justinchuby justinchuby deleted the justinchu/val-name-update branch July 22, 2023 00:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: torchlib Related to the torch/aten function lib in development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants