-
Notifications
You must be signed in to change notification settings - Fork 73
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
Add a function to predict a value from a csv file. Part2 #260
Conversation
@gieljnssns congrats on pulling all these merges off. There really has been a lot changed since you initially released this PR. I'm impressed. |
mlregressor, add web buttons for mlregressor, add some suggestions
Hi @gieljnssns, is this ready to merge? Other than the needed conflict solve? |
optim_conf["def_end_timestep"] = runtimeparams["def_end_timestep"] | ||
if "treat_def_as_semi_cont" in runtimeparams.keys(): | ||
optim_conf["treat_def_as_semi_cont"] = [ | ||
eval(str(k).capitalize()) |
Check failure
Code scanning / CodeQL
Code injection Critical
user-provided value
] | ||
if "set_def_constant" in runtimeparams.keys(): | ||
optim_conf["set_def_constant"] = [ | ||
eval(str(k).capitalize()) for k in runtimeparams["set_def_constant"] |
Check failure
Code scanning / CodeQL
Code injection Critical
user-provided value
perform_backtest = eval(str(runtimeparams['perform_backtest']).capitalize()) | ||
params['passed_data']['perform_backtest'] = perform_backtest | ||
if 'model_predict_publish' not in runtimeparams.keys(): | ||
perform_backtest = eval(str(runtimeparams["perform_backtest"]).capitalize()) |
Check failure
Code scanning / CodeQL
Code injection Critical
user-provided value
This code execution depends on a
user-provided value
params['passed_data']['model_predict_publish'] = model_predict_publish | ||
if 'model_predict_entity_id' not in runtimeparams.keys(): | ||
model_predict_publish = eval( | ||
str(runtimeparams["model_predict_publish"]).capitalize() |
Check failure
Code scanning / CodeQL
Code injection Critical
user-provided value
This code execution depends on a
user-provided value
Yes, I think so |
#147