Skip to content
Snippets Groups Projects

Implement addToMS for Split step

Merged Implement addToMS for Split step
splitaddtoms into master
Merged Tammo Jan Dijkema requested to merge splitaddtoms into master

Created by: tammojan

Fixes #153 (closed)

Merge request reports

Approval is optional

Merged by avatar (Mar 14, 2025 5:08am UTC)

Merge details

  • Changes merged into master with cad0ee7c (commits were squashed).
  • Deleted the source branch.

  • Closed #153 (closed)

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tammo Jan Dijkema
    Tammo Jan Dijkema @dijkema started a thread on commit cad0ee7c
    • DPPP/Split.cc
    102 103 // Show the steps.
    103 104 for (unsigned int i=0; i<itsSubsteps.size(); ++i) {
    104 105 os << "Split substep "<<(i+1)<<" of "<<itsSubsteps.size()<<endl;
    105 DPStep::ShPtr step = itsSubsteps[0];
  • Tammo Jan Dijkema
    Tammo Jan Dijkema @dijkema started a thread on commit cad0ee7c
    • DPPP/Split.cc
    102 103 // Show the steps.
    103 104 for (unsigned int i=0; i<itsSubsteps.size(); ++i) {
    104 105 os << "Split substep "<<(i+1)<<" of "<<itsSubsteps.size()<<endl;
    105 DPStep::ShPtr step = itsSubsteps[0];
    • Tammo Jan Dijkema
      Tammo Jan Dijkema @dijkema ·
      Author Owner

      Created by: tammojan

      Yes it was; ideally should have been fixed in a separate commit. Fairly minor though: initially this showed only the first substep to save on output. I added the loop to show all substeps, but instead showed the first substep n times..

    Please register or sign in to reply