Skip to content
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

Server failing when requesting non-existent step #33

Closed
dmasad opened this issue Oct 30, 2014 · 1 comment
Closed

Server failing when requesting non-existent step #33

dmasad opened this issue Oct 30, 2014 · 1 comment
Labels
bug Release notes label

Comments

@dmasad
Copy link
Member

dmasad commented Oct 30, 2014

Currently, nothing stops the server from requesting a step which exceeds the range of model steps created; this causes the server to fail. Should be very easy to fix.

@dmasad dmasad added the bug Release notes label label Oct 30, 2014
@dmasad dmasad mentioned this issue Oct 30, 2014
dmasad added a commit that referenced this issue Nov 1, 2014
Fixing issue #33, bug which caused the server to crash when an
out-of-range step was requested.
dmasad added a commit that referenced this issue Nov 7, 2014
A very simple ASCII server that renders the text visualization in the
browser.

Cleaned up the ascii server

Added a real websocket protocol, fixed the ugly template path hack, and
implemented resetting.

Minor tweak

Added some documentation

add tornado

Style cleanup

Cleaning up some of the code following @kfuruya review.

Fixed out-of-range step bug

Fixing issue #33, bug which caused the server to crash when an
out-of-range step was requested.

Implementing Play button

Adding a play button, which runs the model at the set frame rate until
it hits the end.

Tabs to spaces

Bringing it in line with the master...
@dmasad
Copy link
Member Author

dmasad commented Nov 21, 2014

Oh hey I resolved this at some point.

@dmasad dmasad closed this as completed Nov 21, 2014
EwoutH pushed a commit to EwoutH/mesa that referenced this issue Oct 15, 2024
…a#33)

- update maybe_sell_spice() so calculate_MRS() takes potential sugar and spice based on trade
- update calculate_MRS() with kwargs so it can take potential trade but defaults to agents sugar and spice
Update increases agent_reporter due to number of trades creating memory issues
- Update data collection so non-relevant data rmeoved on each step (i.e. sugar and spice agents don't have trade partners)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Release notes label
Projects
None yet
Development

No branches or pull requests

1 participant