-
Notifications
You must be signed in to change notification settings - Fork 0
Dmytro/add ng state format #1007
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
base: main
Are you sure you want to change the base?
Conversation
…_task is called first time
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1007 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 172 172
Lines 8389 8404 +15
=========================================
+ Hits 8389 8404 +15 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
@@ -113,7 +113,7 @@ def create_trace_feedback_v0_task( | |||
raise ValueError("trace_feedback_v0 requires 'task_id' in kwargs") | |||
|
|||
# Get the original task | |||
original_task = get_task(project_name=project_name, task_id=original_task_id) | |||
original_task = get_task(project_name=project_name, task_id=original_task_id, process_ng_state=False) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why does this need process_ng_state=False
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because here we use get_task just to get original_task to create other task, I think we do not need to generate ng_state in this case, we just copy ng_state from original task in format { "seed_id": ... }
No description provided.