Masterchest Wallet Alpha Testing Thread

[zathras]

Hey guys,

I'd like some help testing the first proper alpha of the Windows Masterchest Wallet for the Master Protocol & Mastercoin.

The source & binary will be posted up here later tonight are below.  For starters you'll need to get your transaction index re-enabled in bitcoind/qt.  Please see the instructions for further information.

I'd suggest the very first testers have perhaps done sends via the advisor script & are technically minded.  

If possible I also suggest creating a new bitcoin wallet.dat (back up the old one of course - several times ), grabbing the new address and sending a small amount of BTC and/or MSC (via the advisor script) to this new address.  Then test the wallet with this new bitcoin wallet.dat.  Remember this is alpha software - it would be foolish to test it on a wallet containing significant sums of any currency.

This initial alpha will focus on getting the wallet up and running along with simple sends.  Distributed Exchange functionality is disabled in this build.  I'll then release a build with DEx around the end of the week and the focus will switch to testing DEx.

EDIT: Alpha 0.2 is up for initial testing of the DEx functionality.  Instructions have been briefly updated.  Thanks!
EDIT: Alpha 0.3 is up for initial testing of the DEx functionality, including real MSC.  Thanks
EDIT: Alpha 0.3c is up for initial testing of the DEx functionality, including real MSC.  Thanks
EDIT: Alpha 0.4a is up for smart property support (display/send).  Thanks

Remember this is alpha software and still contains a lot of bugs - it would be incredibly foolish to test this on a bitcoin wallet holding any significant amount of funds.



Source:  Github
Binary:  Github
Instructions:  here

Requirements: .NET Framework 4, bitcoind/qt with transaction index enabled

Thanks
Zathras

[chinabreak123]

that's great

[jeroenn13]

This is good news! Will help testing.

[littleblue]

great job！
buy msc！
 

[dacoinminster]

I've already been using this wallet, and it's great!!

Zathras, can you also post this to the mastercoin forum?

[zathras]

Links up!

[zathras]

I've already been using this wallet, and it's great!!

Zathras, can you also post this to the mastercoin forum?

Done

[jeroenn13]

Can't wait to test. Sync now with QT :\

[prophetx]

Zathras this is a better link

https://github.com/zathras-crypto/masterchest-wallet/blob/master/bin/MasterchestWalletAlpha_Bin.zip?raw=true

[dexX7]

Hey,

very nice to see this update. Looks very, very good!

Didn't try sending, but wallets are identified, balances are correct, transaction history is there also!

Here are two problems I found so far:


1. RPC usernames and passwords are case sensitive

After reading the wallet.cfg the username and password are currently parsed and casted to lower case. This shouldn't happen.

This affects:

https://github.com/zathras-crypto/masterchest-wallet/blob/master/source/Form1.vb#L95
https://github.com/zathras-crypto/masterchest-wallet/blob/master/source/Form1.vb#L138
https://github.com/zathras-crypto/masterchest-wallet/blob/master/source/Form1.vb#L141

Fix: remove .ToLower.



2. This was a nasty one! In some regions the decimal mark is a comma and somewhere else a point, e.g. "16058.671616199374" vs. "16058,671616199374".

Using a comma results in faulty SQL queries like this one:

Code:

insert into balances (address, cbalance, cbalancet) VALUES ('1EXoDusjGwvnjZUyKkxZ4UHEf77z6A5S4P', 1605867161619,94,0)

Which is invalid, because "1605867161619,94" is interpreted as two different values.

So far, I identified this line:

https://github.com/zathras-crypto/masterchest-wallet/blob/master/source/Form1.vb#L564

Code:

cmd.CommandText = "insert into balances (address, cbalance, cbalancet) VALUES ('1EXoDusjGwvnjZUyKkxZ4UHEf77z6A5S4P', " & (devmsc * 100000000) & ",0)"

Fix for this one:

Code:

