Skip to content

feat(atenlib): where, full #286

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 3 commits into from
Jan 6, 2023
Merged

feat(atenlib): where, full #286

merged 3 commits into from
Jan 6, 2023

Conversation

justinchuby
Copy link
Collaborator

@justinchuby justinchuby commented Jan 6, 2023

Implement where, full, new_full, full_like

@justinchuby justinchuby added the module: torchlib Related to the torch/aten function lib in development label Jan 6, 2023
@justinchuby justinchuby mentioned this pull request Jan 6, 2023
@justinchuby justinchuby changed the title feat(atenlib): where, fill, fill_like feat(atenlib): where, full Jan 6, 2023
@codecov
Copy link

codecov bot commented Jan 6, 2023

Codecov Report

Merging #286 (c6ce881) into main (a733ec2) will increase coverage by 0.57%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##             main     #286      +/-   ##
==========================================
+ Coverage   72.15%   72.73%   +0.57%     
==========================================
  Files          95       95              
  Lines        9055     9063       +8     
==========================================
+ Hits         6534     6592      +58     
+ Misses       2521     2471      -50     
Impacted Files Coverage Δ
...t/function_libs/torch_aten/ops_correctness_test.py 93.96% <ø> (ø)
onnxscript/function_libs/torch_aten/ops/core.py 58.22% <87.50%> (+0.39%) ⬆️
onnxscript/analysis.py 94.07% <0.00%> (+0.65%) ⬆️
onnxscript/irbuilder.py 79.92% <0.00%> (+0.75%) ⬆️
onnxscript/tensor.py 87.85% <0.00%> (+0.93%) ⬆️
onnxscript/converter.py 92.03% <0.00%> (+1.48%) ⬆️
onnxscript/utils.py 60.31% <0.00%> (+1.58%) ⬆️
onnxscript/test/functions/onnxfns_test.py 97.67% <0.00%> (+2.32%) ⬆️
onnxscript/test/common/onnx_script_test_case.py 92.30% <0.00%> (+12.50%) ⬆️
onnxscript/test/functions/onnxfns1A_test.py 97.43% <0.00%> (+15.38%) ⬆️
... and 1 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@@ -169,6 +169,7 @@ def wrapped(fn):
"exp": core_ops.aten_exp,
"exp2": core_ops.aten_exp2,
"fmod": core_ops.aten_fmod,
"full_like": core_ops.aten_full_like,
Copy link
Contributor

Choose a reason for hiding this comment

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

Why don't we add aten::full here?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

we don't have an OpInfo for full so... I think it is innocuous enough to not include it for now?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I added a todo inline to note.

@xiaowuhu xiaowuhu merged commit 677ca1f into main Jan 6, 2023
@xiaowuhu xiaowuhu deleted the justinchu/where branch January 6, 2023 01:53
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