Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Generating Pascal's triangle with lazy list generator fails (integration/advent2012-day04.t) #3788

Closed
p6rt opened this issue Apr 18, 2015 · 4 comments

Comments

@p6rt
Copy link

p6rt commented Apr 18, 2015

Migrated from rt.perl.org#124330 (status was 'resolved')

Searchable as RT124330$

@p6rt
Copy link
Author

p6rt commented Apr 18, 2015

From @usev6

There is a spectest failure in integration/advent2012-day04.t (test 3, "Problem 53", generating Pascal's triangle).

< bartolin_> m​: sub pascal { [1], { [0, @​^p Z+ @​^p, 0] } ... * }; .say for pascal[^10];
<+camelia> rakudo-moar a24488​: OUTPUT«1␤1␤1␤1␤1␤1␤1␤1␤1␤1␤»
< bartolin_> star-m​: sub pascal { [1], { [0, @​^p Z+ @​^p, 0] } ... * }; .say for pascal[^10];
<+camelia> star-m 2015.03​: OUTPUT«1␤1 1␤1 2 1␤1 3 3 1␤1 4 6 4 1␤1 5 10 10 5 1␤1 6 15 20 15 6 1␤1 7 21 35
  35 21 7 1␤1 8 28 56 70 56 28 8 1␤1 9 36 84 126 126 84 36 9 1␤»
< bartolin_> ^^ that's from http://rosettacode.org/wiki/Pascal%27s_triangle#Perl_6 (and
  integration/advent2012-day04.t)
< bartolin_> I remember breaking it a month ago (by improperly flattening values within SEQUENCE). but it
  was fixed afterwards.
< bartolin_> seems to fail since 2015-04-10

There was a rewrite of the infix​:<...> operator on 2014-05-09, probably causing the changed behaviour (cmp. https://github.com/rakudo/rakudo/commits/nom/src/core/operators.pm).

@p6rt
Copy link
Author

p6rt commented Apr 19, 2015

From @usev6

This was fixed by TimToady++. The test is passing again.

I added two tests to S03-sequence/basic.t which should cover the underlying problem (containers returned from the seq iterator were not respected)​: Raku/roast@2be443bf3f

I'm closing this ticket as 'resolved'.

1 similar comment
@p6rt
Copy link
Author

p6rt commented Apr 19, 2015

From @usev6

This was fixed by TimToady++. The test is passing again.

I added two tests to S03-sequence/basic.t which should cover the underlying problem (containers returned from the seq iterator were not respected)​: Raku/roast@2be443bf3f

I'm closing this ticket as 'resolved'.

@p6rt p6rt closed this as completed Apr 19, 2015
@p6rt
Copy link
Author

p6rt commented Apr 19, 2015

@usev6 - Status changed from 'new' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant