Skip to content

Clarify objective signs in getting started guide - #5284

Closed
saitcakmak wants to merge 1 commit into
facebook:mainfrom
saitcakmak:export-D119166605
Closed

saitcakmak wants to merge 1 commit into
facebook:mainfrom
saitcakmak:export-D119166605

Conversation

@saitcakmak

Copy link
Copy Markdown
Contributor

Summary:
Clarify that objective-expression signs control optimization direction while raw_data should contain the original observed metric values. Distinguish the negative sign intrinsic to Hartmann6 from the negative objective sign used to minimize it.

Resolves #5283.

Differential Revision: D119166605

Summary:
Clarify that objective-expression signs control optimization direction while `raw_data` should contain the original observed metric values. Distinguish the negative sign intrinsic to Hartmann6 from the negative objective sign used to minimize it.

Resolves facebook#5283.

Differential Revision: D119166605
@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Sep 8, 2026
@meta-codesync

meta-codesync Bot commented Sep 8, 2026

Copy link
Copy Markdown

@saitcakmak has exported this pull request. If you are a Meta employee, you can view the originating Diff in D119166605.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.57%. Comparing base (778e22f) to head (58a5c59).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5284      +/-   ##
==========================================
- Coverage   96.57%   96.57%   -0.01%     
==========================================
  Files         621      621              
  Lines       70911    70911              
==========================================
- Hits        68485    68480       -5     
- Misses       2426     2431       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@meta-codesync meta-codesync Bot closed this in 02c0757 Sep 8, 2026
@meta-codesync meta-codesync Bot added the Merged label Sep 8, 2026
@meta-codesync

meta-codesync Bot commented Sep 8, 2026

Copy link
Copy Markdown

This pull request has been merged in 02c0757.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. Merged meta-exported

Projects

None yet

2 participants