5 kyu
Pandigital Sequence
48 of 175FilipH
Loading description...
Fundamentals
Algorithms
Mathematics
This comment has been reported as {{ abuseKindText }}.
Show
This comment has been hidden. You can view it now .
This comment can not be viewed.
- |
- Reply
- Edit
- View Solution
- Expand 1 Reply Expand {{ comments?.length }} replies
- Collapse
- Spoiler
- Remove
- Remove comment & replies
- Report
{{ fetchSolutionsError }}
-
-
Your rendered github-flavored markdown will appear here.
-
Label this discussion...
-
No Label
Keep the comment unlabeled if none of the below applies.
-
Issue
Use the issue label when reporting problems with the kata.
Be sure to explain the problem clearly and include the steps to reproduce. -
Suggestion
Use the suggestion label if you have feedback on how this kata can be improved.
-
Question
Use the question label if you have questions and/or need help solving the kata.
Don't forget to mention the language you're using, and mark as having spoiler if you include your solution.
-
No Label
- Cancel
Commenting is not allowed on this discussion
You cannot view this solution
There is no solution to show
Please sign in or sign up to leave a comment.
python new test framework
Quite funny and mathematical kata.
This comment has been hidden.
This comment has been hidden.
The tests in the python 3.6 version ignore the offset. (Sample test work fine, the actual tests need fixed)
The current Elixir reference solution (brute force) is not viable. It times out before it can complete a single edge case such as
(4987653201,20), or(2987654301,10), etc.you're right, I'll add some more tests and rewrite my solution thanks
Nice kata. A couple recommendations:
50or greater(4987653201,20)When I initially approached this problem, I misinterpreted
offsetto ben-th iteration, rather than starting from the value>= offset. The Description is fine; I just misread. I bring this up to gauge interest in a kata similar to this, but whereoffsetrefers to then-th iteration offset.This comment has been hidden.
For the Python translation, the failed test message isn't displaying as intended:
someone forgot the
fat the beginning... x)Right thanks :) fixed it now and learned something new in python
Any plans on other translations?
I'll try put some effort in it soon. Any favorites from your side?
I was planning on Python, Elixir, Kotlin, JavaScript and maybe some C.
Python and JS are among two of the most widely used languages on CW, so that's something to consider if you want to draw more users. I think this is a good kata challenge, especially for a functional language; I wrote a solution in Elixir a little while back so if you write it in Elixir that would be cool, too.
May be helpful to include at least one sample test involving a negative number as the
offsetvalue.thanks for the response; will try to add some soon
This comment has been hidden.
thanks for your reply, Andrew I added an extra line in the rules for this
Added some specific and formatted description to clarify.
Current there is only 1 random test, which is far from enough.
Usually 100 random tests are needed ;-)
Hi,
Ok thanks I'll add some addititional random tests.
I just modified description and test cased. Plz submit again :)
What is
offset? What are its specs? I see things from0to big positive numbers to big negative numbers in the tests.Hi
Like you noticed, the offset can be any number. Positive or negative, big or small. The only constraints is in the pandigital numbers itself. I'll add some additional explaining to the details for this.
Needs Random Tests.
o_O
I just modified description and test cased. Plz submit again :)