fix: solarViz attributeError with component arg#286
fix: solarViz attributeError with component arg#286disgruntled-penguin wants to merge 3 commits intomesa:mainfrom
Conversation
|
Thank @disgruntled-penguin, I need to look into why the tests are failing this is a likely a repo name change issue. However, could you elaborate on the specific bugs and show examples of the failure. Please also provide your dependency list (e.g., what version of Mesa you are using) Thank you again for your support of Mesa. |
|
hello @tpike3 no problem! this was the error message when |
Thanks @disgruntled-penguin, try using mesa <= 3.3 -- there is a known issue with mesa-geo that it needs to be pinned to Mesa < 3.3. |
|
Thanks @disgruntled-penguin for the fix. Changing components to keyword argument would indeed fix the error. However I didn't merge this PR because there's another commit about the game of life model, that I'm unsure what it was about. For all GIS models I have created a separated PR to use keyword argument for soalra viz components: #311 |
|
This might be a good argument why we should require keyword only arguments in more places in Mesa (see mesa/mesa#2972 (comment)). |


This PR fixes a small bug (
AttributeError: 'list' object has no attribute 'value') that was preventing it from rendering. Explicitly statingcomponents=fixed it.