Changeset 2820 for trunk/src

Timestamp:

07/02/13 13:11:02 (11 years ago)

Author:

Malte Marquarding

Message:

Issue #293: added scantbale.drop_history and added extra parameters to scantable.history to allow selection of rows

Location:

trunk/src

Files:

• STHistory.cpp (modified) (2 diffs)
• STHistory.h (modified) (1 diff)
• Scantable.h (modified) (1 diff)
• ScantableWrapper.h (modified) (1 diff)
• python_Scantable.cpp (modified) (1 diff)

trunk/src/STHistory.cpp

@@ -20 +20 @@
 #include "STDefs.h"
 #include "STHistory.h"
-
+#include "MathUtils.h"
 
 using namespace casa;
@@ -81 +81 @@
 {
   const Table& t = other.table();
-  addEntry(asap::SEPERATOR);
-  TableCopy::copyRows(table_, t, table_.nrow(), 0, t.nrow());
-  addEntry(asap::SEPERATOR);
+  if (other.nrow() > 0) {
+    addEntry(asap::SEPERATOR);
+    TableCopy::copyRows(table_, t, table_.nrow(), 0, t.nrow());
+    addEntry(asap::SEPERATOR);
+  }
 
 }
 
-std::vector<std::string> asap::STHistory::getHistory( ) const
+std::vector<std::string> asap::STHistory::getHistory( int nrow, 
+                                                      int start) const
 {
-  std::vector<std::string> stlout;
-  for (uInt i=0; i<table_.nrow(); ++i) {
-    stlout.push_back(itemCol_(i));
+  if (nrow < 0) {
+    nrow = this->nrow();
   }
-  return stlout;
+  AlwaysAssert(nrow <= this->nrow(), AipsError);
+  Vector<String> rows;
+  Slicer slice(IPosition(1, start), IPosition(1, nrow));
+  
+  rows = itemCol_.getColumnRange(slice);
+  return mathutil::tovectorstring(rows);
 }
 
+  void asap::STHistory::drop() {
+    table_.removeRow(table_.rowNumbers());
+  }
+
 }

trunk/src/STHistory.h

@@ -62 +62 @@
    * @return a vector of strings
    */
-  std::vector<std::string> getHistory( ) const;
+  std::vector<std::string> getHistory(int nrow=-1, int start=0 ) const;
 
   const casa::String& name() const { return name_; }
+
+  int nrow() const { return table_.nrow(); }
+
+
+  void drop();
 
 private:

trunk/src/Scantable.h

@@ -378 +378 @@
   void makePersistent(const std::string& filename);
 
-  std::vector<std::string> getHistory() const
-    { return historyTable_.getHistory(); };
+  std::vector<std::string> getHistory(int nrow=-1, int start=0) const
+    { return historyTable_.getHistory(nrow, start); }
+
+  uint historyLength()
+  { return historyTable_.nrow(); }
+  
+  void dropHistory() { historyTable_.drop(); }
 
   void addHistory(const std::string& hist) { historyTable_.addEntry(hist); }

trunk/src/ScantableWrapper.h

@@ -242 +242 @@
   }
 
-  std::vector<std::string> getHistory() const
-    { return table_->getHistory(); }
+  std::vector<std::string> getHistory(int nrow=-1, int start=0) const
+    { return table_->getHistory(nrow, start); }
+
+  uint historyLength() {
+    return table_->historyLength();
+  }
+
+  void dropHistory() { table_->dropHistory(); }
 
   void addHistory(const std::string& hist)

trunk/src/python_Scantable.cpp

@@ -131 +131 @@
     .def("_gethistory", &ScantableWrapper::getHistory)
     .def("_addhistory", &ScantableWrapper::addHistory)
+    .def("drop_history", &ScantableWrapper::dropHistory)
+    .def("_historylength", &ScantableWrapper::historyLength)
     .def("_getselection", &ScantableWrapper::getSelection)
     .def("_setselection", &ScantableWrapper::setSelection)