new graph: availability of wares

Bug #970840 reported by Angelo Locritani
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
widelands
Fix Released
Wishlist
Borim

Bug Description

based on idea of DaggeTeo in http://wl.widelands.org/forum/topic/1021/?page=1#post-7680 a new graph could be added: a grap showing how ware amount has changed over time, for every ware.
A simple approach would be using data reported in stock window (sum of wares in all warehouses)

Tags: statistics

Related branches

Revision history for this message
Chuck Wilder (chuckw20) wrote :

But what of those wares that go directly from producer to consumer and never enter a warehouse?

Revision history for this message
SirVer (sirver) wrote : Re: [Bug 970840] [NEW] new graph: availability of wares

So you basically suggesting also showing the integral of the currently
available graphs?

Revision history for this message
Angelo Locritani (alocritani) wrote :

@Chuck: as in stock window case, we could distinguish between all wares and stored ones, but I think that stored wares would be enough.

@Sirver: a new graph showing the amount of wares in warehouses. It can be considered as the change over time of data shown in stock windows, or the integral of the graph

tags: added: statistics
Revision history for this message
Borim (borim) wrote :

An interesting suggestion. I will give it a try ;)

Changed in widelands:
assignee: nobody → Borim (borim)
Borim (borim)
Changed in widelands:
status: New → In Progress
Revision history for this message
Borim (borim) wrote :

to move the discussion here:

SirVer oppinon:
>I am against this feature. It contains the same information as the
>economy health and therefore imho adds to clutter in the UI more than to
>the support of the player.
>
>I think it is unfortunate that this discussion was not done before you
>put work into it, Borim. However, we will need to finish this discussion
>now before merging.

Yeah you are right the stock chart represent nothing completely new, but it is another sight of view. The economy health represent a global view on the whole economy. The stock chart focus only on your storage. So the ammountof wares what was temporarly put aside.

The economy health show only the changes, but not the current state of your economy. The problem with the differentila view (economy health) that it is only usefull, when you know the current state.

A short example: When the economy health show that wood is always around the zero mark and only few times below zero, than this can be acceptable. You will not gain additional wood, but your stock does not shrink.
But if you simply does not own any wood reserves, this is a completely other situation. The demand is much higher than the production. The reason why the economy health chart show no negative balance, is simply nothing is there to be consumed. So the real demand is not visible in the economy health chart.

To bring this to the point: If the current stock is enough information to interprete the economy health chart (over the whole game lenth), the stock chart is not needed.

I think this is not the case. Especially if you look at a bigger time range. Also the request to show an indicator for the stock (Bug #584134 ) is a hint that the information is not sufficient.
With the stock chart you provide this information. And this not only by a simple arrow, but over the complete game length.

------------------------

The stock chart was not much work. So I am not angry if it does not make it into the game ;)

Revision history for this message
Patrick H. (daggeteo) wrote :

I hope this stock chart makes it into the game! Like Borim mentioned, it is another way of illustrating information that as of now is not available. And regarding the clutter issue, I can't see one more graph making it too cluttery. I am biased thou because I really want this feature and I have needed it many times when trying to figure out where to optimize my production and find needs.

Revision history for this message
SirVer (sirver) wrote :

2 vs 1 it is then. I was hoping for more opinions, but well. Let's merge this then. I will not be able to do this till end of may though. feel free to merge anybody.

Revision history for this message
SirVer (sirver) wrote :

Merged borims work in r6390.

Changed in widelands:
status: In Progress → Fix Committed
Changed in widelands:
milestone: none → build18-rc1
Revision history for this message
SirVer (sirver) wrote :

Released in build-18 rc1.

Changed in widelands:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.