-
Notifications
You must be signed in to change notification settings - Fork 877
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
Move examples into mesa #2387
Move examples into mesa #2387
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The move makes sense and should resolve the various issues we have been running into. Thanks!
Performance benchmarks:
|
655ecf3
to
eddb8f8
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, luckily for git its just a rename operation, since git doesn't know about the concept of folders. |
Based on previous discussion and to rectify some errors made in #2381
See commits for atomic changes:
__init__.py
file to each basic example. While I find this a bit annoying, I think its actually the best way to make the examples truly portable, combined with relative imports. Each example becomes its own "package" in python speak. Also updated the imports so now you can dofrom mesa.examples import BoltzmannWealthModel
instead offrom mesa.examples.basic import BoltzmannWealthModel