Refactor market simulation to be more readable and set the start conf…

[tomschw]

…iguration to be computable in less than 1 minute.

[Bouncner]

Have you tested the whole pipeline?

I wont be able to test it until next week. But I would trust you and merge when you say you tested the whole pipeline. ;)

[tomschw]

Except for a small error in the monte_carlo file, everything seems to work as expected and the graphs don't seem abnormal.
One current problem is, that the demand_learning notebook wants to load the S1 notebooks, which currently aren't getting created. If we keep it like that, it would probably be wise to simply run all notebooks with every strategy, simply so we can expect them. I found an easy way to do, which can be found in issue #8 .

[Bouncner]

Then please change demand_learning to use the scenario that we are currently creating as the default.

Which bug do you mean?

[tomschw]

I already use the current scenario as default but in demand_learning we compare the two scenarios at the end. Shall I comment it out for the moment?

The problem was that I didn't remove all unnecessary output, which was only used when Rainer still calculated the logit in the notebook. I will commit the changed notebook at the end of today.

[Bouncner]

I see your point.

Let’s ignore that issue for now, as I think it’s clear that users have to generate both conpare scenarios.

I’ll review after your fix.

[tomschw]

fixed

[Bouncner]

Please commit a cleared or ran-once notebook and I’ll merge.

[tomschw]

Done