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.

Sign up for GitHub

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

Jump to bottom

[TypeHints] Random Transforms #5714

Merged
merged 10 commits into from
Oct 15, 2022
Merged

[TypeHints] Random Transforms #5714

merged 10 commits into from
Oct 15, 2022

Conversation

Padarn
Copy link
Contributor

@Padarn Padarn commented Oct 15, 2022

No description provided.

@Padarn Padarn self-assigned this Oct 15, 2022
@codecov
Copy link

codecov bot commented Oct 15, 2022 •
edited
Loading

Codecov Report

Merging #5714 (2465288) into master (52bf94b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 2465288 differs from pull request most recent head 846b9cb. Consider uploading reports for the commit 846b9cb to get more accurate results

@@           Coverage Diff           @@
##           master    #5714   +/-   ##
=======================================
  Coverage   84.01%   84.02%           
=======================================
  Files         349      349           
  Lines       19281    19288    +7     
=======================================
+ Hits        16199    16206    +7     
  Misses       3082     3082
Impacted Files Coverage Δ
torch_geometric/transforms/random_flip.py 100.00% <100.00%> (ø)
torch_geometric/transforms/random_jitter.py 100.00% <100.00%> (ø)
torch_geometric/transforms/random_rotate.py 100.00% <100.00%> (ø)
torch_geometric/transforms/random_scale.py 100.00% <100.00%> (ø)

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

@rusty1s rusty1s changed the title [TypeHints] Random Tansforms [TypeHints] Random Transforms Oct 15, 2022
rusty1s and others added 6 commits October 15, 2022 12:19
@rusty1s @wsad1
Update torch_geometric/transforms/random_flip.py
797182b 
Co-authored-by: Jinu Sunil <jinu.sunil@gmail.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
ad09f10 
for more information, see https://pre-commit.ci
@rusty1s @wsad1
Update torch_geometric/transforms/random_rotate.py
8aa67fb 
Co-authored-by: Jinu Sunil <jinu.sunil@gmail.com>
@rusty1s @wsad1
Update torch_geometric/transforms/random_rotate.py
2465288 
Co-authored-by: Jinu Sunil <jinu.sunil@gmail.com>
@rusty1s @wsad1
Update torch_geometric/transforms/random_scale.py
d10ffd6 
Co-authored-by: Jinu Sunil <jinu.sunil@gmail.com>
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
ce81711 
for more information, see https://pre-commit.ci
rusty1s
rusty1s approved these changes Oct 15, 2022
View reviewed changes
torch_geometric/transforms/random_flip.py Outdated Show resolved Hide resolved
torch_geometric/transforms/random_jitter.py Outdated Show resolved Hide resolved
@rusty1s rusty1s enabled auto-merge (squash) October 15, 2022 10:22
@rusty1s rusty1s mentioned this pull request Oct 15, 2022
Closed
@rusty1s rusty1s merged commit 489d5f3 into master Oct 15, 2022
@rusty1s rusty1s deleted the random_transform_type_hints branch October 15, 2022 10:25
JakubPietrakIntel pushed a commit to JakubPietrakIntel/pytorch_geometric that referenced this pull request Nov 25, 2022
@Padarn @rusty1s
@wsad1 @pre-commit-ci @JakubPietrakIntel
[TypeHints] Random Transforms (pyg-team#5714)
d96c49c 
Co-authored-by: Matthias Fey <matthias.fey@tu-dortmund.de>
Co-authored-by: Jinu Sunil <jinu.sunil@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wsad1 wsad1 wsad1 approved these changes

@rusty1s rusty1s rusty1s approved these changes

Assignees

@Padarn Padarn

Labels
0 - Priority P0 transform type hints
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Padarn @rusty1s @wsad1