Your definition of causality seems sufficient, but you may want to mention in your definition of non-causal if the outputs are only affected by future inputs or if past and present inputs can affect the output as well.
