-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: Allow random and defined activity simulation #161
Comments
Can you provide an example of how the file would look for this? |
The following sim file would:
Not sure that this is the best way to express it tbh, I specifically want to keep the feature that only specifying |
I'm trying to think about the implications that this will have in the codebase. Would the I guess in the case of the former we can simply filter the node collection to remove whatever is in All in all, I think it makes sense, it should be less work to define an exclusion list as far as we assume that the default behavior for channels is to be sending payments between them instead of being stale (which it's a reasonable assumption for me at least) |
Both - our current implementation of random activity is that each node sends and receives, so we'd just exclude
Yeah exactly what I was thinking 👍 |
Right now, the type of activity that we will simulate is determined by the
sim-file
:activity
specified: run the user-defined set of activities.activity
: run random activity for all of thenodes
provided.We should allow users to specify both random and defined activity to support more complicated use cases, while still preserving the "easy start" nature of just being able to specify
nodes
for random activity.Suggested Sim File Interpretation
Random Activity - All Nodes
sim.json:
Action: run default random activity on A and B.
Defined Activity
sim.json:
Action: run only the defined activity specified between A and B.
Defined and Random Activity
sim.json:
Action: run defined activity specified between A and B and random activity between A and C.
Rationale for
no_random
is that I think the likely use case is specifying a few specificactivity
flows and then wanting those nodes to be excluded from the random activity.Eg: Running a jamming simulation
D
andE
to run a jamming attack.D
andE
to send random payments.D
andE
inno_random
.The text was updated successfully, but these errors were encountered: