@@ -38,10 +38,6 @@ pub struct ModelConfig {
3838 pub ( crate ) base_url : Option < String > ,
3939 #[ builder( default = "None" , setter( custom) ) ]
4040 pub ( crate ) provider : Option < String > ,
41- #[ builder( default = "None" ) ]
42- pub ( crate ) temperature : Option < f32 > ,
43- #[ builder( default = "None" ) ]
44- pub ( crate ) max_output_tokens : Option < usize > ,
4541
4642 #[ builder( setter( custom) ) ]
4743 pub ( crate ) name : ModelName ,
@@ -86,10 +82,6 @@ pub struct Config {
8682 pub ( crate ) base_url : Option < String > ,
8783 #[ builder( default = "DEFAULT_PROVIDER.to_string()" , setter( custom) ) ]
8884 pub ( crate ) provider : String ,
89- #[ builder( default = "0.8" ) ]
90- pub ( crate ) temperature : f32 ,
91- #[ builder( default = "1024" ) ]
92- pub ( crate ) max_output_tokens : usize ,
9385
9486 #[ builder( default = "RoutingMode::Random" ) ]
9587 pub ( crate ) routing_mode : RoutingMode ,
@@ -131,13 +123,6 @@ impl Config {
131123 if model. provider . is_none ( ) {
132124 model. provider = Some ( self . provider . clone ( ) ) ;
133125 }
134-
135- if model. temperature . is_none ( ) {
136- model. temperature = Some ( self . temperature ) ;
137- }
138- if model. max_output_tokens . is_none ( ) {
139- model. max_output_tokens = Some ( self . max_output_tokens ) ;
140- }
141126 }
142127 self
143128 }
@@ -176,24 +161,6 @@ impl ConfigBuilder {
176161 ) ) ;
177162 }
178163
179- if let Some ( max_output_tokens) = model. max_output_tokens {
180- if max_output_tokens <= 0 {
181- return Err ( format ! (
182- "Model '{}' max_output_tokens must be positive." ,
183- model. name
184- ) ) ;
185- }
186- }
187-
188- if let Some ( temperature) = model. temperature {
189- if temperature < 0.0 || temperature > 1.0 {
190- return Err ( format ! (
191- "Model '{}' temperature must be between 0.0 and 1.0." ,
192- model. name
193- ) ) ;
194- }
195- }
196-
197164 // check the existence of API key in environment variables
198165 if let Some ( provider) = & model. provider {
199166 let env_var = format ! ( "{}_API_KEY" , provider) ;
@@ -251,20 +218,10 @@ mod tests {
251218 assert ! ( valid_simplest_models_cfg. is_ok( ) ) ;
252219 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . provider == DEFAULT_PROVIDER ) ;
253220 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . base_url == None ) ;
254- assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . temperature == 0.8 ) ;
255- assert ! (
256- valid_simplest_models_cfg
257- . as_ref( )
258- . unwrap( )
259- . max_output_tokens
260- == 1024
261- ) ;
262221 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . routing_mode == RoutingMode :: Random ) ;
263222 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . models. len( ) == 1 ) ;
264223 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . base_url == None ) ;
265224 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . provider == None ) ;
266- assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . temperature == None ) ;
267- assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . max_output_tokens == None ) ;
268225 assert ! ( valid_simplest_models_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . weight == -1 ) ;
269226
270227 // case 2:
@@ -299,7 +256,6 @@ mod tests {
299256 // AMRS_API_KEY is set in .env.test already.
300257 let valid_cfg_with_customized_provider = Config :: builder ( )
301258 . base_url ( "http://example.ai" )
302- . max_output_tokens ( 2048 )
303259 . model (
304260 ModelConfig :: builder ( )
305261 . name ( "custom-model" )
@@ -325,8 +281,6 @@ mod tests {
325281 from_filename ( ".env.test" ) . ok ( ) ;
326282
327283 let mut valid_cfg = Config :: builder ( )
328- . temperature ( 0.5 )
329- . max_output_tokens ( 1500 )
330284 . model (
331285 ModelConfig :: builder ( )
332286 . name ( "model-1" . to_string ( ) )
@@ -338,8 +292,6 @@ mod tests {
338292
339293 assert ! ( valid_cfg. is_ok( ) ) ;
340294 assert ! ( valid_cfg. as_ref( ) . unwrap( ) . models. len( ) == 1 ) ;
341- assert ! ( valid_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . temperature == Some ( 0.5 ) ) ;
342- assert ! ( valid_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . max_output_tokens == Some ( 1500 ) ) ;
343295 assert ! ( valid_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . provider == Some ( "OPENAI" . to_string( ) ) ) ;
344296 assert ! (
345297 valid_cfg. as_ref( ) . unwrap( ) . models[ 0 ] . base_url
0 commit comments