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

dummy pr: logger pl callback #3

Merged
merged 2 commits into from
Apr 22, 2022
Merged

Conversation

aniketmaurya
Copy link
Owner

cc: @Borda

@codecov
Copy link

codecov bot commented Apr 22, 2022

Codecov Report

Merging #3 (1bb9618) into implement_module (39305a5) will decrease coverage by 0.04%.
The diff coverage is 61.29%.

@@                 Coverage Diff                  @@
##           implement_module       #3      +/-   ##
====================================================
- Coverage             82.64%   82.60%   -0.05%     
====================================================
  Files                   313      313              
  Lines                 16366    16397      +31     
====================================================
+ Hits                  13525    13544      +19     
- Misses                 2841     2853      +12     
Impacted Files Coverage Δ
torch_geometric/graphgym/logger.py 76.40% <61.29%> (-3.19%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 39305a5...1bb9618. Read the comment docs.

@aniketmaurya aniketmaurya merged commit 985937c into implement_module Apr 22, 2022
@aniketmaurya aniketmaurya deleted the logger_callback branch April 22, 2022 15:23
aniketmaurya added a commit that referenced this pull request May 14, 2022
…4531)

* implement model steps and configure_optimizer

* fix imports

* dummy pr: logger pl callback (#3)

implement logger pl callback

* update logger

* add test

* fix test

* fix

* fixes

* add test

* update test

* test configure optimizer

* test configure optimizer

* commit suggested change

Co-authored-by: Jirka Borovec <[email protected]>

* remove redundant parameters

* add typing

* apply pr suggestions

* apply suggestions

* apply suggestions

* apply suggestions

* test logger

* remove graphgym from minimal installation

* fix

* fix minimal test

* changelog

* update

* update

* update

* lint

Co-authored-by: Jirka Borovec <[email protected]>
Co-authored-by: Matthias Fey <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants