Difference between revisions of "GlueX Offline Meeting, November 4, 2009"
(→Minutes) |
m (Text replacement - "www/halldweb1/" to "www/halldweb/") |
||
(5 intermediate revisions by the same user not shown) | |||
Line 19: | Line 19: | ||
===Slides=== | ===Slides=== | ||
− | Talks can be deposited in the directory <code>/group/halld/www/ | + | Talks can be deposited in the directory <code>/group/halld/www/halldweb/html/talks/2009-11</code> on the JLab CUE (you have to be a member of the "halld" Unix group). This directory is accessible from the web at https://halldweb.jlab.org/talks/2009-11/ . |
=Agenda= | =Agenda= | ||
Line 38: | Line 38: | ||
==Review of Last Meeting's Minutes== | ==Review of Last Meeting's Minutes== | ||
− | We glanced over the items from the | + | We glanced over the items from the [[GlueX Offline Meeting, October 21, 2009#Minutes|October 21]] meeting. No additional discussion was generated. |
==Announcements== | ==Announcements== | ||
Line 46: | Line 46: | ||
==Hall D Group Membership== | ==Hall D Group Membership== | ||
− | Elliott led us through the | + | Elliott led us through the [https://halldweb.jlab.org/talks/2009-11/halld.txt list of members of the "halld"] Unix group at JLab. IT Division has suggested that we review the list and drop those users who do not need the associated privileges. Elliott will drop the obvious candidates from the list. |
==GlueX Coding Conventions== | ==GlueX Coding Conventions== | ||
− | David presented the | + | David presented the [http://www.jlab.org/Hall-D/software/GlueXCodingStandard.html results of his editorial work] on the CLEO conventions that we agreed to use as a starting point for our own conventions. We went over on a few items that David, Richard and Elliott flagged as requiring discussion. We came to some conclusions on most of them: |
* drop namespace requirement for variables in a library | * drop namespace requirement for variables in a library | ||
Line 62: | Line 62: | ||
* drop the integer less than greater than thing for testing intervals | * drop the integer less than greater than thing for testing intervals | ||
* scripts for skeletal template should include link to coding standards[?] | * scripts for skeletal template should include link to coding standards[?] | ||
+ | * there was significant controversy over tabs vs. spaces for indenting code | ||
+ | * there was controversy over the capitalization of function names | ||
Alex and Mark asked what constitutes thread safe code. The answer is not simple. | Alex and Mark asked what constitutes thread safe code. The answer is not simple. | ||
− | We did not decide what to do from here on | + | We did not decide what to do from here on what to do with these rules. |
+ | ==Parameterized B-field== | ||
+ | David presented his work on [[Media:20091104_parameterized_bfield.pdf|parametrizing the magnetic field]]. He does a series of fits to a set of orthogonal polynomials resulting in a two-dimensional functional form for the field in r and z. The using the parametrization saves in start up time and memory usage but access to the field values is slower compared to using a field map. Suggestions were made on how things could be speeded up. | ||
+ | |||
+ | ==Action Item Review== | ||
+ | |||
+ | We went over the list. Some things need crossing off. We decided to drop the trajectory curvature correction in HDGeant from the list and add it back should need for it arise. | ||
==New Action Items== | ==New Action Items== | ||
# Schedule a presentation on OSG from Richard -> Mark | # Schedule a presentation on OSG from Richard -> Mark | ||
− | # | + | # Schedule discussion of a "warning-free code" policy -> Craig |
Latest revision as of 17:23, 31 March 2015
Contents
Location
The meeting will be at 2:00 pm in CEBAF Center A110
Telephone
To connect by telephone:
- dial:
- 800-377-8846 : US
- 888-276-7715 : Canada
- 302-709-8424 : International
- enter participant code: 77438230# (remember the "#")
Video Conferencing
A) ESNet: 8542553
B) EVO: direct meeting link
Slides
Talks can be deposited in the directory /group/halld/www/halldweb/html/talks/2009-11
on the JLab CUE (you have to be a member of the "halld" Unix group). This directory is accessible from the web at https://halldweb.jlab.org/talks/2009-11/ .
Agenda
- Review minutes from last meeting: all
- Announcements
- Meeting time schedule decision
- Hall D Group Membership: Elliott
- GlueX Coding Conventions: David
- Parameterized B-field: David
- Action item review: all
Minutes
JLab: Craig Bookwalter, Mark Ito (chair), David Lawrence, Sascha Somov, Simon Taylor, Aram Teymurazyan, Elliott Wolin, Beni Zihlmann
UConn: Richard Jones
Review of Last Meeting's Minutes
We glanced over the items from the October 21 meeting. No additional discussion was generated.
Announcements
Meeting Time: We looked at the scheduler and settled on Tuesday at 2:00 pm as a suitable time for regular bi-weekly meetings. The next offline meeting will therefore be on Tuesday, November 17.
Hall D Group Membership
Elliott led us through the list of members of the "halld" Unix group at JLab. IT Division has suggested that we review the list and drop those users who do not need the associated privileges. Elliott will drop the obvious candidates from the list.
GlueX Coding Conventions
David presented the results of his editorial work on the CLEO conventions that we agreed to use as a starting point for our own conventions. We went over on a few items that David, Richard and Elliott flagged as requiring discussion. We came to some conclusions on most of them:
- drop namespace requirement for variables in a library
- drop prohibition against inline constructor
- "at least one constructor" not "defined default constructor"
- c not k for constants
- lack of virtual destructor causes compiler errors, drop the rule
- "if (!p)" where p is a pointer, is not that bad
- go ahead and declare more than one variable per line
- "use doxygen style comments" not "use //"
- drop the integer less than greater than thing for testing intervals
- scripts for skeletal template should include link to coding standards[?]
- there was significant controversy over tabs vs. spaces for indenting code
- there was controversy over the capitalization of function names
Alex and Mark asked what constitutes thread safe code. The answer is not simple.
We did not decide what to do from here on what to do with these rules.
Parameterized B-field
David presented his work on parametrizing the magnetic field. He does a series of fits to a set of orthogonal polynomials resulting in a two-dimensional functional form for the field in r and z. The using the parametrization saves in start up time and memory usage but access to the field values is slower compared to using a field map. Suggestions were made on how things could be speeded up.
Action Item Review
We went over the list. Some things need crossing off. We decided to drop the trajectory curvature correction in HDGeant from the list and add it back should need for it arise.
New Action Items
- Schedule a presentation on OSG from Richard -> Mark
- Schedule discussion of a "warning-free code" policy -> Craig