The original author doesn't seem to be around, but I've forked this translation such that

• aspect_ratio now returns a Tuple[int, int] instead of a List[int]. The justification for this is that aspect_ratio alwasy returns a pair of two ints, instead of a list of an unknown amount of ints.
• Changed the format of the test headers slightly
• Created a random instance for tests to use before importing the user solution so that users cannot modify the random instance.

You should use suggestions for comments which suggest changes to the Kata, not for general questions. Without looking at your code, my assumption is that you create a vector of which increases in size as the value of the argument increases. The solution is to not do that.

Are you using Node v8.1.3? This issue doesn't seem to occur in Node v10 or newer.

Python Translation (Author inactive)

Python translation

The JavaScript version (and possibly the Haskell translation if based off the original tests) does not test that the user solution considers the first term a to be part of the sequence, and as such the Kata can be passed with solutions that do not consider a to be part of the sequence.

EDIT: For more context, view my solution which only checks from the second term.

Is it possible you can add a fixed test and/or random tests which test for the first term in the sequence? It's currently possible to pass the JS version with solutions that don't check for the first term.

Python Translation (Author inactive)

Please review this forked Python Translation.

Forked

LGTM

C Translation Kumited, please accept.

.