search

radical-collaboration / CyberManufacturing

CDSE Multi-scale CI Project
1 stars 0 forks source link

Feature/log messages #42

Closed iparask closed 6 years ago

iparask commented 7 years ago

These changes added debug and info logging messages in the executor script. Also they added a reporter. This means that the terminal will look like:

================================================================================
 Starting Two-way DEM PBM simulation
================================================================================

                                                                              ok
create pilot manager                                                          ok
create unit manager                                                           ok
submit 1 pilot(s)
        .                                                                     ok
add 1 pilot(s)                                                                ok

--------------------------------------------------------------------------------
Starting DEM simulation

submit 1 unit(s)
        .                                                                     ok
wait for 1 unit(s)
        +                                                                     ok
submit 1 unit(s)
        .                                                                     ok
wait for 1 unit(s)
        +                                                                     ok

--------------------------------------------------------------------------------
Canceling DEM simulation

wait for 1 unit(s)
        -                                                                     ok

--------------------------------------------------------------------------------
Starting PBM simulation

submit 1 unit(s)
        .                                                                     ok
wait for 1 unit(s)
        +                                                                     ok
submit 1 unit(s)
        .                                                                     ok
wait for 1 unit(s)
        +
================================================================================
 Simulations Finished.
================================================================================

close unit manager                                                            ok
close pilot manager                                                           ok

The purpose of this pull request is two fold.

  1. To make sure that bugs which are fixed in devel are not being reintroduced
  2. Make sure RADICAL-Pilot API is not miss used.
iparask commented 7 years ago

Please do not merge unless both reviewers agree with the changes and code

csampat commented 6 years ago

Conflicts have been resolved. I guess we can merge now

iparask commented 6 years ago

We are waiting for @vivek-bala review