<whitequark[cis]>
I'll take a look at the rest later, or perhaps someone else will
<mcc111[m]>
<whitequark[cis]> "add_sync_process is deprecated" <- then i will try to remove it
<mcc111[m]>
the docs for add_process say "at each point in time, all of the non-waiting processes are executed in an arbitrary order that may be different between individual simulation runs"
<mcc111[m]>
should i understand that to mean that *within* an individual simulation run, the arbitrary order is stable
Wanda[cis] has joined #amaranth-lang
<Wanda[cis]>
it... technically is, but we do not guarantee it
<whitequark[cis]>
that is not meant to be a contract that is provided
<whitequark[cis]>
but the current implementation does happen to make it stable
<mcc111[m]>
hm, ok.
<mcc111[m]>
it is my recommendation that you change the wording in the docs from "be different between individual simulation runs" to "vary between (or within) individual simulation runs"
<mcc111[m]>
would you agree with this, and would it be helpful if i made an issue and/or PR