• ###### fenring76commented on "Calculate Kata Satisfaction" kata

FArekkusu,

No one would use the term 'decimal number' if they intended to be that precise in the first place. A 'decimal number' is most commonly referring to a number with a decimal point. If it's referring to any base-10 number, it's already sort-of wrong even though such numbers are part of the decimal system—we have more accurate terms for that.

That the author shouldn't claim that "this is how it works on CW" when his explanation (and possibly understanding) is wrong?

Representing it as an accurate picture of how it works is a different problem than whether it is actually an accurate picture of how it works. This is a fine criticism, but you've moved the goalposts. Additionally, the author never said what you've attributed to him (or her) in your quotation. What the author did say seemed accurate (he never said his calculation method was the way it is done), but if it wasn't, it is probably more helpful to point him to a resource that explains it or explain it yourself than to offer unearned condemnation of his kata idea.

Of course, understanding all of this requires some subtlety. It is neither a criticism nor a compliment when I say I don't see you as a particularly subtle person.

Maybe the solution here is to rename the kata "An alternative calculation of Kata Satisfaction."

• ###### fenring76commented on "Calculate Kata Satisfaction" kata

I normally agree with your criticisms of kata, but yours is an odd post this time.

This is supposed to be "a floating point number", "not an integer", or anything in the same spirit, but definitely not "decimal".

I agree the phrasing could be made clearer, perhaps by saying something like "All values should be rounded down to the nearest whole number/integer." But even then we are losing the information that the value should not be represented as a decimal.

A "decimal" is very much in the same spirit as 'floating-point number.' A decimal is a representation of any real or integer using a decimal point. A floating-point number is a representation of a decimal number on a computer. Not all languages even use float as a type and in Python you can certainly represent such numbers using the decimal module.

"Not an integer" wouldn't work here. 1.0 is a decimal (and floating-point) representation of the integer 1. 1.0 is not an int, but it is an integer.

That's not how it really works...
This is not it works on CW either...

Okay, but what is the actionable take-away from these criticisms? There are a lot of kata that don't work 'how it really works.'

I agree with your other two criticisms. Specifically:

• What counts as 'irrelevant' should be made clear
• How to actually calculate the results should be specified.
• ###### fenring76created an issue for "Calculate Kata Satisfaction" kata

If the answer is a decimal, the answer should be rounded to the whole number

Tests require that the number is specifically rounded down (or floored or truncated). "Rounded to the whole number" is ambiguous.

If there are >= 4 consecutive "Not Satisfied", your function should return “Retired!”

While the Kata Description does specify that bad input should be ignored, it would be clearer if this requirement specified that four consecutive "Not Satisfied" trigger this excluding bad/ignored input. Relatedly, while the kata only gives bad strings as input, it wouldn't hurt to either make that an official constraint or to test for non-string bad input. (This part is not really an issue, just a suggestion.)

• ###### fenring76commented on "DevOps legacy roasting -> disco inferno -> burn baby burn" kata

I agree that the Kata Description is ambiguous as to whether the set of complaints is complete. It is not explicitly stated that the set is complete rather than exemplary nor is an implication of the same necessitated by anything in the Kata Description.

• ###### fenring76commented on "Format the playlist" python solution

That line lambda is great. Nice job!

• ###### fenring76commented on "Simple Fun #206: Factor Digits" python solution

Thank you for marking this as spoiler for me. I try to remember to use the spoiler flag when possible, but this is a UI issue, not a user issue.

I suggest comments should not be visible from the dashboard by default, should only be visible if the user has access to the solution, or comments to solutions should have the spoiler flag enabled by default, possibly allowing the user to disable it or not.

• ###### fenring76commented on "Simple Fun #206: Factor Digits" python solution

This comment is hidden because it contains spoiler information about the solution

• ###### fenring76commented on "Will you survive the zombie onslaught?" python solution

This comment is hidden because it contains spoiler information about the solution

• ###### fenring76commented on "Sum of angles" python solution

This comment is hidden because it contains spoiler information about the solution

• ###### fenring76commented on "Did she say hallo?" python solution

This comment is hidden because it contains spoiler information about the solution

• ###### fenring76commented on "Binary operations #1" python solution

^ is bitwise exclusive or

It compares corresponding bits in two numbers. It sets the corresponding bit in the output to 0 if the two corresponding input bits are the same and 1 if the bits are different.

X Y X ^ Y
0 0 0
0 1 1
1 0 1
1 1 0
• ###### fenring76commented on "Nth Perfect Number" kata

@FArekkusu

I think it's just another way of stating the issue you raised.

• ###### fenring76created an issue for "Convert Def To Lambda" kata

Random tests include function definitions which begin with a digit.

Example:

def 8QPB0Nzjhh(C):
return C(C - 4)

Identifiers in Python, including function names, cannot begin with a digit.

• ###### fenring76commented on "Read the time" kata

The real reason is that it wasn't changed two years ago when I made the suggestion...

• ###### fenring76commented on "Numbers without numbers" python solution

This comment is hidden because it contains spoiler information about the solution