Nice work on these sine and cosine functions -- however, `j` and `k` are not very informative variable names. I am not sure what their purpose is, just from looking at the code. In general, we would like to avoid one-letter variable names for this reason.