On 14/01/10 12:54, Takuya OSHIMA wrote:
Dear Gmsh developers,
Thanks for hearing our comments. We see and really appreciate that the
tutorials (especially t2.geo) have improved and are a lot more
comprehensive now.
Thanks for additional comments: they've been checked in as
https://geuz.org/trac/gmsh/changeset/7091
This time here are our comments about t4.geo and t5.geo. As our
intended audience are real novices the followings are what we do
really consider, but we admit they may be overkill for usual users. We
appreciate if the Gmsh developers take some of them into account.
[t4.geo]
* It might be helpful for users if there are brief notes about each
function (Hypot, Sqrt, T2, T3, StrCat, Today).
* Line 90-91 "Since this surface has a hole, its definition now
requires two lines loops": Perhaps a bit more explanation is needed
that, if we have two or more line loops specified in a Plane Surface
command, all line loops other than the first one define holes.
[t5.geo]
* Line 9: be -> by
* Lines 21-29: In reality, the Windows version of Gmsh 2.4.2 generated
1322 nodes in volumes and 11,665 tetrahedra with -clscale 1
specified, and about 300,000 nodes in volumes and about 2 million
tetrahedra with -clscale 0.2 (the numbers are important for the
users to feel confident that they are operating Gmsh correctly). It
might be further better if there is a note that the numbers can be
checked via Tools->Statistics.
* Line 128: It's obvious, but would be clearer and draws reader's
attention if thers's an annotation that Call is the command to call
the function.
* Line 136 "the format string should only contain valid floating point
format specifiers": It may be more helpful to non-programmer users
if there is an explanation that %g is the floating point specifier.
* Line 156: Like t4.geo, "the surface loops other than the first one
define holes..."
p.s. Just noticed one more thing: the bottom of the revised t2.geo
// To save all the tetrahedra discretizing the volumes 119 and 120
should be 129 and 130.
Takuya
Takuya OSHIMA, Ph.D.
Faculty of Engineering, Niigata University
8050 Ikarashi-Ninocho, Nishi-ku, Niigata, 950-2181, JAPAN
From: Christophe Geuzaine<[email protected]>
Subject: Re: [Gmsh] comments about tutorial files
Date: Mon, 11 Jan 2010 15:07:08 +0100
On 21/12/09 12:00, Takuya OSHIMA wrote:
Hi,
We are doing some detailed review of the tutorial files (t1.geo -
t9.geo that come with Gmsh 2.4.2) as a process of creating teaching
materials for our FE code that uses Gmsh as preprocessing front
end. In the process, at the stage of the review until t3.geo, we came
to have the following comments (we hope we are not nitpicking). What
do you think?
Dear Takuya - many thanks for your insightful comments. We've tried to
address them in the new version of the tutorial
https://geuz.org/trac/gmsh/changeset/7024 (login: gmsh passwd: gmsh)
Let us know if you have other ideas to improve the documentation: we
would be very glad to use them.
Best,
Christophe and David
Thanks,
Takuya
[t1.geo]
Line 62 "specifying their orientation": it might be better if we have
an explanation that this applies only for lines and surfaces, and
have an example in one of the following physical entity definitions
so that one can see how to specify the orientation.
[t2.geo]
Overall, one may hardly be able to follow the tutorial since there is
little explanation about which entities Points 8, 12, 16, 7, Lines
102-105, etc. are referring to (and which exact process generated
the entities). If the entity numbers are to be obtained by saving
unrolled geometry as explained at the bottom of t2.geo, the
explanation might be better to come at the top of the file and to be
clearly stated as such.
Lines 32-33 "a list": it might be better if it is clearly stated that
the list is referring to tmp[] (obvious for programmers, but not for
non-programmers).
[t3.geo]
Line 40 "news": the special meaning of the variable may be worth
being mentioned.
Lines 76-77 "`Tools->Options->Save as defaults' button": there does
not seem to be such a button on the GUI?
Takuya OSHIMA, Ph.D.
Faculty of Engineering, Niigata University
8050 Ikarashi-Ninocho, Nishi-ku, Niigata, 950-2181, JAPAN
_______________________________________________
gmsh mailing list
[email protected]
http://www.geuz.org/mailman/listinfo/gmsh
--
Prof. Christophe Geuzaine
University of Liege, Electrical Engineering and Computer Science
http://www.montefiore.ulg.ac.be/~geuzaine
_______________________________________________
gmsh mailing list
[email protected]
http://www.geuz.org/mailman/listinfo/gmsh
--
Prof. Christophe Geuzaine
University of Liege, Electrical Engineering and Computer Science
http://www.montefiore.ulg.ac.be/~geuzaine
_______________________________________________
gmsh mailing list
[email protected]
http://www.geuz.org/mailman/listinfo/gmsh
