Skip to content

Fix issue #2191:I have moved the examples files from https://github.c… #159

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

Closed
wants to merge 2 commits into from

Conversation

VinayakPaka
Copy link

I have moved the examples files from mesa to mesa-examples.

If u need anything else to update. Drop a message I will solve that.

@EwoutH
Copy link
Member

EwoutH commented Aug 14, 2024

Thanks for the PR. I clearly should have described the issue better.

I'm going to give @jayash1973 with #152 the initiative for a week or so, so putting this PR on hold for that time.

@jayash1973
Copy link

hey @EwoutH is there any other place where the discussions take place for Mesa?

@jayash1973
Copy link

@EwoutH there are some errors in the current code in the https://github.com/projectmesa/mesa/tree/main/mesa/experimental/devs/examples

should i fix them and also add solara visualization to them all in one file insted of 3 files like agents.py, app.py and model.py like i did before. after the changes are made shold we copy and transfer them after that? how does that sound?

@EwoutH
Copy link
Member

EwoutH commented Aug 15, 2024

is there any other place where the discussions take place for Mesa?

Yes, two places:

all in one file insted of 3 files like agents.py, app.py and model.py like i did before

No. I updated projectmesa/mesa#2191 to be a bit more clear (hopefully). Feel free to ask any questions about that there.

@EwoutH
Copy link
Member

EwoutH commented Aug 19, 2024

@VinayakPaka if you want to move forward with this PR go ahead. Try to make atomic commits for each step in the process.

To manage expectations, I don't have much time to review, so try to follow projectmesa/mesa#2191 as closely as possible. @jayash1973 has done some work on Readme's and the visualisation, maybe he will allow you to use those.

@EwoutH EwoutH marked this pull request as draft August 19, 2024 14:31
@EwoutH
Copy link
Member

EwoutH commented Aug 29, 2024

Closing for now due to lack of progress.

@EwoutH EwoutH closed this Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants