Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MHS5s variable #6

Open
michelem09 opened this issue Apr 24, 2014 · 3 comments
Open

MHS5s variable #6

michelem09 opened this issue Apr 24, 2014 · 3 comments

Comments

@michelem09
Copy link

Hello,
I found a very small issue with the web interface.

If you change the miner.conf "log" setting to something different than "5" you will not get anymore the "Hashrate 5s" field updated neither the main html title.

This is due to the fact you have hard-coded the variable you get (MHS5s) while this variable changes name (key) if you change that setting (log).

For example if you change "log" to "3" you have to get the "MHS3s" instead the "MHS5s".

So if you don't see anymore updates on title or "Hashrate 5s" the workaround is: put 5 as log setting (leave the default)

@DocGonzzo
Copy link
Contributor

Could you add some clarity to this issue? What build of Scripta are you using and which is the core mining binary you use CG or BFGminer?

@michelem09
Copy link
Author

I'm using the master branch with bfgminer instead cgminer.

If you look at https://github.com/scriptamining/scripta/blob/master/var/www/f_status.php file, you could see the problem, line 58 for example:

$devices += $dev['MHS5s']>0?1:0; // Only count hashing devices

the array key $dev['MHS5s'] doesn't exist if you change "log" variable in miner.conf settings (and I think this is the same for cgminer and bfgminer).
If you change log to for example "3" you should look at $dev['MHS3s'] instead the previous one.

Hope this help.

@DocGonzzo
Copy link
Contributor

I do see what you are saying. We will look at adding the ability to add a variable log rate to the status page. Thank you for bringing this issue to our attention and for supporting Scripta

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants