search

coin-or / CHiPPS-ALPS

This is the Abstract Library for Parallel Search (ALPS), the abstract base layer of the COIN-OR High Performance Parallel Search framework.
Eclipse Public License 1.0
9 stars 8 forks source link

default clock type #1

Closed kibaekkim closed 5 years ago

kibaekkim commented 7 years ago

The default clock type is CPU. The comment in the header file needs corrected.

tkralphs commented 7 years ago

OK, I will correct this. It's not clear we should even have a parameter for clock type. On Windows, you only really have wall clock time (at least this used to be the case) and on Linux and OS X, I guess you always want CPU time in sequential mode and wall clock in parallel mode (CPU time doesn't have a very nice interpretation there). Perhaps in a shared memory parallel mode, CPU time would make sense for a parallel code, but with MPI, it seems wall time is what one wants to know.

kibaekkim commented 7 years ago

Here is the line corrected in my fork. I can create a pull request for this issue also with #2.

tkralphs commented 5 years ago

Fixed by 4eb7d5ccd4684d3a98b26b95a038ee4c571345a0