Skip to content

Update ARS comments#528

Closed
czxttkl wants to merge 1 commit intofacebookresearch:masterfrom
czxttkl:export-D30343990
Closed

Update ARS comments#528
czxttkl wants to merge 1 commit intofacebookresearch:masterfrom
czxttkl:export-D30343990

Conversation

@czxttkl
Copy link
Contributor

@czxttkl czxttkl commented Aug 22, 2021

Summary: Simplify the comments

Differential Revision: D30343990

Summary: Simplify the comments

Differential Revision: D30343990

fbshipit-source-id: 6476025adfa31a10f4966d2777e5d5a15e1c2bea
@facebook-github-bot
Copy link

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

@codecov-commenter
Copy link

codecov-commenter commented Aug 22, 2021

Codecov Report

Merging #528 (6f9001e) into master (9190030) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #528   +/-   ##
=======================================
  Coverage   85.09%   85.09%           
=======================================
  Files         331      331           
  Lines       19464    19464           
  Branches       44       44           
=======================================
  Hits        16562    16562           
  Misses       2876     2876           
  Partials       26       26           
Impacted Files Coverage Δ
reagent/training/gradient_free/ars_util.py 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 9190030...6f9001e. Read the comment docs.

@facebook-github-bot
Copy link

This pull request has been merged in 81eda73.

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

Simplify the comments

Reviewed By: xuruiyang

Differential Revision: D30343990

fbshipit-source-id: 8b3c4172a4af9e01c27e8e511486bed68c1032b5
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