search

JuliaDynamics / ABMFrameworksComparison

Benchmarks and comparisons of leading ABM frameworks
Other
13 stars 8 forks source link

Add ForestFire model for MASON #32

Closed Tortar closed 1 year ago

Tortar commented 1 year ago

This is a modified version of ForestFire which should be in line with the other implementations as discussed in #31.

One thing I'm doing (and I did also for the other models) is to have two different files for the small and the large versions, if you have any idea on how to make it possible to have only one file let me know, the fact is that I wasn't able to find a way to pass parameters from the bash benchmark file to the mason model so I specified them inside the class

Datseris commented 1 year ago

This PR deletes many files in other folders, is this intended?

No, there doesn't seem to be a way to have one file, because the MASON developers themselves also have set up two files. This seems to be one of the many clunkiness-es of MASON.

Tortar commented 1 year ago

yes, it's intended, they are compiled files which now are ignored in .gitignore, they are created when the benchmark run with javac command

Tortar commented 1 year ago

I think I missed a thing, here they are not using Manhattan distance, but I'm pushing the change

Tortar commented 1 year ago

@Datseris everything should be alright, so I'll merge if you approve it!

Tortar commented 1 year ago

Didn't look into it till now, I think it would be better to have it but I think it would take me a bit of time to understand and implement the video version, and I have to say I'm running out of time since I have some work to do for my Uni also in this moment :-)