plumed2 icon indicating copy to clipboard operation
plumed2 copied to clipboard

Problems with rendering of new manual

Open GiovanniBussi opened this issue 5 years ago • 8 comments

I noticed some problem in the rendering of the manual for 2.7. In particular, these pages seem to be incorrectly interrupted: https://www.plumed.org/doc-v2.7/user-doc/html/_r_e_w_e_i_g_h_t__w_h_a_m.html https://www.plumed.org/doc-v2.7/user-doc/html/trieste-5.html etc (it seems the pattern is that there are problems with the test requires MPI, but I am not sure).

I am actually not able to reproduce the problem on my laptop, so there might be some subtle problem on the Travis-CI configuration. @gtribello any hints about where I should look at?

GiovanniBussi avatar Oct 28 '20 17:10 GiovanniBussi

It is a problem with flushing, I tried to solve it and it worked in my Linux workstation but it does not on Travis

Sent from my iPhone

Il giorno 28 ott 2020, alle ore 18:59, Giovanni [email protected] ha scritto:

 I noticed some problem in the rendering of the manual for 2.7. In particular, these pages seem to be incorrectly interrupted: https://www.plumed.org/doc-v2.7/user-doc/html/_r_e_w_e_i_g_h_t__w_h_a_m.html https://www.plumed.org/doc-v2.7/user-doc/html/trieste-5.html etc (it seems the pattern is that there are problems with the test requires MPI, but I am not sure).

I am actually not able to reproduce the problem on my laptop, so there might be some subtle problem on the Travis-CI configuration. @gtribello any hints about where I should look at?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

carlocamilloni avatar Oct 28 '20 18:10 carlocamilloni

Urka! Is it possible that when you run with mpirun you have multiple replicas and thus they write multiple times on the example html file? This could be fixed making sure that only the first replica writes (checking Intercomm.Get_rank()).

GiovanniBussi avatar Oct 28 '20 18:10 GiovanniBussi

Maybe here: https://github.com/plumed/plumed2/blob/257b4a19700a4e82a4131d0f3ec4758c8b129dcc/src/cltools/GenExample.cpp#L118

One could make it such that other replicas open ofile as "/dev/null".

GiovanniBussi avatar Oct 28 '20 18:10 GiovanniBussi

That is a good point

Sent from my iPhone

Il giorno 28 ott 2020, alle ore 19:17, Giovanni [email protected] ha scritto:

 Maybe here: https://github.com/plumed/plumed2/blob/257b4a19700a4e82a4131d0f3ec4758c8b129dcc/src/cltools/GenExample.cpp#L118

One could make it such that other replicas open ofile as "/dev/null".

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

carlocamilloni avatar Oct 28 '20 18:10 carlocamilloni

@carlocamilloni if your fix works, we have to backport it to branch v2.7

GiovanniBussi avatar Oct 29 '20 14:10 GiovanniBussi

Sure

On 29 Oct 2020, at 15:26, Giovanni [email protected] wrote:

@carlocamilloni https://github.com/carlocamilloni if your fix works, we have to backport it to branch v2.7

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/plumed/plumed2/issues/631#issuecomment-718788837, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABC6L5W2GHB5QBBHDJZV7A3SNF3SPANCNFSM4TCVNM3A.

carlocamilloni avatar Oct 29 '20 14:10 carlocamilloni

unfortunately it does not seem to fix the problem

carlocamilloni avatar Oct 29 '20 15:10 carlocamilloni

I think this is still a problem. @gtribello do you know how to fix it? Input files with replicas basically screw up the page in which they are included

GiovanniBussi avatar Mar 12 '21 09:03 GiovanniBussi