Skip to content

Remove additional layer of aggregation from logging#435

Closed
MisterTea wants to merge 1 commit intofacebookresearch:masterfrom
MisterTea:export-D27436575
Closed

Remove additional layer of aggregation from logging#435
MisterTea wants to merge 1 commit intofacebookresearch:masterfrom
MisterTea:export-D27436575

Conversation

@MisterTea
Copy link
Contributor

Differential Revision: D27436575

@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D27436575

…#435)

Summary: Pull Request resolved: facebookresearch#435

Reviewed By: DavidV17

Differential Revision: D27436575

fbshipit-source-id: 7d6304afc825e297efc66c4986b9a1bdb67c42f4
@facebook-github-bot
Copy link

This pull request was exported from Phabricator. Differential Revision: D27436575

@codecov-io
Copy link

Codecov Report

Merging #435 (44de463) into master (43158c1) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #435      +/-   ##
==========================================
+ Coverage   82.32%   82.36%   +0.04%     
==========================================
  Files         312      312              
  Lines       17876    17850      -26     
  Branches       42       42              
==========================================
- Hits        14717    14703      -14     
+ Misses       3135     3123      -12     
  Partials       24       24              
Impacted Files Coverage Δ
reagent/workflow/reporters/reporter_base.py 93.02% <ø> (+13.57%) ⬆️
reagent/training/reagent_lightning_module.py 83.33% <100.00%> (+0.68%) ⬆️
...eagent/workflow/reporters/discrete_crr_reporter.py 66.66% <100.00%> (ø)
...eagent/workflow/reporters/discrete_dqn_reporter.py 100.00% <100.00%> (ø)

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 43158c1...44de463. Read the comment docs.

@facebook-github-bot
Copy link

This pull request has been merged in 0e5fc53.

xuruiyang pushed a commit that referenced this pull request Sep 20, 2025
Summary: Pull Request resolved: #435

Reviewed By: DavidV17

Differential Revision: D27436575

fbshipit-source-id: cbcc0439fca2e0258a1aac5ceff3ae1bb29258c2
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.

3 participants