Thanks Barry, that looks good! Chris
dr. ir. Christiaan Klaij | Senior Researcher | Research & Development MARIN | T +31 317 49 33 44 | mailto:[email protected] | http://www.marin.nl MARIN news: http://www.marin.nl/web/News/News-items/New-correction-procedure-for-shallowwater-effects-in-ship-speed-trials.htm ________________________________________ From: Smith, Barry F. <[email protected]> Sent: Monday, April 09, 2018 11:51 PM To: Klaij, Christiaan Cc: [email protected]; Koos Huijssen Subject: Re: nested timers transfers and reductions Hopefully all these problems are resolved with https://bitbucket.org/petsc/petsc/pull-requests/925/fix-some-missing-functionality-for-nested/diff Barry > On Apr 9, 2018, at 4:37 AM, Klaij, Christiaan <[email protected]> wrote: > > Hi Barry, > > I've put the functions and fortran bindings in the attached > files. In our code, we use them as follows: > > CALL mpi_bcast(value,1,MPI_LOGICAL,iproc,MPI_COMM_WORLD,ier) > IF(ier/=0) CALL tracing_abort('ERROR in bcast logical root !') > > IF (iproc==thisprocess) THEN > ier = PetscAdminSend(1*(nprocesses-1),MPI_LOGICAL) > ELSE > ier = PetscAdminRecv(1,MPI_LOGICAL) > END IF > > I also noted that you changed the treshold from 0.1% to 0.01%, so > I tried to change it as a user with PetscLogSetThreshold, but > this function seems not accessible to users (and there's no > fortran version or manual page). > > Chris > > > dr. ir. Christiaan Klaij | Senior Researcher | Research & Development > MARIN | T +31 317 49 33 44 | mailto:[email protected] | http://www.marin.nl > > MARIN news: > http://www.marin.nl/web/News/News-items/Kom-zaterdag-10-november-naar-de-open-dag-in-Wageningen.htm > > ________________________________________ > From: Smith, Barry F. <[email protected]> > Sent: Saturday, April 07, 2018 5:36 PM > To: Klaij, Christiaan > Cc: [email protected]; Koos Huijssen > Subject: Re: nested timers transfers and reductions > >> On Apr 6, 2018, at 5:22 AM, Klaij, Christiaan <[email protected]> wrote: >> >> Hi Barry, >> >> In our version of the nested timers, we had the functions >> PetscAdminSend, PetscAdminRecv and PetscAdminReduce. So I was >> expecting to find them at the end of the file >> petsc-3.8.4/src/sys/logging/xmllogevent.c but apparently >> you (re)moved them? > > I do not remember why I would have removed these functions. If you can send > a patch that would return them I would gladly try to get them working with > PETSc again. > > Barry > >> >> We basically used these functions to fill the transfer(GiB/s) and >> reductions/s columns of the log just like we use PetscLogFlops to >> fill the compute(Mflops) column for our registered events. >> >> How should I do this with the petsc version of the nested timers? >> >> Chris >> >> >> dr. ir. Christiaan Klaij | Senior Researcher | Research & Development >> MARIN | T +31 317 49 33 44 | mailto:[email protected] | http://www.marin.nl >> >> MARIN news: >> http://www.marin.nl/web/News/News-items/Application-of-potential-flow-methods-to-fast-displacement-ships-at-transcritical-speeds-in-shallow-water-1.htm >> > > <petsc_MPIadminF.F90><petsc_MPIadmin.c>
