Error in piecing-it-together instructions

glennj · March 21, 2026, 5:30pm

The instructions define aspectRatio as

ratio of columns to rows

Then the first example, Portrait, says

It has 3 rows and 2 columns. The aspect ratio is 1.5 (3/2)

That’s inverse: the ratio of columns:rows with 2 columns and 3 rows is 2/3 (0.66…)

exercism/problem-specifications/blob/main/exercises/piecing-it-together/instructions.md

# Instructions

Given partial information about a jigsaw puzzle, add the missing pieces.

If the information is insufficient to complete the details, or if given parts are in contradiction, the user should be notified.

The full information about the jigsaw puzzle contains the following parts:

- `pieces`: Total number of pieces
- `border`: Number of border pieces
- `inside`: Number of inside (non-border) pieces
- `rows`: Number of rows
- `columns`: Number of columns
- `aspectRatio`: Aspect ratio of columns to rows
- `format`: Puzzle format, which can be `portrait`, `square`, or `landscape`

For this exercise, you may assume square pieces, so that the format can be derived from the aspect ratio:

- If the aspect ratio is less than 1, it's `portrait`
- If it is equal to 1, it's `square`

This file has been truncated. show original

BNAndras · March 21, 2026, 7:49pm

Yeah, aspect ratio is width (columns) to height (rows) so that example is wrong as you noted. +1 from me for a fix

BethanyG · March 21, 2026, 8:20pm

+1 from me for a fix as well.

IsaacG · March 21, 2026, 10:30pm

Happy to see a PR

kotp · March 22, 2026, 3:16am

That’s another +1 from me.

glennj · March 22, 2026, 3:58am

BNAndras · March 22, 2026, 8:14pm

PR was merged.