Thomas Eizinger
Thomas Eizinger
> cargo test shows me log for the integration tests. It's enough for me to run `cargo test flappy_ice_lite_state` to get my reasonable defaults. cargo's default behaviour is to not...
> > cargo's default behaviour is to not show logs if the test is passing. To override that, you need to pass `--nocapture`. Taken that into account, running `cargo test...
> Could we filter them instead? > > As I see it, `Candidate` should not have a state. The discarded flag is probably there or convenience, but we don't want...
> I think because we use usize indexes to the vec holding them, we can't move them out easily. We could make some wrapper Entry type to hold the state...
A few more thoughts: - Candidate pairs on the same relay should be prioritzed over pairs on different relays to reduce latency (1 less hop). If we find a candidate...
> > I think testing host relay candidates is very unlikely to be useful. We may even want to drop those candidate pairs altogether? TURN servers are deployed to the...
> > Candidate pairs on the same relay should be prioritzed over pairs on different relays to reduce latency (1 less hop). If we find a candidate pair on the...
> Or taking a step back, why would removing any pairs be an advantage? Less noise? - Less noise - Less resources being consumed (i.e. TURN channels can be expired...
Relevant: https://github.com/algesten/str0m/pull/476
> Relevant: #476 For anybody following along, the issue turned out to be a combination of: - Not refreshing our local timer from `poll_timeout` after changing the agent's state (i.e....