Changes between Version 6 and Version 7 of FilesMonitoring

Timestamp:

Dec 11, 2012, 12:50:02 PM (14 years ago)

Author:

Serge CHASTEL

Comment:

--

FilesMonitoring

@@ -65 +65 @@
  * The lifetime of the file is obtained from both the stage and the role: raw (keep the file till the end of times and possibly after), permanent (a product to be kept), ephemeral (product that will be cleaned up one day), unknown (it's unknown as soon as one is unknown)
 
-To adding a new stage, a new role:
+To add a new stage, a new role:
  * The {{{initialize_IppTemplates}}} function has to be updated;
  * Add a stage definition in the first lines of the  {{{initialize_IppTemplates}}} function. A stage definition is made of a unique name (a Python str) and a Python list of regular expressions (a regular expression is a string as well). You need to define the lifetime of the role you want to the next lines. For instance, when the detrend stage was added, I added:
@@ -77 +77 @@
     Lifetime.add_lifetime( Lifetime('detrend', 'b1jpg', 'permanent') )
 }}}
-Note: I try to keep it clean and easily maintanable by grouping the lifetime definitions under the role associated to it.
  * It is perfectly acceptable to have a stage with no associated lifetime (e.g. skycal and staticsky)
+ * I try to keep it clean and easily maintainable by grouping the lifetime definitions under the role associated to it. Please do the same...
+ * It is strongly advised to add a relevant set of unit tests to the {{{Template}}} class when a new definition is added (Did I hear someone say "If it's not tested it doesn't work"?)
+
+There is a "subtle" behavior for fits files in the classifier. When a filename is initially classified as fits, its analysis is pushed further to check if it is not a particular fits case (lines around the {{{if role.name != 'fits':}}} statement).
+
 = {{{Summary/refresh_table.py}}} Details =