Remove n_jobs from STree
This commit is contained in:
parent
f088df14fd
commit
d3cb580387
@ -61,7 +61,6 @@ namespace pywrap {
|
|||||||
void PyWrap::clean(const clfId_t id)
|
void PyWrap::clean(const clfId_t id)
|
||||||
{
|
{
|
||||||
// Remove Python interpreter if no more modules imported left
|
// Remove Python interpreter if no more modules imported left
|
||||||
// std::cout << "*Cleaning module " << id << std::endl;
|
|
||||||
std::lock_guard<std::mutex> lock(mutex);
|
std::lock_guard<std::mutex> lock(mutex);
|
||||||
auto result = moduleClassMap.find(id);
|
auto result = moduleClassMap.find(id);
|
||||||
if (result == moduleClassMap.end()) {
|
if (result == moduleClassMap.end()) {
|
||||||
@ -75,11 +74,11 @@ namespace pywrap {
|
|||||||
PyErr_Print();
|
PyErr_Print();
|
||||||
errorAbort("Error cleaning module ");
|
errorAbort("Error cleaning module ");
|
||||||
}
|
}
|
||||||
|
// With boost you can't remove the interpreter
|
||||||
|
// https://www.boost.org/doc/libs/1_83_0/libs/python/doc/html/tutorial/tutorial/embedding.html#tutorial.embedding.getting_started
|
||||||
// if (moduleClassMap.empty()) {
|
// if (moduleClassMap.empty()) {
|
||||||
// RemoveInstance();
|
// RemoveInstance();
|
||||||
// std::cout << "*Python interpreter cleaned" << std::endl;
|
|
||||||
// }
|
// }
|
||||||
// std::cout << "*Module " << id << " cleaned" << std::endl;
|
|
||||||
}
|
}
|
||||||
void PyWrap::errorAbort(const std::string& message)
|
void PyWrap::errorAbort(const std::string& message)
|
||||||
{
|
{
|
||||||
|
@ -8,7 +8,7 @@ namespace pywrap {
|
|||||||
void STree::setHyperparameters(nlohmann::json& hyperparameters)
|
void STree::setHyperparameters(nlohmann::json& hyperparameters)
|
||||||
{
|
{
|
||||||
// Check if hyperparameters are valid
|
// Check if hyperparameters are valid
|
||||||
const std::vector<std::string> validKeys = { "C", "n_jobs", "kernel", "max_iter", "max_depth", "random_state", "multiclass_strategy" };
|
const std::vector<std::string> validKeys = { "C", "kernel", "max_iter", "max_depth", "random_state", "multiclass_strategy" };
|
||||||
checkHyperparameters(validKeys, hyperparameters);
|
checkHyperparameters(validKeys, hyperparameters);
|
||||||
this->hyperparameters = hyperparameters;
|
this->hyperparameters = hyperparameters;
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user