selaux / miner-dashboard

Node.js based app to show the current status of your miner in a browser.
36 stars 15 forks source link

New Revenue Config formatting issues? #55

Closed jedimstr closed 10 years ago

jedimstr commented 10 years ago

So I reformatted my config file for the new revenue module style, but now the Revenue Module only shows the yellow bar'd No Data Yet. Old style configuration works instantly on startup.

Here's my updated config... I tried with and without the undocumented title, but no go:

{
                title: 'Revenue Estimate (BTC Pools Only)',
                module: 'revenue/solo',
                miners:[
                   {
                        miner: [
                                'galactica',
                                'pegasus',
                                'atlantia',
                                'valkyrie',
                                'columbia',
                                'triton',
                                'solaria',
                                'athena',
                                'erasmus',
                                'yashuman',
                                'bellerophon',
                                'perseus',
                                'artemis',
                                'prometheus'
                                ],
                        market: 'market1',
                        technical: 'technical1'
                   }
                ],
                chartTimespan: 24 * 60 * 60 * 1000,
                chartPrecision: 5 * 60 * 1000
        },

I must be missing something here...but not seeing it.

selaux commented 10 years ago

An array for miner ist not valid anymore. You need to specify market and technical for each miner:

miners: [
    {
        miner: 'galactica',,
        market: 'market1',
        technical: 'technical1'
    },
    {
        miner: 'pegasus',,
        market: 'market1',
        technical: 'technical1'
    },
    {
        miner: 'atlantia',,
        market: 'market1',
        technical: 'technical1'
    },
    ...
]
...

Since this might lead to a lot of duplication in your case, so you could do something like (untested):

title: 'Revenue Estimate (BTC Pools Only)',
module: 'revenue/solo',
miners:[
        'galactica',
        'pegasus',
        'atlantia',
        'valkyrie',
        'columbia',
        'triton',
        'solaria',
        'athena',
        'erasmus',
        'yashuman',
        'bellerophon',
        'perseus',
        'artemis',
        'prometheus'
    ].map(function (miner) {
        return {
            miner: miner,
            market: 'market1',
            technical: 'technical1'
        };
    }),
chartTimespan: 24 * 60 * 60 * 1000,
chartPrecision: 5 * 60 * 1000

You can also combine multiple coins into a single revenue module now.

jedimstr commented 10 years ago

Thanks, I'll try the untested solution first... You'll have to update your documentation then, it still lists arrays as useable options: https://github.com/selaux/miner-dashboard/wiki/solo

miner (Type: String|Array Default: empty): An id or an array of ids of miner modules for which you want to calculate revenue.