do we definitely want ring.zero here and not ring.one?