Skip to content

Conversation

@RolandMinrui
Copy link
Contributor

@RolandMinrui RolandMinrui commented Jun 27, 2025

Description

Motivation and Context

How Has This Been Tested?

  • If you are adding a new feature, test on your own test scripts.

Screenshots of Test Results (if appropriate):

  1. Your own tests:

Types of changes

  • Fix bugs
  • Add new feature
  • Update documentation

📚 Documentation preview 📚: https://RDAgent--995.org.readthedocs.build/en/995/

@RolandMinrui RolandMinrui marked this pull request as draft June 27, 2025 06:51
@RolandMinrui RolandMinrui changed the title init commit fix: add spec for hyperparameters in task design and coder Jun 27, 2025
@RolandMinrui RolandMinrui removed the request for review from TPLin22 June 27, 2025 08:33
@you-n-g you-n-g force-pushed the minrui/fix_hyperparameter_problems branch 2 times, most recently from 2c7fa6e to 3619c95 Compare June 29, 2025 09:11
Xu and others added 3 commits July 3, 2025 05:07
…p and use sample datasets (2)add some generation tops in coding (3)add evaluation guidelines in evaluation (4)polish the json schema and description
{% include "scenarios.data_science.share:guidelines.refine" %}
# Refinement Specification
## Hypothesis: {{ hypothesis.hypothesis }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we can discuss this.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At last we think we should implement this in runner.

def gen(self, trace: DSTrace) -> DSExperiment:
# Step 0: Prepare
pipeline = DS_RD_SETTING.coder_on_whole_pipeline
component_desc = T("scenarios.data_science.share:component_description_in_pipeline").r()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

use include to simplify it.

You might receive exploratory data analysis (EDA) details about the source data. Do not use this EDA information to create assertions or raise errors. We might generate sample data for quick coding (so your code may run on sample data which is part of the full-size data), but remember that the EDA details are based on the full-size data.
draft: |-
TODO
refine: |-
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

low hanging fruit

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't refine part now

return DSRefineExpGen(scen=self.scen).gen(trace=trace)

# Propose
if DS_RD_SETTING.proposal_version == "v1":
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hope to remove proposal_version in this version.

{% include "scenarios.data_science.share:guidelines.refine" %}
# Refinement Specification
## Hypothesis: {{ hypothesis.hypothesis }}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

At last we think we should implement this in runner.

@@ -0,0 +1,105 @@
import json
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need this now.

You might receive exploratory data analysis (EDA) details about the source data. Do not use this EDA information to create assertions or raise errors. We might generate sample data for quick coding (so your code may run on sample data which is part of the full-size data), but remember that the EDA details are based on the full-size data.
draft: |-
TODO
refine: |-
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't refine part now

Hoder-zyf added a commit that referenced this pull request Jul 4, 2025
@peteryang1 peteryang1 marked this pull request as ready for review July 8, 2025 07:11
@peteryang1 peteryang1 merged commit 10246fd into main Jul 8, 2025
9 checks passed
@peteryang1 peteryang1 deleted the minrui/fix_hyperparameter_problems branch July 8, 2025 07:22
@peteryang1 peteryang1 restored the minrui/fix_hyperparameter_problems branch July 8, 2025 10:07
licong01-cloud pushed a commit to licong01-cloud/RD-Agent that referenced this pull request Dec 13, 2025
…#995)

* init commit

* remove the 5-fold spec from prompts

* refine the hyperparameter specification

* do not sample data

* a small spelling issue

* refine prompt to avoid submission cheating

* do not sample data

* simplify code

* refine the coder evaluator prompt

* refine wording

* remove runtime from proposal

* refine wording

* refine prompt

* add gpu info in runtime_info.py

* modify the spec

* add router and add refinement exp gen

* fix prompt bug

* use rule-based logic for router

* complete the prompt

* fix circular import bug

* fix bug

* make refine_decision optional

* update pipeline prompts: (1) add scenary: in an iterative cooding loop and use sample datasets (2)add some generation tops in coding (3)add evaluation guidelines in evaluation (4)polish the json schema and description

* fix a small bug

* fix a small bug

* rdagent/scenarios/data_science/loop.py back to the original version

* refactor: replace _get_exp_gen with default_exp_gen for exp generation

* import

* refactor: make the __init__ back to main

* fix small bugs

* fix bugs for proposal_version

* move refine into runner

* check early stop

* EDA improvement & coder classes number

* fix CI

* slightly refine the prompt

* remove rule_base_eval and remove useless prompt

---------

Co-authored-by: Xu <[email protected]>
Co-authored-by: TPLin22 <[email protected]>
Co-authored-by: amstrongzyf <[email protected]>
Co-authored-by: Xu Yang <[email protected]>
Co-authored-by: Xu Yang <[email protected]>
Co-authored-by: Young <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants