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

feat: Simplify the usage of NeptuneLogger #38

Merged
merged 2 commits into from
Nov 21, 2022

Conversation

Higgcz
Copy link
Contributor

@Higgcz Higgcz commented Nov 17, 2022

Let NeptuneLogger initialize a new Neptune Run if the user doesn't specify it.

Waiting for #37

@Higgcz Higgcz self-assigned this Nov 17, 2022
@Higgcz Higgcz added the enhancement New feature or request label Nov 17, 2022
@Higgcz Higgcz added this to the 0.4.0 milestone Nov 17, 2022
@Higgcz Higgcz changed the title Simplify the usage of NeptuneLogger feat: Simplify the usage of NeptuneLogger Nov 17, 2022
@codecov-commenter
Copy link

codecov-commenter commented Nov 19, 2022

Codecov Report

Merging #38 (0258204) into master (eaac131) will decrease coverage by 0.00%.
The diff coverage is 28.57%.

❗ Current head 0258204 differs from pull request most recent head 5f01913. Consider uploading reports for the commit 5f01913 to get more accurate results

@@            Coverage Diff             @@
##           master      #38      +/-   ##
==========================================
- Coverage   52.18%   52.18%   -0.01%     
==========================================
  Files          43       43              
  Lines        6241     6245       +4     
==========================================
+ Hits         3257     3259       +2     
- Misses       2984     2986       +2     
Impacted Files Coverage Δ
src/evotorch/logging.py 29.09% <28.57%> (+0.22%) ⬆️
src/evotorch/neuroevolution/net/vecrl.py 40.06% <0.00%> (ø)
src/evotorch/algorithms/distributed/gaussian.py 19.21% <0.00%> (ø)
src/evotorch/neuroevolution/net/statefulmodule.py 36.36% <0.00%> (+1.58%) ⬆️

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

@Higgcz Higgcz force-pushed the feature/improve-neptune-logger branch 3 times, most recently from e30a570 to e8408c1 Compare November 19, 2022 21:56
@Higgcz Higgcz force-pushed the feature/improve-neptune-logger branch from e8408c1 to 52a89df Compare November 21, 2022 14:11
@Higgcz Higgcz marked this pull request as ready for review November 21, 2022 14:24
@Higgcz Higgcz merged commit 7220d5b into master Nov 21, 2022
@Higgcz Higgcz deleted the feature/improve-neptune-logger branch November 21, 2022 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants