-
Notifications
You must be signed in to change notification settings - Fork 20
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
Urban opt50 #615
Urban opt50 #615
Conversation
cleanup
…oint convert create_data_point_filename: 'create_data_point.rb' to create_and_run_datapoint_filename: 'create_and_run_datapoint_uniquegroups.R'
add check for it in nsga_nrel.rb add boundary check to morris.R add create_and_run_datapoint_urbanopt.R
add urbanopt flag to DataPoint and Analysis add --UO run option to api_create_datapoint.rb
…ck/receipt move UO logs to function call
add uo_debug branch for UO gems add SEB_LHS test
Nice! I'll take a look at this early next week! |
@brianlball Can you merge in the latest develop branch? We enabled Github actions and disabled Travis. You can add all the algo tests in now as we have more run time allotted. |
add faster urbanopt tests
@tijcolem merged in develop, green. this still ran the tests in this dir, correct? https://github.com/NREL/OpenStudio-server/tree/develop/server/spec/features |
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.
Looks great! I'll merge as soon as the tests pass.
Looks like the urbanopt tests run within docker! Should be good to go. I have one more tweak that I'm going to commit. |
Damn thats a beautiful sight.... |
I have one last commit that is nothing more than a couple typos... then I'll merge down. |
No description provided.