<div dir="ltr"><div><div><div><div><div><div><div><div><div>Last week I modified the vtu and hdf5 io code according to Markus'  guide to make it  looks better:<br></div>1. Changed the method and class names according to Camel-case  to adhere to dune guidelines.<br>
</div>2. Remove all the indent and trailing white space in the code.<br></div>3. Put all the functions that deals with io into one class(vtkPWriter). <br></div>4.Extend the template parameters of vtkPWriter. It could only deal with 2 dimensional psurface and float. Not it support 1 dimensional psurface, and the float.<br>
</div><div>5. Use command line parameters to get the input filename. <br></div><div>The modified code have been pushed to git://<a href="http://gitorious.org/gcos/psurface-convert.git">gitorious.org/gcos/psurface-convert.git</a><br>
<br></div>Issues that I have not solved includes:<br>+ Moving headers<br>"Some your <span class="">headers</span> seem to be usable by other programs. E.g. one might want to read xmf files in one's own applications. This <span class="">headers</span>
 should be under dune and be installed, the cpp file should become part 
of a library. You can see how this is done in dune-common/dune/common. "<br></div>+The include path issue <br>  What is up with the weired AC_ARG_WITH clauses in <a href="http://configure.ac" target="_blank">configure.ac</a>? I do not<br>

   think that they are needed and doubt that they serve their intended<br>
   purpose.</div>+Please try to document your code along (or even before) writing it<br>
   using the doxygen syntax. Most developers (me, too) are not<br>
   disciplined enough to document their code after writing it. In the<br>
   rare occasion the we are and that there is time left, we are often<br>
   having a hard time understanding it ourselves (which is a shame and<br>
   waste of time). I would recommend documenting it properly _now_.<br>
<br>I will address these issues in the next week  and then move to task #2:<br></div>"start with input from a gmsh file and construct a
PSurface object from a given domain surface."</div><div>In order to to that I need a gmsh file to start with. Maybe I should first write functions that can write PSurface object into gmsh file.  The file should contain all information I need to build psurface. <br>
<br></div><div>Best,<br></div><div>Xiaoxue<br></div></div>