2018-06-16 10:37:11
addresses do not drop out of the pool stats until the pool restarts?
Rotonen
2018-06-16 13:32:54
not even then
Fireduck
2018-06-16 13:33:55
they drop out after 5 blocks of shares
Fireduck
2018-06-16 13:45:55
kk, thank you
Rotonen
2018-06-16 14:42:42
PPLNS is the name of the scheme
Fireduck
2018-06-16 14:48:31
minor adjustments
cleanup
prepare for master repo
GitHub
2018-06-16 14:51:02
[snowblossomcoin/snowblossom] #80 deployment reorg, common configs, new systemd impl
GitHub
2018-06-16 14:51:02
*https://github.com/snowblossomcoin/snowblossom/compare/c741afe37051...488eb2e53340*
https://github.com/snowblossomcoin/snowblossom/commit/5442da92abfcf7eb4ff87bd5a4e6ab9150042015 - deployment reorg, common configs, new systemd impl
https://github.com/snowblossomcoin/snowblossom/commit/488eb2e533402532b47b8b964028604d348c9870 - Merge pull request #80 from cluelessperson/master
GitHub
2018-06-16 14:53:19
@Clueless does that new config move the wallet or db location?
Fireduck
2018-06-16 14:53:55
verifying
Clueless
2018-06-16 14:55:46
windows deployment had `wallet`
linux (example configs had `wallets/mainnet/`
new config is now `wallets`
Clueless
2018-06-16 14:56:33
-
Clueless
2018-06-16 14:58:21
windows: `db_path=snowdb`
linux: `node_db/mainnet`
new config is now `node_db/mainnet`
Clueless
2018-06-16 14:58:58
-
linux: `pool_db/mainnet`
new config: `pool_db/mainnet`
Clueless
2018-06-16 15:00:28
@Fireduck It means users might have to move things around a bit to get on the same schema.
I'm not sure how to prevent that when cleaning up.
Clueless
2018-06-16 15:01:28
Not worried about node db
Fireduck
2018-06-16 15:01:33
It can resync
Fireduck
2018-06-16 15:01:52
The wallet has the potential to be nightmare
Fireduck
2018-06-16 15:02:14
Can you make it just wallet?
Fireduck
2018-06-16 15:02:40
And that should allow our windows users to not freak out or accidentally lose funds
Fireduck
2018-06-16 15:02:51
sure, sounds like a good idea.
Clueless
2018-06-16 15:04:11
*https://github.com/snowblossomcoin/snowblossom/compare/488eb2e53340...ecfb7a39375f*
https://github.com/snowblossomcoin/snowblossom/commit/ecfb7a39375f491d2ba25aea76fc8bead1d605c5 - Update client.conf
GitHub
2018-06-16 15:07:53
@Fireduck The other thing to be aware of.
`/example/deployment/build.sh` is what outputs the linux/windows release.
The linux install scripts treat the **latest** github release attachment as the thing to install.
Clueless
2018-06-16 15:47:56
*https://github.com/snowblossomcoin/snowblossom/compare/ecfb7a39375f...a5c91e4c1995*
https://github.com/snowblossomcoin/snowblossom/commit/c2c697ef411363a9fe0b5e6f3cbc92e84bc2e913 - removing .project from git
https://github.com/snowblossomcoin/snowblossom/commit/a5c91e4c19955770fe41c46322b6b0495ac3b3a4 - fixing formatting in MemPool.java
GitHub
2018-06-16 15:53:22
@Fireduck, what is causing output like this: Jun 16, 2018 8:22:50 AM snowblossom.node.MemPool rebuildPriorityMap
Tyler Boone
2018-06-16 15:53:50
It's got to be some kind of reflection going on somewhere because there is no log or print statement like that
Tyler Boone
2018-06-16 15:56:02
Pretty sure you are wrong
Fireduck
2018-06-16 15:56:13
I recall writing that log line
Fireduck
2018-06-16 15:57:26
you know what
Tyler Boone
2018-06-16 15:57:37
I just realized I might be running a node that is like a month old...
Tyler Boone
2018-06-16 15:57:42
but looking at code that is current
Tyler Boone
2018-06-16 16:00:16
I could be wrong too
Fireduck
2018-06-16 16:03:43
same thing with this: Jun 16, 2018 9:01:56 AM snowblossom.lib.PowUtil calcNextTarget
Tyler Boone
2018-06-16 16:04:36
you know what... could be something in the logger...
Tyler Boone
2018-06-16 16:04:45
Ah, that is the context for the subsequent line
Fireduck
2018-06-16 16:05:15
If you want to fix logging that would be great
Fireduck
2018-06-16 16:05:30
The logging properties file does notging
Fireduck
2018-06-16 16:06:09
do you have an idea for how you want to manage "log this string" vs. "show this string to the user"?
Tyler Boone
2018-06-16 16:10:02
I'd be happy to have any log level control at all
Fireduck
2018-06-16 16:11:23
Here is what I want: All output is via logging api
Fireduck
2018-06-16 16:11:40
we have a config file that says snowblossom.* = info level
Fireduck
2018-06-16 16:12:00
OK. so maybe output to console INFO and above?
Tyler Boone
2018-06-16 16:12:10
and if someone things they might have a peering problem they could set snowblossom.peering to level finest
Fireduck
2018-06-16 16:12:22
assume info and above is usually output to console
Fireduck
2018-06-16 16:12:26
and that should be the default config
Fireduck
2018-06-16 16:13:22
in theory, the log config allows this but I can't get it to fing work
Fireduck
2018-06-16 16:13:51
LogSetup loads the config if set
Fireduck
2018-06-16 16:13:59
but I can't get it to actually change any behavior
Fireduck
2018-06-16 16:16:00
lol
Tyler Boone
2018-06-16 16:17:31
and we are somewhat tied to java.util.logging beccause that is what grpc and friends are outputting to
Fireduck
2018-06-16 16:17:36
and we want to silence that shit
Fireduck
2018-06-16 16:17:53
I guess we could silence java.util.logging and then use something else for our stuff
Fireduck
2018-06-16 16:18:01
I'm rather dissapointed with intellij to be honest
Tyler Boone
2018-06-16 16:18:19
ReSharper is so f-ing good. I was expecting similar
Tyler Boone
2018-06-16 16:39:59
I thought Microsoft's MSDN documentation was bad
Tyler Boone
2018-06-16 16:40:24
Oracle blows it out of the water (badness wise)
Tyler Boone
2018-06-16 16:40:37
it's like they want developers to hate Java
Tyler Boone
2018-06-16 17:04:41
The logging is a particular fucjstorn
Fireduck
2018-06-16 17:04:55
I am good with java and that shit still makes no sense
Fireduck
2018-06-16 17:11:06
I have it generating a log file with the lower log levels
Tyler Boone
2018-06-16 17:11:26
but I don't understand this FileHandler.count bullshit
Tyler Boone
2018-06-16 17:12:10
Is it my bullshit?
Fireduck
2018-06-16 17:12:22
no
Tyler Boone
2018-06-16 17:17:06
weird.. every time I start Node it generates 2 log files...
Tyler Boone
2018-06-16 17:23:16
OK, figured it out. had file size limit set too small
Tyler Boone
2018-06-16 17:33:17
*https://github.com/snowblossomcoin/snowblossom/compare/a5c91e4c1995...5bba5416dbb4*
https://github.com/snowblossomcoin/snowblossom/commit/5bba5416dbb4dcc1f4073558b8fc744d44432d84 - removing extra lines
GitHub
2018-06-16 17:48:16
1. adding logging to file
2. improving log lines
GitHub
2018-06-16 17:49:05
If one of you gets a chance, will you build a Windows release with that and send it to me
Tyler Boone
2018-06-16 17:49:43
I am at nordstroms
Fireduck
2018-06-16 17:49:54
no rush
Tyler Boone
2018-06-16 17:49:58
just whenever you get a change
Tyler Boone
2018-06-16 17:50:00
chance
Tyler Boone
2018-06-16 21:29:16
@Tyler Boone Sure, I'll add it
Clueless
2018-06-16 21:29:42
the config and properties configs just need to be added to the example/deployment/snowblossom/ folder, and make sure the configs are modified to use it
Clueless
2018-06-16 21:29:50
and make sure it works on linux AND windows. :)
Clueless
2018-06-16 21:30:11
*https://github.com/snowblossomcoin/snowblossom/compare/5bba5416dbb4...dd92072573a0*
https://github.com/snowblossomcoin/snowblossom/commit/dd92072573a0373fcfca30d677b8e2715536b662 - Make build executable
GitHub
2018-06-16 21:37:23
https://foreverbucket-qa.appspot.com/a/zooom/20180616T213654/snowblossom-1.0.7-9-gdd92072.zip
Fireduck
2018-06-16 22:04:36
I don't think you created that from my branch
Tyler Boone
2018-06-16 22:11:24
@Tyler Boone Does the file logging interrupt console logging?
Clueless
2018-06-16 22:11:37
no
Tyler Boone
2018-06-16 22:11:45
I only know about one branch and that is called master
Fireduck
2018-06-16 22:11:52
git pull
Tyler Boone
2018-06-16 22:12:22
git checkout tster/add-file-logging
Tyler Boone
2018-06-16 22:12:38
git pull
Tyler Boone
2018-06-16 22:13:17
fine
Fireduck
2018-06-16 22:16:26
it is one of those
Fireduck
2018-06-16 22:16:59
https://foreverbucket-qa.appspot.com/a/zooom/20180616T221557/snowblossom-1.0.7-9-g73d5173.zip
Fireduck
2018-06-16 22:19:48
hmmm, it's missing logging.properties
Tyler Boone
2018-06-16 22:22:18
I thought this line would copy all the files in configs/:
for i in configs/*; do awk 'sub("$", "\r")' "$i" > "$1/$i"; done
Tyler Boone
2018-06-16 22:23:45
awkward
Fireduck
2018-06-16 22:24:14
I don't know what example/configs vs example/deployment/configs is...
Tyler Boone
2018-06-16 22:24:58
example/depoyment/configs is some insanity
Tyler Boone
2018-06-16 22:25:02
pointer files?
Tyler Boone
2018-06-16 22:25:26
C:\Users\thboo\source\repos\snowblossom\snowblossom>type example\deployment\configs\client.conf
../../configs/client.conf
Tyler Boone
2018-06-16 22:25:36
yeah, symlinks
Fireduck
2018-06-16 22:26:11
they are text files
Tyler Boone
2018-06-16 22:26:28
on linux, they are symlinks
Fireduck
2018-06-16 22:32:53
OK, well I guess you'll have to add a symlink to logging.properties
Tyler Boone
2018-06-16 22:34:37
also, pull the latest commit
Tyler Boone
2018-06-16 22:34:50
and add a symlink and it should work
Tyler Boone
2018-06-16 22:44:37
we should probably change the release to include lib and all the lib requirements in a single jar, and then have each component in a separate jar and just add the big jar to the jarpath
Tyler Boone
2018-06-16 22:45:09
kind of silly that every jar is 29mb...
Tyler Boone
2018-06-16 22:45:30
that's probably a nice idea.
Clueless
2018-06-16 22:45:44
heh
Fireduck
2018-06-16 22:47:07
https://foreverbucket-qa.appspot.com/a/zooom/20180616T224552/snowblossom-1.0.7-11-gfe20657.zip
Fireduck
2018-06-16 22:49:47
OK cools good
Tyler Boone
2018-06-16 22:50:41
wtf
Tyler Boone
2018-06-16 22:50:52
why is it using node_db instead of snowdb...
Tyler Boone
2018-06-16 22:51:12
bah
Tyler Boone
2018-06-16 22:53:04
@Tyler Boone because before it was "snowdb" which isn't very clear or informative, especially when new db folders may show up.
Clueless
2018-06-16 22:53:11
In my case, `pool_db`
Clueless
2018-06-16 22:53:23
What's more. `node_db/mainnet`
Clueless
2018-06-16 22:53:26
OK, this is by design then
Tyler Boone
2018-06-16 22:53:39
ah, I didn't know. :)
Clueless
2018-06-16 22:53:39
yeah, people might end up resyncing but it is fine
Fireduck
2018-06-16 22:53:47
might==>will
Tyler Boone
2018-06-16 22:53:55
that's fine
Tyler Boone
2018-06-16 22:54:00
just was unexpected
Tyler Boone
2018-06-16 22:54:08
sure
Fireduck
2018-06-16 22:54:44
@Tyler Boone yeah What's worse is that my linux examples did `node_db/mainnet` and windows `snowdb` And the database doesn't differentiate between mainnet and testnet
Clueless
2018-06-16 22:54:54
we should have a release notes or something I guess
Tyler Boone
2018-06-16 22:55:04
great idea!
Clueless
2018-06-16 22:55:15
otherwise there will be mass confusion
Tyler Boone
2018-06-16 22:55:16
especially if it just said "welcome to your doom"
Fireduck
2018-06-16 22:55:21
troof
Tyler Boone
2018-06-16 22:55:46
Other coins have fancy web sites and marketing budgets
Fireduck
2018-06-16 22:55:55
@Tyler Boone also, I linked this earlier, but not sure you saw it.
https://github.com/cluelessperson/snowblossom/wiki
Want your opinions.
Clueless
2018-06-16 22:55:57
We have torrent seeds hidden above microwaves
Fireduck
2018-06-16 22:56:03
not everything is fully there yet, but it's taking shape.
Clueless
2018-06-16 23:00:51
looks pretty good
Tyler Boone
2018-06-16 23:01:04
later we should add a page on tips to get the best hashrate
Tyler Boone
2018-06-16 23:01:54
That will definitely go under "Further"
Clueless
2018-06-16 23:02:11
eh, maybe not. It is a FAQ thing too
Clueless
2018-06-16 23:02:16
we'll see
Clueless
2018-06-16 23:02:31
Any Release Note information?
Clueless
2018-06-16 23:02:38
runs to `git log`
Clueless
2018-06-16 23:02:44
a FAQ can contain a link to a page for more details
Tyler Boone
2018-06-16 23:03:07
that sounds sane
Clueless
2018-06-16 23:04:38
lots of hate for issue 4
Fireduck
2018-06-16 23:04:39
Issue 4: Yes: 0, No: 91, Both: 0
Fireduck
2018-06-16 23:05:52
might have to skip that or modify the config. Problem is I ran into trouble with people not understanding voting
Clueless
2018-06-16 23:06:11
```vote_yes=1,2,3
vote_no=4```
works very well
Clueless
2018-06-16 23:06:22
except now people are voting against 4
Clueless
2018-06-16 23:06:54
```
vote_yes=a,b,c
vote_no=d
```
Clueless
2018-06-16 23:06:56
might work
Clueless
2018-06-16 23:07:24
I like it
Fireduck
2018-06-16 23:07:38
SIP4 - save the kittens from drowning
Fireduck
2018-06-16 23:08:00
like which option, skipping or example using letters instead
Clueless
2018-06-16 23:08:05
then it'll error if they don't read it
Clueless
2018-06-16 23:08:08
in the quickstart
Tyler Boone
2018-06-16 23:08:25
you need to tell users how to figure out which is the current snowfield
Tyler Boone
2018-06-16 23:08:51
I created an issue to **loudly** report the current snowfield in node.
Clueless
2018-06-16 23:09:04
but I can add to check the explorer for the `activated_field`
Clueless
2018-06-16 23:09:19
actually, that's another good FAQ. "When will we be on snowfield x"
Tyler Boone
2018-06-16 23:09:27
with each block too, not just end of sync
Clueless
2018-06-16 23:09:43
@Tyler Boone nice! I was linking people to the code map describing difficulty earlier. Will add
Clueless
2018-06-16 23:10:19
What else is going into 1.0.8 ?
Clueless
2018-06-16 23:10:38
We should make 1.1.0 soon
Fireduck
2018-06-16 23:10:42
to include sip1
Fireduck
2018-06-16 23:12:33
what block did sip1 pass?
Clueless
2018-06-16 23:14:12
5250 or so
Fireduck
2018-06-16 23:14:17
not really sure
Fireduck
2018-06-16 23:18:46
[snowblossomcoin/snowblossom] #81 logging improvements
GitHub
2018-06-16 23:18:46
*https://github.com/snowblossomcoin/snowblossom/compare/dd92072573a0...2069236abe7d*
https://github.com/snowblossomcoin/snowblossom/commit/2069236abe7d10444e4fbf74311b9475cf9dce5f - logging improvements (#81)
GitHub
2018-06-16 23:41:10
here's what I got so far
Clueless
2018-06-16 23:59:04
Fuck yeah, release notes
Fireduck