インタプリタの定期休止の時間とステップ数をホストが指定できるようにする

[salano-ym]

一定ステップ毎にsleepを挟む仕組みがあって、現在はステップ数と休止時間は固定だが、ホストが設定できるようにしてもいいかもしれない？

aiscript/src/interpreter/index.ts

Line 16 in 9e61804

const IRQ_RATE = 300;

aiscript/src/interpreter/index.ts

Line 253 in 9e61804

if (this.stepCount % IRQ_RATE === IRQ_AT) await new Promise(resolve => setTimeout(resolve, 5));

[uzmoi]

休止時間の方はsetTimeout以外（requestIdleCallbackとか）も使えるようにPromiseを返す関数を渡すようにすると良いかも

[FineArchs]

休止時間の方はsetTimeout以外（requestIdleCallbackとか）も使えるようにPromiseを返す関数を渡すようにすると良いかも

利便性を考えると、数値だけでsetTimeoutの秒数を指定することもできると嬉しいです。
例えばinterpreterのオプションにするとして、

{
  irqCallback: () => Promise<void>;
}

か

{
  irqRate: number
}

のいずれかを指定できる（両方指定されるとエラー）という形がいいと思います。

[uzmoi]

easyよりsimpleのほうがいい気もしますが
irqRateはステップ数の方だと思うので、そうするならirqSleepとか……？

[FineArchs]

あっすみません、休止時間のこと忘れてました

{
  irqRate: number;
  irqSleep: number;
}

でどうでしょうか？

[FineArchs]

easyよりsimpleのほうがいい気もしますが

私としては、使用頻度が高くなるであろう設定を簡易に出来るようにする、という目的に限っては、ある程度簡便を図るべきであると考えています。
そうでないものは容易に内輪ネタと化してしまうからです。

[uzmoi]

まあこのくらいはできていいかもですね。

コールバックもirqSleepプロパティにしてnumber | (() => Promise<void>)型を受け付けると排他チェックが要らなくていいですかね

[FineArchs]

排他チェックがなくなる分、型チェックが増えるので性能としてはあまり変わらないような？
言葉の分かりやすさを考えるとirqSleepとirqCallbackは別々にしたほうが良いように思います。

[uzmoi]

排他チェックとかの実行コストは全体から見ればほぼゼロなので性能のことは気にしていなくて、排他なプロパティーがあるとオプションの複雑さが増して良くないかなと思って提案しました。
同じ目的のものを簡単な方法で指定できるだけなので、一つのプロパティで済むならそのほうが良いかなと。

[FineArchs]

うーん、一つのプロパティが型によって２つの意味を持つのもそれはそれで複雑な気がしますが…

[uzmoi]

どちらでもいつ再開するかを指定していて同じ意味だと思っています。
数値での指定は() => new Promise(resolve => setTimeout(resolve, irqSleep))のエイリアスなので。

[salano-ym]

どちらも一長一短ですが、休止にしか使用しないということであれば同じにする方が誤解が少ないような気がします。

[FineArchs]

うーんまあ確かに