]> Creatis software - gdcm.git/commitdiff
New method SerieHelper::AddSeriesDetail() to allow user to specifiy
authorjpr <jpr>
Fri, 16 Dec 2005 13:48:46 +0000 (13:48 +0000)
committerjpr <jpr>
Fri, 16 Dec 2005 13:48:46 +0000 (13:48 +0000)
his own Tags Of Interest,
that will be used in SerieHelper::CreateUniqueSeriesIdentifier.
Mathieu's default ones are not enought for 'Tagging' series.

src/gdcmSerieHelper.cxx
src/gdcmSerieHelper.h

index 6928c3aa434bc1ac3cb86d77c6d1ab6707bba64d..412d719e82c7aecbd4bb68e27804dd5379c178b8 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmSerieHelper.cxx,v $
   Language:  C++
-  Date:      $Date: 2005/11/30 08:48:17 $
-  Version:   $Revision: 1.39 $
+  Date:      $Date: 2005/12/16 13:48:46 $
+  Version:   $Revision: 1.40 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -209,11 +209,13 @@ void SerieHelper::AddGdcmFile(File *header)
 }
 
 /**
- * \brief add a rules for restricting a DICOM file to be in the serie we are
- * trying to find. For example you can select only the DICOM file from a
+ * \brief add a rule for restricting a DICOM file to be in the serie we are
+ * trying to find. For example you can select only the DICOM files from a
  * directory which would have a particular EchoTime==4.0.
  * This method is a user level, value is not required to be formatted as a DICOM
  * string
+ * \todo find a trick to allow user if he wants the Rectrictions to be *ored*
+ *       (and not only *anded*)
  * @param   key  Target tag we want restrict on a given value
  * @param value value to be checked to exclude File
  * @param op  operator we want to use to check
@@ -229,6 +231,45 @@ void SerieHelper::AddRestriction(TagKey const &key,
    ExRestrictions.push_back( r ); 
 }
 
+/**
+ * \brief add a rule for restricting a DICOM file to be in the serie we are
+ * trying to find. For example you can select only the DICOM file from a
+ * directory which would have a particular EchoTime==4.0.
+ * This method is a user level, value is not required to be formatted as a DICOM
+ * string
+ * \todo find a trick to allow user if he wants the Rectrictions to be *ored*
+ *       (and not only *anded*)
+ * @param   group tag group number we want restrict on a given value
+ * @param   elem  tag element number we want restrict on a given value 
+ * @param value value to be checked to exclude File
+ * @param op  operator we want to use to check
+ */
+void SerieHelper::AddRestriction(uint16_t group, uint16_t elem, 
+                                 std::string const &value, int op)
+{
+   ExRule r;
+   r.group = group;
+   r.elem  = elem;
+   r.value = value;
+   r.op    = op;
+   ExRestrictions.push_back( r ); 
+}
+
+/**
+ * \brief add an extra  'SerieDetail' for building a 'Serie Identifier'
+ *        that ensures (hope so) File constistency (Series Instance UID doesn't.
+ * @param   group tag group number we want restrict on a given value
+ * @param   elem  tag element number we want restrict on a given value 
+ */
+void SerieHelper::AddSeriesDetail(uint16_t group, uint16_t elem)
+{
+   m_UseSeriesDetails = true;
+   
+   ExDetail d;
+   d.group = group;
+   d.elem  = elem;
+   ExDetails.push_back( d ); 
+}
 /**
  * \brief Sets the root Directory
  * @param   dir Name of the directory to deal with
@@ -768,7 +809,7 @@ bool SerieHelper::FileNameOrdering(FileList *fileList)
 bool SerieHelper::UserOrdering(FileList *fileList)
 {
    std::sort(fileList->begin(), fileList->end(), 
-                                    SerieHelper::UserLessThanFunction);
+                                             SerieHelper::UserLessThanFunction);
    if (!DirectOrder) 
    {
       std::reverse(fileList->begin(), fileList->end());
@@ -776,7 +817,18 @@ bool SerieHelper::UserOrdering(FileList *fileList)
    return true;
 }
 
-std::string SerieHelper::CreateUniqueSeriesIdentifier( File * inFile )
+/**
+ * \brief Heuritics to *try* to build a Serie Identifier that would ensure
+ *        all the images are coherent.
+ *
+ *  We allow user to add his own critierions, using AddSerieDetail
+ *        (he knows more than we do about his images!)
+ *        ex : in tagging series, the only pertnent tag is
+ *        0018|1312 [In-plane Phase Encoding Direction] value : ROW/COLUMN
+ * @param inFile gdcm::File we want to build a Serie Identifier for.
+ * @return the SeriesIdentifier
+ */
+std::string SerieHelper::CreateUniqueSeriesIdentifier( File *inFile )
 {
    if( inFile->IsReadable() )
    {
@@ -830,13 +882,29 @@ std::string SerieHelper::CreateUniqueSeriesIdentifier( File * inFile )
          {
            sColumns = "";
          }
-   
+
   // Concat the new info
          std::string num = sNum.c_str();
          num += sName.c_str();
          num += sThick.c_str();
          num += sRows.c_str();
          num += sColumns.c_str();
+  // Add a loop, here, to deal with any extra user supplied tag.
+  //      We allow user to add his own critierions 
+  //      (he knows more than we do about his images!)
+  //      ex : in tagging series, the only pertinent tag is
+  //          0018|1312 [In-plane Phase Encoding Direction] value : ROW/COLUMN
+  
+      std::string s;  
+      for(SeriesExDetails::iterator it2 = ExDetails.begin();
+          it2 != ExDetails.end();
+          ++it2)
+      {
+         const ExDetail &r = *it2;
+         s = inFile->GetEntryString( r.group, r.elem );        
+         num += s.c_str();
+      }
    
   // Append the new info to the SeriesUID
          id += ".";
@@ -848,7 +916,7 @@ std::string SerieHelper::CreateUniqueSeriesIdentifier( File * inFile )
        for(unsigned int i=0; i<id.size(); i++)
        {
          while(i<id.size()
-               && !( id[i] == '.'
+               && !( id[i] == '.' || id[i] == '-'
                     || (id[i] >= 'a' && id[i] <= 'z')
                     || (id[i] >= '0' && id[i] <= '9')
                     || (id[i] >= 'A' && id[i] <= 'Z')))
index ae6ca5a42ef6d499b2b4a800b7678c464e433b36..f0c486f21fb170c8b3e01716e51f8af73865554d 100644 (file)
@@ -3,8 +3,8 @@
   Program:   gdcm
   Module:    $RCSfile: gdcmSerieHelper.h,v $
   Language:  C++
-  Date:      $Date: 2005/12/09 12:23:38 $
-  Version:   $Revision: 1.33 $
+  Date:      $Date: 2005/12/16 13:48:46 $
+  Version:   $Revision: 1.34 $
                                                                                 
   Copyright (c) CREATIS (Centre de Recherche et d'Applications en Traitement de
   l'Image). All rights reserved. See Doc/License.txt or
@@ -100,7 +100,12 @@ public:
                              { return  (*ItFileSetHt).first; }
    /// All the following allow user to restrict DICOM file to be part
    /// of a particular serie
+   /// \todo : find a trick to allow user to say the retrictetons are ored
+   ///         (not only anded) 
+   ///         ex : keep the images whose SerieNumber is 101 or 102 or 103.
    void AddRestriction(TagKey const &key, std::string const &value, int op);
+   void AddRestriction(uint16_t group, uint16_t elem, std::string const &value,
+                                                                    int op);
 
    /// \brief Use additional series information such as ProtocolName
    ///        and SeriesName to identify when a single SeriesUID contains
@@ -108,7 +113,9 @@ public:
    void SetUseSeriesDetails( bool useSeriesDetails )
                                    { m_UseSeriesDetails = useSeriesDetails;}
    bool GetUseSeriesDetails( ){ return m_UseSeriesDetails; }
-
+   
+   void AddSeriesDetail(uint16_t group, uint16_t elem);
+   
    // \brief Create a string that uniquely identifies a series.   By default
    //         uses the SeriesUID.   If UseSeriesDetails(true) has been called,
    //         then additional identifying information is used.
@@ -173,6 +180,14 @@ private:
    } ExRule;
    typedef std::vector<ExRule> SerieExRestrictions;
    SerieExRestrictions ExRestrictions;
+   
+   typedef struct {
+      uint16_t group;
+      uint16_t elem;
+   } ExDetail;
+   typedef std::vector<ExDetail> SeriesExDetails; 
+   SeriesExDetails ExDetails;
+    
    bool m_UseSeriesDetails;
    
    /// \brief Bit string integer (each one considered as a boolean)