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

Temporary solution for demo | chore(torchlib) #930

Merged
merged 13 commits into from
Jul 28, 2023

Conversation

wschin
Copy link
Contributor

@wschin wschin commented Jul 27, 2023

Hack for #493 for demo. Will need a more proper solution.

@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #930 (fe59488) into main (176141e) will decrease coverage by 0.10%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main     #930      +/-   ##
==========================================
- Coverage   76.87%   76.78%   -0.10%     
==========================================
  Files         112      112              
  Lines       13736    13760      +24     
  Branches     1407     1408       +1     
==========================================
+ Hits        10560    10566       +6     
- Misses       2824     2842      +18     
  Partials      352      352              
Files Changed Coverage Δ
...ipt/function_libs/torch_lib/graph_building_test.py 77.86% <21.73%> (-13.05%) ⬇️
...nxscript/function_libs/torch_lib/graph_building.py 83.82% <100.00%> (+0.04%) ⬆️

@wschin wschin marked this pull request as ready for review July 28, 2023 00:21
@justinchuby
Copy link
Collaborator

Merging for demo

@justinchuby justinchuby changed the title Temporal solution for demo Temporal solution for demo | chore(torchlib) Jul 28, 2023
@justinchuby justinchuby added the topic: torch_lib Related to the torch/aten function lib in development label Jul 28, 2023
.github/workflows/main.yaml Outdated Show resolved Hide resolved
noxfile.py Outdated Show resolved Hide resolved
@wschin
Copy link
Contributor Author

wschin commented Jul 28, 2023

@justinchuby, I disable that test in CI because CI is not ready. That new line is always run by other to_model_proto calls, so it should be fine.

@justinchuby justinchuby changed the title Temporal solution for demo | chore(torchlib) Temporary solution for demo | chore(torchlib) Jul 28, 2023
@justinchuby justinchuby merged commit 0be9a64 into microsoft:main Jul 28, 2023
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: torch_lib Related to the torch/aten function lib in development
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants