Skip to content

Commit

Permalink
warnings--
Browse files Browse the repository at this point in the history
  • Loading branch information
henricasanova committed Oct 17, 2023
1 parent 231da96 commit 986040f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/wrench/services/compute/batch/BatchComputeService.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -810,7 +810,7 @@ namespace wrench {

if ((requested_hosts > this->available_nodes_to_cores.size()) or
(requested_num_cores_per_host >
this->available_nodes_to_cores.begin()->first->get_core_count()) or
(unsigned long)this->available_nodes_to_cores.begin()->first->get_core_count()) or
(required_ram_per_host >
S4U_Simulation::getHostMemoryCapacity(this->available_nodes_to_cores.begin()->first))) {
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -254,7 +254,7 @@ namespace wrench {
if (num_nodes > cs->available_nodes_to_cores.size()) {
throw std::runtime_error("CONSERVATIVE_BFBatchScheduler::scheduleOnHosts(): Asking for too many hosts");
}
if (cores_per_node > cs->available_nodes_to_cores.begin()->first->get_core_count()) {
if (cores_per_node > (unsigned long) cs->available_nodes_to_cores.begin()->first->get_core_count()) {
throw std::runtime_error("CONSERVATIVE_BFBatchScheduler::scheduleOnHosts(): Asking for too many cores per host (asking for " +
std::to_string(cores_per_node) + " but hosts have " +
std::to_string(cs->available_nodes_to_cores.begin()->first->get_core_count()) + "cores)");
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ namespace wrench {
if (num_nodes > cs->available_nodes_to_cores.size()) {
throw std::runtime_error("FCFSBatchScheduler::scheduleOnHosts(): Asking for too many hosts");
}
if (cores_per_node > cs->available_nodes_to_cores.begin()->first->get_core_count()) {
if (cores_per_node > (unsigned long)cs->available_nodes_to_cores.begin()->first->get_core_count()) {
throw std::runtime_error("FCFSBatchScheduler::scheduleOnHosts(): Asking for too many cores per host");
}

Expand Down

0 comments on commit 986040f

Please sign in to comment.