search

assume-framework / assume

ASSUME - Agent-based Simulation for Studying and Understanding Market Evolution
https://assume.readthedocs.io
20 stars 5 forks source link

fix bugs in tutorial 6 #324

Closed adamsjohanna closed 4 months ago

adamsjohanna commented 4 months ago

When running in Colab, the Tutorial 6 returns Warnings when running the simulation with flexableEOM strategy: "WARNING:pyomo.repn.plugins.lp_writer:EXPORT Suffix 'dual' found on 1 block: dual LP writer cannot export suffixes to LP files. Skipping."

This is not fixed yet! Does anyone know how to fix this? I run pip install pyomo in the beginning of the notebook. It does not help to install it again...

Also the visualization did not return the wanted output. This is fixed by this PR.

codecov[bot] commented 4 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.42%. Comparing base (3d44db7) to head (5e943b1).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #324 +/- ## ======================================= Coverage 77.42% 77.42% ======================================= Files 43 43 Lines 4996 4996 ======================================= Hits 3868 3868 Misses 1128 1128 ``` | [Flag](https://app.codecov.io/gh/assume-framework/assume/pull/324/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework) | Coverage Δ | | |---|---|---| | [pytest](https://app.codecov.io/gh/assume-framework/assume/pull/324/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework) | `77.42% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=assume-framework#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

nick-harder commented 4 months ago

@adamsjohanna I believe this is a pure collab problem and the glpk solver that is used there. I didn't figure out a way to surpress this warning. But it is fine, everything is still working I believe, or?

adamsjohanna commented 4 months ago

Yes, with the few fixes for the visualization, it is running :-)