looks good I'd rather just have `ms = ms || 1e9`