Fixed compile issues introduced in merge

Former-commit-id: 059b7c84b3

src/modelchecker/csl/SparseCtmcCslModelChecker.cpp

@@ -34,7 +34,7 @@ namespace storm {
         template <typename SparseCtmcModelType>
         bool SparseCtmcCslModelChecker<SparseCtmcModelType>::canHandle(CheckTask<storm::logic::Formula> const& checkTask) const {
             storm::logic::Formula const& formula = checkTask.getFormula();
-            return formula.isInFragment(storm::logic::csrl().setGloballyFormulasAllowed(false).setLongRunAverageRewardFormulasAllowed(false).setLongRunAverageProbabilitiesAllowed(true).setExpectedTimeAllowed(true));
+            return formula.isInFragment(storm::logic::csrl().setGloballyFormulasAllowed(false).setLongRunAverageRewardFormulasAllowed(false).setLongRunAverageProbabilitiesAllowed(true).setTimeAllowed(true));
         }
         
         template <typename SparseCtmcModelType>
@@ -114,12 +114,12 @@ namespace storm {
         }
 
         template <typename SparseCtmcModelType>
-        std::unique_ptr<CheckResult> SparseCtmcCslModelChecker<SparseCtmcModelType>::computeExpectedTimes(CheckTask<storm::logic::EventuallyFormula> const& checkTask) {
+        std::unique_ptr<CheckResult> SparseCtmcCslModelChecker<SparseCtmcModelType>::computeReachabilityTimes(storm::logic::RewardMeasureType rewardMeasureType, CheckTask<storm::logic::EventuallyFormula> const& checkTask) {
             storm::logic::EventuallyFormula const& eventuallyFormula = checkTask.getFormula();
             std::unique_ptr<CheckResult> subResultPointer = this->check(eventuallyFormula.getSubformula());
             ExplicitQualitativeCheckResult& subResult = subResultPointer->asExplicitQualitativeCheckResult();
 
-            std::vector<ValueType> numericResult = storm::modelchecker::helper::SparseCtmcCslHelper<ValueType>::computeExpectedTimes(this->getModel().getTransitionMatrix(), this->getModel().getBackwardTransitions(), this->getModel().getExitRateVector(), this->getModel().getInitialStates(), subResult.getTruthValuesVector(), checkTask.isQualitativeSet(), *linearEquationSolverFactory);
+            std::vector<ValueType> numericResult = storm::modelchecker::helper::SparseCtmcCslHelper<ValueType>::computeReachabilityTimes(this->getModel().getTransitionMatrix(), this->getModel().getBackwardTransitions(), this->getModel().getExitRateVector(), this->getModel().getInitialStates(), subResult.getTruthValuesVector(), checkTask.isQualitativeSet(), *linearEquationSolverFactory);
             return std::unique_ptr<CheckResult>(new ExplicitQuantitativeCheckResult<ValueType>(std::move(numericResult)));
         }
 

src/modelchecker/csl/SparseCtmcCslModelChecker.h

@@ -28,7 +28,7 @@ namespace storm {
             virtual std::unique_ptr<CheckResult> computeInstantaneousRewards(storm::logic::RewardMeasureType rewardMeasureType, CheckTask<storm::logic::InstantaneousRewardFormula> const& checkTask) override;
             virtual std::unique_ptr<CheckResult> computeReachabilityRewards(storm::logic::RewardMeasureType rewardMeasureType, CheckTask<storm::logic::EventuallyFormula> const& checkTask) override;
             virtual std::unique_ptr<CheckResult> computeLongRunAverageProbabilities(CheckTask<storm::logic::StateFormula> const& checkTask) override;
-            virtual std::unique_ptr<CheckResult> computeExpectedTimes(CheckTask<storm::logic::EventuallyFormula> const& checkTask) override;
+            virtual std::unique_ptr<CheckResult> computeReachabilityTimes(storm::logic::RewardMeasureType rewardMeasureType, CheckTask<storm::logic::EventuallyFormula> const& checkTask) override;
 
         private:
             // An object that is used for solving linear equations and performing matrix-vector multiplication.

src/modelchecker/csl/helper/SparseCtmcCslHelper.cpp

@@ -656,7 +656,7 @@ namespace storm {
             }
 
             template <typename ValueType>
-            std::vector<ValueType> SparseCtmcCslHelper<ValueType>::computeExpectedTimes(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative, storm::utility::solver::LinearEquationSolverFactory<ValueType> const& linearEquationSolverFactory) {
+            std::vector<ValueType> SparseCtmcCslHelper<ValueType>::computeReachabilityTimes(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative, storm::utility::solver::LinearEquationSolverFactory<ValueType> const& linearEquationSolverFactory) {
                 // Compute expected time on CTMC by reduction to DTMC with rewards.
                 storm::storage::SparseMatrix<ValueType> probabilityMatrix = computeProbabilityMatrix(rateMatrix, exitRateVector);
 
@@ -677,7 +677,7 @@ namespace storm {
 
             
             template <typename ValueType>
-            std::vector<ValueType> SparseCtmcCslHelper<ValueType>::computeExpectedTimesElimination(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative) {
+            std::vector<ValueType> SparseCtmcCslHelper<ValueType>::computeReachabilityTimesElimination(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative) {
                 // Use "normal" function again, if RationalFunction finally is supported.
                 // Compute expected time on CTMC by reduction to DTMC with rewards.
                 storm::storage::SparseMatrix<ValueType> probabilityMatrix = computeProbabilityMatrix(rateMatrix, exitRateVector);
@@ -704,7 +704,7 @@ namespace storm {
 
 #ifdef STORM_HAVE_CARL
             template std::vector<storm::RationalFunction> SparseCtmcCslHelper<storm::RationalFunction>::computeUntilProbabilitiesElimination(storm::storage::SparseMatrix<storm::RationalFunction> const& rateMatrix, storm::storage::SparseMatrix<storm::RationalFunction> const& backwardTransitions, std::vector<storm::RationalFunction> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& phiStates, storm::storage::BitVector const& psiStates, bool qualitative);
-            template std::vector<storm::RationalFunction> SparseCtmcCslHelper<storm::RationalFunction>::computeExpectedTimesElimination(storm::storage::SparseMatrix<storm::RationalFunction> const& rateMatrix, storm::storage::SparseMatrix<storm::RationalFunction> const& backwardTransitions, std::vector<storm::RationalFunction> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative);
+            template std::vector<storm::RationalFunction> SparseCtmcCslHelper<storm::RationalFunction>::computeReachabilityTimesElimination(storm::storage::SparseMatrix<storm::RationalFunction> const& rateMatrix, storm::storage::SparseMatrix<storm::RationalFunction> const& backwardTransitions, std::vector<storm::RationalFunction> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative);
 #endif
         }
     }

src/modelchecker/csl/helper/SparseCtmcCslHelper.h

@@ -29,8 +29,8 @@ namespace storm {
                 
                 static std::vector<ValueType> computeLongRunAverageProbabilities(storm::storage::SparseMatrix<ValueType> const& probabilityMatrix, storm::storage::BitVector const& psiStates, std::vector<ValueType> const* exitRateVector, bool qualitative, storm::utility::solver::LinearEquationSolverFactory<ValueType> const& linearEquationSolverFactory);
                 
-                static std::vector<ValueType> computeExpectedTimes(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative, storm::utility::solver::LinearEquationSolverFactory<ValueType> const& minMaxLinearEquationSolverFactory);
-                static std::vector<ValueType> computeExpectedTimesElimination(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative);
+                static std::vector<ValueType> computeReachabilityTimes(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative, storm::utility::solver::LinearEquationSolverFactory<ValueType> const& minMaxLinearEquationSolverFactory);
+                static std::vector<ValueType> computeReachabilityTimesElimination(storm::storage::SparseMatrix<ValueType> const& rateMatrix, storm::storage::SparseMatrix<ValueType> const& backwardTransitions, std::vector<ValueType> const& exitRateVector, storm::storage::BitVector const& initialStates, storm::storage::BitVector const& targetStates, bool qualitative);
 
                 /*!
                  * Computes the matrix representing the transitions of the uniformized CTMC.

src/parser/FormulaParser.cpp

@@ -280,7 +280,7 @@ namespace storm {
             booleanLiteralFormula = (qi::lit("true")[qi::_a = true] | qi::lit("false")[qi::_a = false])[qi::_val = phoenix::bind(&FormulaParserGrammar::createBooleanLiteralFormula, phoenix::ref(*this), qi::_a)];
             booleanLiteralFormula.name("boolean literal formula");
             
-            operatorFormula = probabilityOperator | rewardOperator | longRunAverageOperator | expectedTimeOperator;
+            operatorFormula = probabilityOperator | rewardOperator | longRunAverageOperator | timeOperator;
             operatorFormula.name("operator formulas");
             
             atomicStateFormula = booleanLiteralFormula | labelFormula | expressionFormula | (qi::lit("(") > stateFormula > qi::lit(")")) | operatorFormula;
@@ -355,7 +355,7 @@ namespace storm {
             debug(probabilityOperator);
             debug(rewardOperator);
             debug(longRunAverageOperator);
-            debug(expectedTimeOperator);
+            debug(timeOperator);
             debug(pathFormulaWithoutUntil);
             debug(pathFormula);
 //            debug(conditionalFormula);
@@ -379,7 +379,7 @@ namespace storm {
             qi::on_error<qi::fail>(probabilityOperator, handler(qi::_1, qi::_2, qi::_3, qi::_4));
             qi::on_error<qi::fail>(rewardOperator, handler(qi::_1, qi::_2, qi::_3, qi::_4));
             qi::on_error<qi::fail>(longRunAverageOperator, handler(qi::_1, qi::_2, qi::_3, qi::_4));
-            qi::on_error<qi::fail>(expectedTimeOperator, handler(qi::_1, qi::_2, qi::_3, qi::_4));
+            qi::on_error<qi::fail>(timeOperator, handler(qi::_1, qi::_2, qi::_3, qi::_4));
             qi::on_error<qi::fail>(operatorInformation, handler(qi::_1, qi::_2, qi::_3, qi::_4));
             qi::on_error<qi::fail>(pathFormulaWithoutUntil, handler(qi::_1, qi::_2, qi::_3, qi::_4));
             qi::on_error<qi::fail>(pathFormula, handler(qi::_1, qi::_2, qi::_3, qi::_4));

src/storage/BitVector.cpp

@@ -468,10 +468,10 @@ namespace storm {
             if (bitIndexInBucket  < 62) { // bitIndexInBucket + 2 < 64
                 // If the value stops before the end of the bucket, we need to erase some lower bits.
                 mask &= ~((1ull << (62 - (bitIndexInBucket))) - 1ull);
-                return (bucketVector[bucket] & mask) >> (62 - bitIndexInBucket);
+                return (buckets[bucket] & mask) >> (62 - bitIndexInBucket);
             } else {
                 // In this case, it suffices to take the current mask.
-                return bucketVector[bucket] & mask;
+                return buckets[bucket] & mask;
             }
         }
         
@@ -669,29 +669,29 @@ namespace storm {
             storm::storage::BitVector result(length, false);
             
             uint_fast64_t offset = start % 64;
-            uint_fast64_t bucketIndex = start / 64;
-            uint_fast64_t insertBucket = 0;
+            uint64_t* getBucket = buckets + (start / 64);
+            uint64_t* insertBucket = result.buckets;
             uint_fast64_t getValue;
             uint_fast64_t writeValue = 0;
             uint_fast64_t noBits = 0;
             if (offset == 0) {
                 // Copy complete buckets
-                for ( ; noBits + 64 <= length; ++bucketIndex, ++insertBucket, noBits += 64) {
-                    result.bucketVector[insertBucket] = bucketVector[bucketIndex];
+                for ( ; noBits + 64 <= length; ++getBucket, ++insertBucket, noBits += 64) {
+                    *insertBucket = *getBucket;
                 }
             } else {
                 //Get first bits up until next bucket
-                getValue = bucketVector[bucketIndex];
+                getValue = *getBucket;
                 writeValue = (getValue << offset);
                 noBits += (64-offset);
-                ++bucketIndex;
+                ++getBucket;
                 
                 //Get complete buckets
-                for ( ; noBits + 64 <= length; ++bucketIndex, ++insertBucket, noBits += 64) {
-                    getValue = bucketVector[bucketIndex];
+                for ( ; noBits + 64 <= length; ++getBucket, ++insertBucket, noBits += 64) {
+                    getValue = *getBucket;
                     // Get bits till write bucket is full
                     writeValue |= (getValue >> (64-offset));
-                    result.bucketVector[insertBucket] = writeValue;
+                    *insertBucket = writeValue;
                     // Get bits up until next bucket
                     writeValue = (getValue << offset);
                 }
@@ -699,22 +699,23 @@ namespace storm {
             
             // Write last bits
             uint_fast64_t remainingBits = length - noBits;
-            assert(bucketIndex < bucketCount());
+            assert(getBucket != buckets + bucketCount());
             // Get remaining bits
-            getValue = (bucketVector[bucketIndex] >> (64-remainingBits)) << (64-remainingBits);
+            getValue = (*getBucket >> (64-remainingBits)) << (64-remainingBits);
             assert(remainingBits < 64);
             // Write bucket
-            assert(insertBucket < result.bucketCount());
+            assert(insertBucket != result.buckets + result.bucketCount());
             if (offset == 0) {
-                result.bucketVector[insertBucket]= getValue;
+                *insertBucket = getValue;
             } else {
                 writeValue |= getValue >> (64-offset);
-                result.bucketVector[insertBucket] = writeValue;
+                *insertBucket = writeValue;
                 if (remainingBits > offset) {
                     // Write last bits in new value
                     writeValue = (getValue << offset);
-                    assert(insertBucket+1 < result.bucketCount());
-                    result.bucketVector[++insertBucket] = writeValue;
+                    ++insertBucket;
+                    assert(insertBucket != result.buckets + result.bucketCount());
+                    *insertBucket = writeValue;
                 }
             }
 
@@ -752,56 +753,58 @@ namespace storm {
             assert(start + other.bitCount <= bitCount);
             
             uint_fast64_t offset = start % 64;
-            uint_fast64_t bucketIndex = start / 64;
-            uint_fast64_t getBucket = 0;
+            uint64_t* insertBucket = buckets + (start / 64);
+            uint64_t* getBucket = other.buckets;
             uint_fast64_t getValue;
             uint_fast64_t writeValue = 0;
             uint_fast64_t noBits = 0;
             if (offset == 0) {
                 // Copy complete buckets
-                for ( ; noBits + 64 <= other.bitCount; ++bucketIndex, ++getBucket, noBits += 64) {
-                    bucketVector[bucketIndex] = other.bucketVector[getBucket];
+                for ( ; noBits + 64 <= other.bitCount; ++insertBucket, ++getBucket, noBits += 64) {
+                    *insertBucket = *getBucket;
                 }
             } else {
                 //Get first bits up until next bucket
-                getValue = other.bucketVector[getBucket];
-                writeValue = (bucketVector[bucketIndex] >> (64-offset)) << (64-offset);
+                getValue = *getBucket;
+                writeValue = (*insertBucket >> (64-offset)) << (64-offset);
                 writeValue |= (getValue >> offset);
-                bucketVector[bucketIndex] = writeValue;
+                *insertBucket = writeValue;
                 noBits += (64-offset);
-                ++bucketIndex;
+                ++insertBucket;
                 
                 //Get complete buckets
-                for ( ; noBits + 64 <= other.bitCount; ++bucketIndex, noBits += 64) {
+                for ( ; noBits + 64 <= other.bitCount; ++insertBucket, noBits += 64) {
                     // Get all remaining bits from other bucket
                     writeValue = getValue << (64-offset);
                     // Get bits from next bucket
-                    getValue = other.bucketVector[++getBucket];
+                    ++getBucket;
+                    getValue = *getBucket;
                     writeValue |= getValue >> offset;
-                    bucketVector[bucketIndex] = writeValue;
+                    *insertBucket = writeValue;
                 }
             }
             
             // Write last bits
             uint_fast64_t remainingBits = other.bitCount - noBits;
             assert(remainingBits < 64);
-            assert(bucketIndex < bucketCount());
-            assert(getBucket < other.bucketCount());
+            assert(insertBucket != buckets + bucketCount());
+            assert(getBucket != other.buckets + other.bucketCount());
             // Get remaining bits of bucket
-            getValue = other.bucketVector[getBucket];
+            getValue = *getBucket;
             if (offset > 0) {
                 getValue = getValue << (64-offset);
             }
             // Get unchanged part of bucket
-            writeValue = (bucketVector[bucketIndex] << remainingBits) >> remainingBits;
+            writeValue = (*insertBucket << remainingBits) >> remainingBits;
             if (remainingBits > offset && offset > 0) {
                 // Remaining bits do not come from one bucket -> consider next bucket
-                assert(getBucket == other.bucketCount() - 2);
-                getValue |= other.bucketVector[++getBucket] >> offset;
+                ++getBucket;
+                assert(getBucket != other.buckets + other.bucketCount());
+                getValue |= *getBucket >> offset;
             }
             // Write completely
             writeValue |= getValue;
-            bucketVector[bucketIndex] = writeValue;
+            *insertBucket = writeValue;
      
 #ifdef ASSERT_BITVECTOR
             // Check correctness of setter
@@ -919,14 +922,14 @@ namespace storm {
             out << "bit vector(" << getNumberOfSetBits() << "/" << bitCount << ") ";
             uint_fast64_t index = 0;
             for ( ; index * 64 + 64 <= bitCount; ++index) {
-                std::bitset<64> tmp(bucketVector[index]);
+                std::bitset<64> tmp(buckets[index]);
                 out << tmp << "|";
             }
             
             // Print last bits
             if (index * 64 < bitCount) {
-                assert(index == bucketVector.size() - 1);
-                std::bitset<64> tmp(bucketVector[index]);
+                assert(index == bucketCount() - 1);
+                std::bitset<64> tmp(buckets[index]);
                 for (size_t i = 0; i + index * 64 < bitCount; ++i) {
                     // Bits are counted from rightmost in bitset
                     out << tmp[63-i];

src/storage/FlexibleSparseMatrix.cpp

@@ -13,8 +13,8 @@ namespace storm {
         }
         
         template<typename ValueType>
-        FlexibleSparseMatrix<ValueType>::FlexibleSparseMatrix(storm::storage::SparseMatrix<ValueType> const& matrix, bool setAllValuesToOne) : data(matrix.getRowCount()), columnCount(matrix.getColumnCount()), nonzeroEntryCount(matrix.getNonzeroEntryCount()), nontrivialRowGrouping(matrix.hasNontrivialRowGrouping()) {
-            if (nontrivialRowGrouping) {
+        FlexibleSparseMatrix<ValueType>::FlexibleSparseMatrix(storm::storage::SparseMatrix<ValueType> const& matrix, bool setAllValuesToOne) : data(matrix.getRowCount()), columnCount(matrix.getColumnCount()), nonzeroEntryCount(matrix.getNonzeroEntryCount()), trivialRowGrouping(matrix.hasTrivialRowGrouping()) {
+            if (!trivialRowGrouping) {
                 rowGroupIndices = matrix.getRowGroupIndices();
             }
             for (index_type rowIndex = 0; rowIndex < matrix.getRowCount(); ++rowIndex) {
@@ -130,8 +130,8 @@ namespace storm {
         }
         
         template<typename ValueType>
-        bool FlexibleSparseMatrix<ValueType>::hasNontrivialRowGrouping() const {
-            return nontrivialRowGrouping;
+        bool FlexibleSparseMatrix<ValueType>::hasTrivialRowGrouping() const {
+            return trivialRowGrouping;
         }
 
         template<typename ValueType>
@@ -208,7 +208,7 @@ namespace storm {
             }
             out << std::endl;
             
-            if (matrix.hasNontrivialRowGrouping()) {
+            if (!matrix.hasTrivialRowGrouping()) {
                 // Iterate over all row groups
                 FlexibleIndex rowGroupCount = matrix.getRowGroupCount();
                 for (FlexibleIndex rowGroup = 0; rowGroup < rowGroupCount; ++rowGroup) {

src/storage/FlexibleSparseMatrix.h

@@ -149,11 +149,11 @@ namespace storm {
             bool empty() const;
             
             /*!
-             * Retrieves whether the matrix has a (possibly) non-trivial row grouping.
+             * Retrieves whether the matrix has a (possibly) trivial row grouping.
              *
-             * @return True iff the matrix has a (possibly) non-trivial row grouping.
+             * @return True iff the matrix has a (possibly) trivial row grouping.
              */
-            bool hasNontrivialRowGrouping() const;
+            bool hasTrivialRowGrouping() const;
 
             /*!
              * Creates a submatrix of the current matrix in place by dropping all rows and columns whose bits are not
@@ -198,9 +198,9 @@ namespace storm {
             // The number of entries in the matrix.
             index_type nonzeroEntryCount;
             
-            // A flag that indicates whether the matrix has a non-trivial row-grouping, i.e. (possibly) more than one
-            // row per row group.
-            bool nontrivialRowGrouping;
+            // A flag indicating whether the matrix has a trivial row grouping. Note that this may be true and yet
+            // there may be row group indices, because they were requested from the outside.
+            bool trivialRowGrouping;
             
             // A vector indicating the row groups of the matrix.
             std::vector<index_type> rowGroupIndices;

src/storage/SparseMatrix.cpp

@@ -264,9 +264,11 @@ namespace storm {
             // Sort columns per row
             typename SparseMatrix<ValueType>::index_type endGroups;
             typename SparseMatrix<ValueType>::index_type endRows;
-            for (index_type group = 0; group < rowGroupIndices.size(); ++group) {
-                endGroups = group < rowGroupIndices.size()-1 ? rowGroupIndices[group+1] : rowIndications.size();
-                for (index_type i = rowGroupIndices[group]; i < endGroups; ++i) {
+            
+            if (hasCustomRowGrouping) {
+                for (index_type group = 0; group < rowGroupIndices.get().size(); ++group) {
+                    endGroups = group < rowGroupIndices.get().size()-1 ? rowGroupIndices.get()[group+1] : rowIndications.size();
+                    for (index_type i = rowGroupIndices.get()[group]; i < endGroups; ++i) {
                         endRows = i < rowIndications.size()-1 ? rowIndications[i+1] : columnsAndValues.size();
                         // Sort the row
                         std::sort(columnsAndValues.begin() + rowIndications[i], columnsAndValues.begin() + endRows,
@@ -280,44 +282,22 @@ namespace storm {
                                               }));
                     }
                 }
-        }
-        
-        template<typename ValueType>
-        bool SparseMatrixBuilder<ValueType>::replaceColumns(std::vector<index_type> const& replacements, index_type offset) {
-            bool matrixChanged = false;
-            
-            // Walk through all rows.
-            for (index_type row = 0; row < rowIndications.size(); ++row) {
-                bool rowChanged = false;
-                index_type rowEnd = row < rowIndications.size()-1 ? rowIndications[row+1] : columnsAndValues.size();
-                
-                for (auto it = columnsAndValues.begin() + rowIndications[row], ite = columnsAndValues.begin() + rowEnd; it != ite; ++it) {
-                    if (it->getColumn() >= offset && it->getColumn() != replacements[it->getColumn() - offset]) {
-                        it->setColumn(replacements[it->getColumn() - offset]);
-                        rowChanged = true;
-                    }
-                    // Update highest column in a way that only works if the highest appearing index does not become
-                    // lower during performing the replacement.
-                    highestColumn = std::max(highestColumn, it->getColumn());
-                }
-                
-                if (rowChanged) {
-                    matrixChanged = true;
-                    
-                    // Sort the row.
-                    std::sort(columnsAndValues.begin() + rowIndications[row], columnsAndValues.begin() + rowEnd,
+            } else {
+                for (index_type i = 0; i < rowIndications.size(); ++i) {
+                    endRows = i < rowIndications.size()-1 ? rowIndications[i+1] : columnsAndValues.size();
+                    // Sort the row
+                    std::sort(columnsAndValues.begin() + rowIndications[i], columnsAndValues.begin() + endRows,
                               [](MatrixEntry<index_type, value_type> const& a, MatrixEntry<index_type, value_type> const& b) {
                                   return a.getColumn() < b.getColumn();
                               });
                     // Assert no equal elements
-                    STORM_LOG_ASSERT(std::is_sorted(columnsAndValues.begin() + rowIndications[row], columnsAndValues.begin() + rowEnd,
+                    assert(std::is_sorted(columnsAndValues.begin() + rowIndications[i], columnsAndValues.begin() + endRows,
                                           [](MatrixEntry<index_type, value_type> const& a, MatrixEntry<index_type, value_type> const& b) {
                                               return a.getColumn() <= b.getColumn();
-                                                    }), "Must not have different elements with the same column in a row.");
-                }
+                                          }));
                 }
 
-            return matrixChanged;
+            }
         }
 
         template<typename ValueType>

src/storm-dyftee.cpp

@@ -58,7 +58,7 @@ int main(int argc, char** argv) {
             parametric = true;
         } else if (option == "--expectedtime") {
             assert(targetFormula.empty());
-            operatorType = "ET";
+            operatorType = "T";
             targetFormula = "F \"failed\"";
             allowModular = false;
         } else if (option == "--probability") {

src/utility/storm.h

@@ -356,10 +356,10 @@ namespace storm {
             STORM_LOG_THROW(false, storm::exceptions::NotSupportedException, "The parametric engine currently does not support MDPs.");
         } else if (model->getType() == storm::models::ModelType::Ctmc) {
             // Hack to avoid instantiating the CTMC Model Checker which currently does not work for rational functions
-            if (formula->isExpectedTimeOperatorFormula()) {
+            if (formula->isTimeOperatorFormula()) {
                 // Compute expected time for pCTMCs
-                STORM_LOG_THROW(formula->asExpectedTimeOperatorFormula().getSubformula().isReachbilityExpectedTimeFormula(), storm::exceptions::NotSupportedException, "The parametric engine only supports Eventually formulas for this property");
-                storm::logic::EventuallyFormula eventuallyFormula = formula->asExpectedTimeOperatorFormula().getSubformula().asEventuallyFormula();
+                STORM_LOG_THROW(formula->asTimeOperatorFormula().getSubformula().isReachabilityTimeFormula(), storm::exceptions::NotSupportedException, "The parametric engine only supports reachability formulas for this property");
+                storm::logic::EventuallyFormula eventuallyFormula = formula->asTimeOperatorFormula().getSubformula().asReachabilityTimeFormula();
                 STORM_LOG_THROW(eventuallyFormula.getSubformula().isInFragment(storm::logic::propositional()), storm::exceptions::NotSupportedException, "The parametric engine does not support nested formulas on CTMCs");
 
                 // Compute goal states
@@ -368,7 +368,7 @@ namespace storm {
                 std::unique_ptr<storm::modelchecker::CheckResult> subResultPointer = propositionalModelchecker.check(eventuallyFormula.getSubformula());
                 storm::storage::BitVector const& targetStates = subResultPointer->asExplicitQualitativeCheckResult().getTruthValuesVector();
 
-                std::vector<storm::RationalFunction> numericResult = storm::modelchecker::helper::SparseCtmcCslHelper<storm::RationalFunction>::computeExpectedTimesElimination(ctmc->getTransitionMatrix(), ctmc->getBackwardTransitions(), ctmc->getExitRateVector(), ctmc->getInitialStates(), targetStates, false);
+                std::vector<storm::RationalFunction> numericResult = storm::modelchecker::helper::SparseCtmcCslHelper<storm::RationalFunction>::computeReachabilityTimesElimination(ctmc->getTransitionMatrix(), ctmc->getBackwardTransitions(), ctmc->getExitRateVector(), ctmc->getInitialStates(), targetStates, false);
                 result = std::unique_ptr<storm::modelchecker::CheckResult>(new storm::modelchecker::ExplicitQuantitativeCheckResult<storm::RationalFunction>(std::move(numericResult)));
 
             } else if (formula->isProbabilityOperatorFormula()) {