Skip to content
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

Jialei gh219 improve consistency check between gpu vs cpu computation for general qp ei #267

Conversation

jialeiwang
Copy link
Contributor

********* PEOPLE *************
Primary reviewer: @suntzu86

Reviewers: @sc932 @ypwais @yaoster

********* DESCRIPTION **************
Branch Name: jialei_gh219_improve_consistency_check_between_gpu_vs_cpu_computation_for_general_qp_EI
Ticket(s)/Issue(s): Closes #219

********* TESTING DONE *************
make test-no-tox: total test time 27.40s
cpplint.py to check style

jialeiwang and others added 30 commits June 11, 2014 14:49
…ct_GPU_implementation_to_the_new_qp_EI_interface
…ct_GPU_implementation_to_the_new_qp_EI_interface
… style improved, more doc string to add in the next commit
…ct_GPU_implementation_to_the_new_qp_EI_interface
…ct_GPU_implementation_to_the_new_qp_EI_interface
…ct_GPU_implementation_to_the_new_qp_EI_interface
…ve_consistency_check_between_gpu_vs_cpu_computation_for_general_qp_EI

Conflicts:
	moe/optimal_learning/cpp/CMakeLists.txt
…ct_GPU_implementation_to_the_new_qp_EI_interface
…I_interface' of github.com:sc932/MOE into jialei_gh219_improve_consistency_check_between_gpu_vs_cpu_computation_for_general_qp_EI

Conflicts:
	moe/optimal_learning/cpp/gpp_expected_improvement_gpu.cpp
	moe/optimal_learning/cpp/gpp_expected_improvement_gpu.hpp
	moe/optimal_learning/cpp/gpu/gpp_cuda_math.cu
	moe/optimal_learning/cpp/gpu/gpp_cuda_math.hpp
…ve_consistency_check_between_gpu_vs_cpu_computation_for_general_qp_EI
@suntzu86 suntzu86 closed this Jul 29, 2014
@suntzu86 suntzu86 reopened this Jul 29, 2014
@suntzu86
Copy link
Contributor

Sorry! I clicked the wrong button :(

}

OptimalLearningCudaException::OptimalLearningCudaException(const CudaError& _err)
: OptimalLearningException(_err.file_and_line_info, _err.func_info, cudaGetErrorString(_err.err)) {}
Copy link
Contributor

Choose a reason for hiding this comment

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

closing brace goes on its own line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@suntzu86
Copy link
Contributor

mostly documentation and one macro fix. should be done after this!

…e_consistency_check_between_gpu_vs_cpu_computation_for_general_qp_EI

OptimalLearningCudaException::OptimalLearningCudaException(const CudaError& error)
: OptimalLearningException(error.file_and_line_info, error.func_info, cudaGetErrorString(error.err)) {
}
Copy link
Contributor

Choose a reason for hiding this comment

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

no leading whitespace before the }

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@suntzu86
Copy link
Contributor

suntzu86 commented Aug 1, 2014

two whitespace changes

then 🚢 congrats & thanks!! :D

…e_consistency_check_between_gpu_vs_cpu_computation_for_general_qp_EI
jialeiwang added a commit that referenced this pull request Aug 1, 2014
…eck_between_gpu_vs_cpu_computation_for_general_qp_EI

Jialei gh219 improve consistency check between gpu vs cpu computation for general qp ei
@jialeiwang jialeiwang merged commit 1672b4c into master Aug 1, 2014
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.

Improve consistency check between gpu vs cpu computation for general q,p-EI
2 participants