cmd.CommandText = "insert into balances (address, cbalance, cbalancet) VALUES ('1EXoDusjGwvnjZUyKkxZ4UHEf77z6A5S4P', " & (devmsc * 100000000).ToString(Globalization.CultureInfo.InvariantCulture) & ",0)"

It's probably a better solution to globally define the culture though.

[prophetx]

i am not able to get it to connect on win 8, i know bitcoin daemon is running b/c i can connect other apps to it

https://github.com/zathras-crypto/masterchest-wallet/issues/3

[zathras]

Wow thanks guys

Already straight into it  OK - dexX7 great thanks!

1) have already committed a bugfix for the tolower on RPC details which got missed
2) Ouch yeah I'll take a look at that now
2) this was actually a problem with my dev MSC calculation having too many decimals - that query should have only ever been sending a whole number to the db - committed a fix

Prophetx - could you try the latest binary - this way we can see if it's win8 or the rpc case issue.

Thanks
Zathras

[dacoinminster]

I'm using win8 successfully, so I know it isn't that.

Zathras - prophetx is trying to set up a skype chat with you. I've been chatting with him trying to figure it out, but you could probably do it better

[zathras]

I'm using win8 successfully, so I know it isn't that.

Zathras - prophetx is trying to set up a skype chat with you. I've been chatting with him trying to figure it out, but you could probably do it better

Thanks J.R. - just talked with prophetx and located a bug with rpcport detection, commit w/ updated binary has just gone up.

[dacoinminster]

I'm using win8 successfully, so I know it isn't that.

Zathras - prophetx is trying to set up a skype chat with you. I've been chatting with him trying to figure it out, but you could probably do it better

Thanks J.R. - just talked with prophetx and located a bug with rpcport detection, commit w/ updated binary has just gone up.

Nice! That was fast!

Also, there appears to be an emerging consensus problem for real MSC between Masterchest and Masterchain on the dev thread.

I hope you're enjoying finally being full-time. I know my first few days as full-time have been very very full!

[zathras]

I'm using win8 successfully, so I know it isn't that.

Zathras - prophetx is trying to set up a skype chat with you. I've been chatting with him trying to figure it out, but you could probably do it better

Thanks J.R. - just talked with prophetx and located a bug with rpcport detection, commit w/ updated binary has just gone up.

Nice! That was fast!

Also, there appears to be an emerging consensus problem for real MSC between Masterchest and Masterchain on the dev thread.

I hope you're enjoying finally being full-time. I know my first few days as full-time have been very very full!

Just trying to get myself sorted out!  

I'm behind on emails - I wasn't aware of any consensus issues (except the skewed results due to mastercoin-explorer) - can you shoot me a subject line so I know what I'm looking for?

Thanks
Zathras

EDIT: Just checked my consensus page - seems like Masterchest & Masterchain are 100% for real MSC - unless of course the topic was about a problem with my consensus system
EDIT: Oh.  Seems like there is a problem with my consensus system
EDIT: Fixed my consensus system (now ignoring mastercoin-explorer results).  Seems Bitoy & I are 100%, but quite a few differences with Masterchain

EDIT: Looked over the first discrepancy.  Problem in Masterchain I believe.  Will wait for Graz to comment before moving onto investigating others further.
Graz,
Address 15a4XCuWmx2cCQVf8wZK7mqdvj5uwo1vby
You mark 221375a8dd6cc338af826ac9db5a01016e4c7a79a15b29aaf91a60685ecd7acd as invalid because of insufficient funds.  You don't see all the balance because it seems you're not decoding this transaction ecb77ee990de29745de949462e1f6e44584c310a0da12c9fbdf86dbe6ffabcfc (you see it just as a bitcoin payment).

[littleblue]

great  job， great man！ come on！hold your msc  ^buy msc！

[520Bit]

Waiting for official release.

[littleblue]

any good news？

[dacoinminster]

Yup!

http://www.mastercoinwallets.org