Difference between revisions of "GlueX Offline Meeting, November 4, 2009"

From GlueXWiki
Jump to: navigation, search
(Location)
m (Text replacement - "www/halldweb1/" to "www/halldweb/")
 
(10 intermediate revisions by 2 users not shown)
Line 19: Line 19:
 
===Slides===
 
===Slides===
  
Talks can be deposited in the directory <code>/group/halld/www/halldweb1/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://halldweb1.jlab.org/talks/2009-11/ .
+
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 27: Line 27:
 
#* [http://www.jlab.org/Hall-D/software/ScheduleIt/ Meeting time schedule] decision
 
#* [http://www.jlab.org/Hall-D/software/ScheduleIt/ Meeting time schedule] decision
 
# [http://www.jlab.org/~wolin/halld.txt Hall D Group Membership]: Elliott
 
# [http://www.jlab.org/~wolin/halld.txt Hall D Group Membership]: Elliott
 +
# [http://www.jlab.org/Hall-D/software/GlueXCodingStandard.html GlueX Coding Conventions]: David
 +
# [[Media:20091104_parameterized_bfield.pdf| Parameterized B-field]]: David
 
# [[Action Items From Hall-D Software Meetings | Action item]] review: all
 
# [[Action Items From Hall-D Software Meetings | Action item]] review: all
 +
 +
=Minutes=
 +
 +
'''JLab''': Craig Bookwalter, Mark Ito (chair), David Lawrence, Sascha Somov, Simon Taylor, Aram Teymurazyan, Elliott Wolin, Beni Zihlmann<br>
 +
'''UConn''': Richard Jones
 +
 +
==Review of Last Meeting's Minutes==
 +
 +
We glanced over the items from the [[GlueX Offline Meeting, October 21, 2009#Minutes|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 [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==
 +
 +
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 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 [[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==
 +
 +
# 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

Location

The meeting will be at 2:00 pm in CEBAF Center A110

Telephone

To connect by telephone:

  1. dial:
    • 800-377-8846 : US
    • 888-276-7715 : Canada
    • 302-709-8424 : International
  2. 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

  1. Review minutes from last meeting: all
  2. Announcements
  3. Hall D Group Membership: Elliott
  4. GlueX Coding Conventions: David
  5. Parameterized B-field: David
  6. 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

  1. Schedule a presentation on OSG from Richard -> Mark
  2. Schedule discussion of a "warning-free code" policy -> Craig