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