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

Introduce disable_dynamic_shapes experimental flag; adding its use into to_dense_batch function #7246

Merged
merged 4 commits into from
Jun 4, 2023

Conversation

piotrchmiel
Copy link
Contributor

There are devices that do not support dynamic shapes - (compiling and optimizing only static graphs). The ability to set and read the "disable_dynamic_shapes" flag allows implementors to provide static shape-friendly implementations and report user-friendly messages if it is impossible to avoid using dynamic shapes.

@piotrchmiel piotrchmiel force-pushed the piotrc/disable_dynamic_shapes branch from b397d85 to f98c31a Compare April 27, 2023 13:58
@codecov
Copy link

codecov bot commented Apr 27, 2023

Codecov Report

Merging #7246 (812e483) into master (1e57a6c) will increase coverage by 0.08%.
The diff coverage is 100.00%.

❗ Current head 812e483 differs from pull request most recent head 129deee. Consider uploading reports for the commit 129deee to get more accurate results

@@            Coverage Diff             @@
##           master    #7246      +/-   ##
==========================================
+ Coverage   91.42%   91.50%   +0.08%     
==========================================
  Files         446      446              
  Lines       24827    24862      +35     
==========================================
+ Hits        22697    22750      +53     
+ Misses       2130     2112      -18     
Impacted Files Coverage Δ
torch_geometric/experimental.py 98.46% <100.00%> (+56.03%) ⬆️
torch_geometric/utils/to_dense_batch.py 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@piotrchmiel piotrchmiel force-pushed the piotrc/disable_dynamic_shapes branch from f98c31a to 3c1e5bb Compare April 27, 2023 14:04
CHANGELOG.md Outdated Show resolved Hide resolved
torch_geometric/utils/to_dense_batch.py Outdated Show resolved Hide resolved
@rusty1s rusty1s changed the title Introduce "disable_dynamic_shapes" experimental flag. Adding its use into to_dense_batch function. Introduce disable_dynamic_shapes experimental flag; adding its use into to_dense_batch function Apr 28, 2023
@piotrchmiel piotrchmiel requested a review from wsad1 as a code owner May 16, 2023 09:23
@piotrchmiel piotrchmiel force-pushed the piotrc/disable_dynamic_shapes branch from a92cb50 to da67240 Compare May 16, 2023 09:31
@piotrchmiel piotrchmiel requested a review from rusty1s May 16, 2023 09:31
@piotrchmiel piotrchmiel force-pushed the piotrc/disable_dynamic_shapes branch from da67240 to 0a048d8 Compare May 16, 2023 09:33
into to_dense_batch function.

There are devices that do not support dynamic shapes - (compiling and
optimizing only static graphs). The ability to set and read the
"disable_dynamic_shapes" flag allows implementors to provide static
shape-friendly implementations and report user-friendly messages if it
is impossible to avoid using dynamic shapes.
@piotrchmiel piotrchmiel force-pushed the piotrc/disable_dynamic_shapes branch from 0a048d8 to b03d33e Compare May 18, 2023 08:43
Copy link
Member

@rusty1s rusty1s left a comment

Choose a reason for hiding this comment

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

Looks beautiful, thank you!

@rusty1s rusty1s added the utils label Jun 4, 2023
@rusty1s rusty1s enabled auto-merge (squash) June 4, 2023 10:31
@rusty1s rusty1s disabled auto-merge June 4, 2023 11:00
@rusty1s rusty1s merged commit d90842d into pyg-team:master Jun 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants