instead of referring to 'sales', shouldn't the autoMargin function refer to its 'data' parameter? Thanks.

Wow, yes, it totally should!! Good eye, thanks for reading carefully Jonathan! And sorry I missed this comment. Just fixed it.