Opened 11 years ago

Closed 3 years ago

Last modified 3 years ago

#2307 closed defect (fixed)

Using cat in loops gives strange results

Reported by: joafa554 Owned by: probably noone
Priority: low Milestone: 1.16.0
Component: New Instantiation Version: trunk
Keywords: Cc:

Description

function toFlatArray
    input Real[:,:] m;
    output Real[:] arr := fill(0, 0);
algorithm
    for i in 1:size(m, 1) loop
        arr := cat(1, arr, m[i,:]);
    end for;
end toFlatArray;

The resulting array only contains the last part for some reason:

>> toFlatArray({{1,2,3},{4,5,6}})
{4.0,5.0,6.0}

Replacing the for loop with a while loop yield the same result.

Change History (12)

comment:1 Changed 10 years ago by sjoelund.se

  • Milestone changed from 1.9.0 to 1.9.1

Postponed until 1.9.1

comment:2 Changed 9 years ago by sjoelund.se

  • Milestone changed from 1.9.1 to 1.9.2

This ticket was not closed for 1.9.1, which has now been released. It was batch modified for milestone 1.9.2 (but maybe an empty milestone was more appropriate; feel free to change it).

comment:3 Changed 9 years ago by sjoelund.se

  • Milestone changed from 1.9.2 to 1.9.3

Milestone changed to 1.9.3 since 1.9.2 was released.

comment:4 Changed 9 years ago by sjoelund.se

  • Milestone changed from 1.9.3 to 1.9.4

Moved to new milestone 1.9.4

comment:5 Changed 8 years ago by sjoelund.se

  • Milestone changed from 1.9.4 to 1.9.5

Milestone pushed to 1.9.5

comment:6 Changed 8 years ago by sjoelund.se

  • Milestone changed from 1.9.5 to 1.10.0

Milestone renamed

comment:7 Changed 7 years ago by sjoelund.se

  • Milestone changed from 1.10.0 to 1.11.0

Ticket retargeted after milestone closed

comment:8 Changed 7 years ago by sjoelund.se

  • Milestone changed from 1.11.0 to 1.12.0

Milestone moved to 1.12.0 due to 1.11.0 already being released.

comment:9 Changed 6 years ago by casella

  • Component changed from Backend to Command Prompt Environment
  • Milestone changed from 1.12.0 to Future
  • Priority changed from high to low

comment:10 Changed 6 years ago by sjoelund.se

Seems to be due to us flattening to output Real[0] arr = {};

comment:11 Changed 3 years ago by perost

  • Component changed from Command Prompt Environment to New Instantiation
  • Resolution set to fixed
  • Status changed from new to closed

This works fine with the new frontend.

comment:12 Changed 3 years ago by casella

  • Milestone changed from Future to 1.16.0
Note: See TracTickets for help on using tickets.