Skip to content

Commit

Permalink
Remove ranks-cpus warning as can be inaccurate (#66)
Browse files Browse the repository at this point in the history
The warning is misleading and causes users to run simulation with more
resources than needed. Since the magic cutoff is neither a lower bound,
nor an upper bound (by many orders of magnitude), we should simply not
print anything.

For cases where resource consumption is hard to estimate we encourage
user to try the new dry-run
  • Loading branch information
1uc authored Oct 25, 2023
1 parent 5268c85 commit 726aaab
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions neurodamus/node.py
Original file line number Diff line number Diff line change
Expand Up @@ -369,13 +369,6 @@ def compute_load_balance(self):
# Info about the cells to be distributed
target_spec = TargetSpec(circuit.CircuitTarget)
target = self.target_manager.get_target(target_spec)
cell_count = None

if not target.is_void():
cell_count = target.gid_count()
if cell_count > 100 * MPI.size:
logging.warning("Your simulation has a very high count of cells per CPU. "
"Please consider launching it in a larger MPI cluster")

# Check / set load balance mode
lb_mode = LoadBalance.select_lb_mode(SimConfig, self._run_conf, target)
Expand Down

0 comments on commit 726aaab

Please sign in to comment.