User talk:SqrtNegInf: Difference between revisions

From Rosetta Code
Content added Content deleted
Line 43: Line 43:


Doh. Good catch on the Range consolidation task. I cut out a bunch of logic dealing with excluded boundaries and completely missed the the enclosed range case. To be fair, there weren't any test cases that exercised that or I probably would have noticed. Thanks. --[[User:Thundergnat|Thundergnat]] ([[User talk:Thundergnat|talk]]) 23:30, 5 February 2019 (UTC)
Doh. Good catch on the Range consolidation task. I cut out a bunch of logic dealing with excluded boundaries and completely missed the the enclosed range case. To be fair, there weren't any test cases that exercised that or I probably would have noticed. Thanks. --[[User:Thundergnat|Thundergnat]] ([[User talk:Thundergnat|talk]]) 23:30, 5 February 2019 (UTC)

: Actually, there is: <tt>[-6, -1], [-4, -5]</tt>, otherwise I doubt I would have found the problem. I don't know if Paddy3118 was clever or lucky in using that pairing. Your output as shown was correct, so I figured an earlier version worked, but the urge to stream-line the code caught up with you...

Revision as of 00:13, 6 February 2019

Talking to myself... --SqrtNegInf (talk) 18:34, 12 April 2018 (UTC)

RC-run

Hey, when you get a chance, could you give the latest version of RC-run (April 24) a try? It is especially useful with the -d & -b flags set IMO. If you have any observations or suggestions, I'd be interested. --Thundergnat (talk) 00:49, 25 April 2018 (UTC)

I tried to exercise all relevant combinations of command-line options, and everything is working as advertised. The one problem I found was when I ran multiple tasks from a file, after 'Bitmap_B_zier_curves_Cubic', all subsequent tasks ended up rooted below the first, e.g. 'rc/perl6/Bitmap_B_zier_curves_Cubic/rc/perl6/Card_shuffles/'; for some reason the 'chdir $current' didn't work. One suggestion: If the value of '$view' could be set via the command-line, users on other platforms could more easily view graphical results. --SqrtNegInf (talk) 12:58, 25 April 2018 (UTC)
Update: The two items mentioned above were related... When the 'view' command failed in the 'run-it' sub, it exited early, 'chdir' didn't happen. When I set $view as something innocuous, /usr/bin/file, then things were fine. --SqrtNegInf (talk) 18:54, 25 April 2018 (UTC)
Doh! Thinko. Line 169 should be .resume instead of return.--Thundergnat (talk) 19:11, 25 April 2018 (UTC)

Hofstadter-Conway $10,000 sequence / Perl 6

Removed the 'fast' (and less elegant) alternative version as it no longer has any performance advantage with the current release of Rakudo. At the time of the initial 6.c release (2015.12) there was a nearly 5-fold difference in run times.

Thanks

I wanted to say thanks for all the new the Perl and Perl 6 task implementations you've been writing. Love it! Danaj (talk) 17:40, 30 October 2018 (UTC)

File IO

Thanks for finding and fixing the Perl 6 file IO errors, my time is pretty limited for the near future. I have to say though, I feel a little less clueless about making that error / assumption when I see who added the entry for Last_letter_first_letter. --Thundergnat (talk) 14:44, 20 November 2018 (UTC)

Thanks for the advice

>> When you add/edit a task, put a short summary of what you've done

I will manage future edits accordingly.--Hkdtam (talk) 10:14, 3 December 2018 (UTC)

Thanks for fixing my missing tag

Arrg. Didn't notice the missing closing div tag in Count Examples. It looked fine in preview, I got called away just as I submitted and didn't double check. Thanks. --Thundergnat (talk) 02:19, 22 January 2019 (UTC)

Thanks - RE: Problems with 'Addition chains' task

>> Can you double-check that the 'Addition chains' task

Oh thank you very much for pointing it out. I don't why that two less than/greater than equal signs got change to bare equals during copy and paste. Anyway thanks again. --Hkdtam (talk) 16:37, 4 February 2019 (UTC)

Range consolidation

Doh. Good catch on the Range consolidation task. I cut out a bunch of logic dealing with excluded boundaries and completely missed the the enclosed range case. To be fair, there weren't any test cases that exercised that or I probably would have noticed. Thanks. --Thundergnat (talk) 23:30, 5 February 2019 (UTC)

Actually, there is: [-6, -1], [-4, -5], otherwise I doubt I would have found the problem. I don't know if Paddy3118 was clever or lucky in using that pairing. Your output as shown was correct, so I figured an earlier version worked, but the urge to stream-line the code caught up with you...