pyflamegpu comparison #9
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added RTC init time to outputs. Requires a means to disable RTC cache, JitifyCache is not currently exposed via SWIG.
These methods have now been exposed to Python in FLAMEGPU master branch
Visualisation appears the same between FLAMEGPU implementations.
|
Py 3.8, CUDA 12.0, Mavericks (using FLAMEGPU/FLAMEGPU2#1153 branch, as it requires a fix for Schelling) I've stripped out the repeated Could setup alt bench script which forces RTC to use cache to show the difference. |
|
Trying to implement repast schelling in the flamegpu style isn't working. It's not well suited to agents communicating over distance without moving. Hence I need to rewrite the model to instead have static cell agents, and mobile bidding agents. The latter will move to bid for the cells, but they will only hold a state/team. |
Runs to completion on Waimu, but need to decide how to validate it.
+minor tweaks to pyfgpu boids
Remove space and store xy pos as an agent variable Init and print timing info only at rank 0 Write output csv to split log file
Not julia script does not log time of first run where model is compiled.
|
not doing mpi - treating as a separate thing. |
Tortar
reviewed
Apr 17, 2024
Comment on lines
+6
to
17
| using Random | ||
|
|
||
| # Does not use @bencmark, due to jobs being OOM killed for long-running models, with a higher maximum runtime to allow the required repetitions. | ||
| # Does not use @benchmark, due to jobs being OOM killed for long-running models, with a higher maximum runtime to allow the required repetitions. | ||
| # enabling the gc between samples did not resolve this BenchmarkTools.DEFAULT_PARAMETERS.gcsample = false | ||
| # Runs each model SAMPLE_COUNT + 1 times, discarding hte first timing (which includes compilation) | ||
| # Runs each model SAMPLE_COUNT + 1 times, discarding the first timing (which includes compilation) | ||
| SAMPLE_COUNT = 10 | ||
| SEED = 12 | ||
|
|
||
| # Boids | ||
| Random.seed!(SEED) | ||
| times = [] | ||
| for i in 0:SAMPLE_COUNT |
There was a problem hiding this comment.
I think the changes to this file can be removed when #10 is merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
mpi versionisHappy = (float(same_type_neighbours) / (same_type_neighbours + diff_type_neighbours)) > THRESHOLD