1 2017-09-15 03:58:00	0|achow101|was there any agreement on whether there would be a 0.15.0.1 with BlueMatt's fix for the customfeeradio thing?
  2 2017-09-15 03:58:45	0|gmaxwell|No soap radio.
  3 2017-09-15 03:59:48	0|gmaxwell|achow101: I think the plan right now is that we will fix things and get them into the 0.15 branch ASAP, then wladimir will decide to do a release or not.
  4 2017-09-15 04:00:09	0|achow101|ok
  5 2017-09-15 04:00:37	0|achow101|I'm working on the permanent fix right now
  6 2017-09-15 04:01:19	0|luke-jr|ie removing the radio entirely?
  7 2017-09-15 04:01:22	0|gmaxwell|luke-jr: what other moderately important things are there that we know of now
  8 2017-09-15 04:01:38	0|achow101|luke-jr: yes
  9 2017-09-15 04:02:29	0|luke-jr|gmaxwell: if it was important, it'd have blocked 0.15.0; is there a reason *not* to backport the GUI positioning fix & similar stuff, if we're doing another RC cycle?
 10 2017-09-15 04:03:05	0|gmaxwell|oh did we make a post 0.15 positioning fix too
 11 2017-09-15 04:03:25	0|achow101|I don't think so
 12 2017-09-15 04:03:38	0|luke-jr|#11208 ?
 13 2017-09-15 04:03:40	0|gribble|https://github.com/bitcoin/bitcoin/issues/11208 | Fixing offscreen GUI issue by MeshCollider · Pull Request #11208 · bitcoin/bitcoin · GitHub
 14 2017-09-15 04:03:57	0|achow101|I thought that was determined to not be a problem by jonasschnelli
 15 2017-09-15 04:04:18	0|achow101|I believe the problem was I was testing on either an old version or with custom something that gave me the error
 16 2017-09-15 04:04:27	0|achow101|s/error/issue
 17 2017-09-15 04:04:28	0|jonasschnelli|achow101: it may be a problem for the PPA.
 18 2017-09-15 04:04:55	0|jonasschnelli|From what I knew so far was that the offscreen issue only affected windows
 19 2017-09-15 04:05:10	0|luke-jr|gitian binaries are not the only supported (or even preferred) deployment..
 20 2017-09-15 04:05:19	0|gmaxwell|I believe there are still problems. Go position the window as far down and left as you can, then restart... But perhaps for 0.15.0.1 we would just disable remembered position rather than twiddling around with it.
 21 2017-09-15 04:05:22	0|jonasschnelli|Offscreen situations are usually handled by the OS
 22 2017-09-15 04:06:18	0|jonasschnelli|stop remembering position would be reducing a feature for a few having problem with it.
 23 2017-09-15 04:06:58	0|gmaxwell|Yes, but the feature is minor (the software already takes a very long time to start) and the issue is very severe.
 24 2017-09-15 04:07:14	0|gmaxwell|I'm not saying that I think it should go away forever.
 25 2017-09-15 04:07:27	0|jonasschnelli|Some users arrange windows pixel by pixel.. :)
 26 2017-09-15 04:08:01	0|gmaxwell|But if we want to make a quick release it would be an easy way to prevent it from being a problem for sure.
 27 2017-09-15 04:09:25	0|meshcollider|11208 checks for full window inclusion on the screen, and uses available screen geometry not full screen so that whatever taskbars or things dont get in the way either
 28 2017-09-15 04:09:49	0|meshcollider|so I think its still an improvement even if the original issue is gone
 29 2017-09-15 04:10:09	0|gmaxwell|meshcollider: that sounds like the right thing to do.  But also sounds like it deserves non-trivial testing with weird setups.
 30 2017-09-15 04:10:45	0|bitcoin-git|[13bitcoin] 15achow101 opened pull request #11334: Remove custom fee radio group and remove nCustomFeeRadio setting (06master...06rm-nCustomFeeRadio) 02https://github.com/bitcoin/bitcoin/pull/11334
 31 2017-09-15 04:10:50	0|jonasschnelli|gmaxwell: I just moved the windows as far down as possible and it reopened there... is that a problem?
 32 2017-09-15 04:10:52	0|jonasschnelli|Same happends to notepad.exe
 33 2017-09-15 04:10:57	0|jonasschnelli|*happens
 34 2017-09-15 04:11:15	0|jonasschnelli|(Windows 8.1)
 35 2017-09-15 04:12:12	0|gmaxwell|can you get it back out of that position again
 36 2017-09-15 04:13:05	0|jonasschnelli|Yes.
 37 2017-09-15 04:13:23	0|jonasschnelli|(Testing 0.15)
 38 2017-09-15 04:13:29	0|luke-jr|what if you use the keyboard to move it such that the titlebar isn't visible?
 39 2017-09-15 04:13:55	0|bane5000|so i've enabled pruning on my core wallet (by creating bitcoin.conf in ~/.bitcoin/) and adding pruning=10000
 40 2017-09-15 04:14:14	0|bane5000|i've restarted the bitcoin-qt, however, the old blocks do not seem to be disappearing, as my harddrive space has not been reduced
 41 2017-09-15 04:14:16	0|luke-jr|bane5000: you mean prune=10000 ? note that user issues should be in #bitcoin though
 42 2017-09-15 04:14:30	0|gmaxwell|e.g. if your screen is 1920x1080, what happens if the location is 1919,1079
 43 2017-09-15 04:14:42	0|bane5000|luke-jr: Whoopsy :)
 44 2017-09-15 04:15:28	0|bane5000|luke-jr: Would you recommend keeping a certain amount of blocks? Perhaps 20 gigs?
 45 2017-09-15 04:15:43	0|luke-jr|bane5000: #bitcoin please
 46 2017-09-15 04:15:50	0|bane5000|ugh :(
 47 2017-09-15 04:15:58	0|bane5000|You probably give a better opinion though, but fine
 48 2017-09-15 04:16:05	0|luke-jr|I'm there too
 49 2017-09-15 04:16:28	0|jonasschnelli|gmaxwell: I guess you can only position to right-bottom -1,-1 with rededit.exe
 50 2017-09-15 04:16:51	0|bane5000|okay :)
 51 2017-09-15 04:16:52	0|jonasschnelli|IMO it reacts as expected and identical to any other window based windows application
 52 2017-09-15 04:17:28	0|gmaxwell|jonasschnelli: I think you could do so pretty easily like this: increase your resolution, position it to the right bottom of your lower resolution, decrease resolution.
 53 2017-09-15 04:17:45	0|gmaxwell|Though there may be other ways to end up there.
 54 2017-09-15 04:17:49	0|meshcollider|but its not practical if it appears mostly offscreen, having to manually move it, even if thats what other windows do
 55 2017-09-15 04:18:09	0|jonasschnelli|gmaxwell: I did change the screen size and it relocates to center at a certain point.
 56 2017-09-15 04:18:50	0|jonasschnelli|Maybe (like in a flexible screen size vm like I test in) you can end up by only showing a single pixel of the window... but I guess the OS makes sure you always see the left-top icon
 57 2017-09-15 04:19:07	0|jonasschnelli|However, I think this is not a problem anymore in 0.15 gitian binaries...
 58 2017-09-15 04:19:25	0|gmaxwell|if the OS does then thats probably sufficient, but then I don't understand how we ended up completely off the screen.
 59 2017-09-15 04:19:33	0|jonasschnelli|I'm unsure about compiling it manually ... though I'm not sure how you would do this on windows
 60 2017-09-15 04:19:55	0|gmaxwell|jonasschnelli: we still support QT4 and earlier QT5 too.. it's not fixed unless we have a workaround or refuse to build with incompatible versions.
 61 2017-09-15 04:20:07	0|jonasschnelli|gmaxwell: completely off the screen with 0.15 gitian binaries on Windows seems very unlikely (I could not reproduce it and I tried for 1-2 h)
 62 2017-09-15 04:20:48	0|jonasschnelli|gmaxwell: I agree about Qt4. But I don't think anyone builds a Windows binary without our depends system. It would be a nightmare
 63 2017-09-15 04:20:59	0|gmaxwell|maybe for windows you could argue to ignore anything but gitian but we could end up off screen on linux too.. can't we
 64 2017-09-15 04:21:51	0|achow101|is qt4 supported only for the ppa?
 65 2017-09-15 04:21:54	0|jonasschnelli|I have testes offscreen on Ubuntu 14, 16 and could also not reproduce it (by fiddling with the QSettings to set if offscreen). But possible for Qt5.
 66 2017-09-15 04:21:56	0|jonasschnelli|Qt4, I meant
 67 2017-09-15 04:22:25	0|jonasschnelli|achow101: I think BlueMatt has a reason to build it against Qt4... some KDE or something
 68 2017-09-15 04:22:41	0|luke-jr|jonasschnelli: testing merely Ubuntu is not meaningful.. there are many WMs, and Ubuntu only uses one or two
 69 2017-09-15 04:23:10	0|jonasschnelli|luke-jr: Yes. I agree. But IMO this is why we use Qt. Offscreen is either a OS thing or a QT thing.
 70 2017-09-15 04:23:26	0|luke-jr|I suppose testing without a WM (X11 and Wayland?) might be a good approach
 71 2017-09-15 04:23:37	0|jonasschnelli|If we are going to add patch for every WM, we will have a horrific cross-platform source code base
 72 2017-09-15 04:23:45	0|jonasschnelli|*add patches
 73 2017-09-15 04:24:02	0|luke-jr|I don't see why that'd be necessary
 74 2017-09-15 04:24:14	0|luke-jr|fixing it for the dumb-WM-with-no-protection should fix it for all
 75 2017-09-15 04:24:23	0|jonasschnelli|Offscreen seems very upstreamish... we (or someone else) need to fix it there (if its really an issue)
 76 2017-09-15 04:24:40	0|luke-jr|that assumes we're using upstream correctly
 77 2017-09-15 04:24:57	0|gmaxwell|saving location _at all_ seems upstreamish, but we have an implementation of that-- it seems to me that if we're implementing it, we're probably stuck implementing it completely.
 78 2017-09-15 04:25:15	0|jonasschnelli|I'm happy to review a patch... but quick-cross-platform fixed did turn out to be much more complicated then often initially intended
 79 2017-09-15 04:25:17	0|luke-jr|gmaxwell: +1
 80 2017-09-15 04:25:26	0|jonasschnelli|gmaxwell: Indeed
 81 2017-09-15 04:26:57	0|jonasschnelli|related: http://doc.qt.io/qt-4.8/restoring-geometry.html
 82 2017-09-15 04:27:17	0|jonasschnelli|Document still available in the Qt5.9 docs: http://doc.qt.io/qt-5/restoring-geometry.html
 83 2017-09-15 04:27:41	0|achow101|jonasschnelli: can we not just do that then to support the fix for qt4 and 5?
 84 2017-09-15 04:28:00	0|jonasschnelli|achow101: do what?
 85 2017-09-15 04:28:03	0|meshcollider|Should I modify 11208 to just call restoreGeometry
 86 2017-09-15 04:28:39	0|luke-jr|we should investigate if Qt actually implements it sanely
 87 2017-09-15 04:28:41	0|achow101|jonasschnelli: whatever those pages say to restore geometry
 88 2017-09-15 04:29:05	0|meshcollider|Those pages suggest either using restoreGeometry OR doing it how 11208 does it I believe
 89 2017-09-15 04:29:14	0|luke-jr|"The restore function also checks if the restored geometry is outside the available screen geometry, and modifies it as appropriate if it is:" sounds hopeful
 90 2017-09-15 04:29:21	0|jonasschnelli|achow101: Yes. We should update to restoreGeometry... maybe this does internally check the bounds.
 91 2017-09-15 04:29:46	0|jonasschnelli|Seems the much better approach then fiddling with the position (which is what Qt is supposed to do)
 92 2017-09-15 04:30:10	0|jonasschnelli|luke-jr: +1
 93 2017-09-15 04:31:40	0|jonasschnelli|Yes should replace our own saveWindowGeometry/restoreWindowGeometry with the Qt ones (not directly possible for the save)
 94 2017-09-15 04:32:24	0|gmaxwell|I wonder how this stuff interacts with -geometry
 95 2017-09-15 04:32:48	0|achow101|jonasschnelli: I tried with gmaxwell's suggestion of 1919,1079 and the window is definitely not visible on screen
 96 2017-09-15 04:33:11	0|jonasschnelli|achow101: via regedit.exe?
 97 2017-09-15 04:33:15	0|achow101|yeah
 98 2017-09-15 04:33:35	0|jonasschnelli|0.15.0 gitian binaries?
 99 2017-09-15 04:33:40	0|achow101|yes, from bitcoin.org
100 2017-09-15 04:33:58	0|jonasschnelli|What happens if you place then +2 px?
101 2017-09-15 04:35:03	0|jonasschnelli|achow101: Maybe this is the expected behavior: if you place the window in screen (manually with RedEdit.exe), you want to reopen there? To hit this by accident seems very unlikely
102 2017-09-15 04:35:34	0|gmaxwell|jonasschnelli: I explained before. If you increase your resolution, position there. Shut down bitcoin. Reduce resolution.  You will be in this state.
103 2017-09-15 04:35:49	0|jonasschnelli|gmaxwell: That's a different issue.. that is solved.
104 2017-09-15 04:35:56	0|luke-jr|…
105 2017-09-15 04:35:58	0|gmaxwell|It's a hard state to get into, but then "my bitcoins are gone!"   -- a typical user will not be able to recover without a tech guru friend at least.
106 2017-09-15 04:35:58	0|jonasschnelli|achow101: placed it in-screen (-1,-1)
107 2017-09-15 04:36:12	0|luke-jr|jonasschnelli: you can get (1919,1079) the way gmaxwell describes
108 2017-09-15 04:36:22	0|jonasschnelli|luke-jr: how?
109 2017-09-15 04:36:33	0|luke-jr|jonasschnelli: try to position it on the second monitor without snapping
110 2017-09-15 04:36:41	0|achow101|jonasschnelli: anything that is technically on screen like 1919,1079 will not be fixed. but the user can't get to it
111 2017-09-15 04:37:04	0|jonasschnelli|luke-jr: Thats indeed a viable argument
112 2017-09-15 04:37:22	0|jonasschnelli|But wait...
113 2017-09-15 04:37:26	0|gmaxwell|jonasschnelli: with a higher resolution you can easily drag to 1919,1079 (for example).
114 2017-09-15 04:37:29	0|achow101|jonasschnelli: I increased the position so that I could see it. most of the window is off screen, but the corner that matters is
115 2017-09-15 04:37:43	0|achow101|it can easily be covered up by the taskbar too so users can't get to it
116 2017-09-15 04:37:58	0|jonasschnelli|I wonder how other applicationes (test Notepad.exe) would react in this case
117 2017-09-15 04:38:08	0|luke-jr|stupidly IIRC
118 2017-09-15 04:38:21	0|gmaxwell|(and there is probably a few pixel radius which is totally inaccessible but technically on the screen where the current behavior is not sufficient. But meshcollider's or hopefully QT's would be.
119 2017-09-15 04:38:55	0|gmaxwell|Even if notepad is wrong (would be interesting to find out!) that still doesn't excuse bitcoin-- losing notepad is less serious then your wallet. :)
120 2017-09-15 04:38:59	0|achow101|jonasschnelli: it does exactly the same thing; so long as the top left corner is technically on screen, it stays there
121 2017-09-15 04:39:07	0|luke-jr|we can actually continue saving as we do now, but after restoring, call restoreWindowGeometry(saveWindowGeometry())…
122 2017-09-15 04:39:18	0|luke-jr|gmaxwell: but my metadata!
123 2017-09-15 04:39:23	0|meshcollider|Somethign like this: https://github.com/MeshCollider/bitcoin/commit/e70a784486fdda12a8a949dbb3f5fe5ea1054511
124 2017-09-15 04:39:26	0|jonasschnelli|I think we should update our codebase to use the Qt designated functions for restoring geometry (also available on Qt4) and see if this is better,.. then decide to patch this edge-case explicit in our codebase
125 2017-09-15 04:39:27	0|meshcollider|havent tested yet though
126 2017-09-15 04:39:45	0|jonasschnelli|achow101: you mean with Notepad.exe?
127 2017-09-15 04:39:48	0|gmaxwell|yea, well I think we have a target edge case to test now. Hopefully QT does the right thing.
128 2017-09-15 04:40:36	0|achow101|jonasschnelli: yes
129 2017-09-15 04:40:44	0|jonasschnelli|I'm still convinced that this issue needs to be fixed in the cross-platform layer (Qt) rather then in the application GUI logic
130 2017-09-15 04:41:22	0|jonasschnelli|I saw many cross-platform projects struggle after adding to much cross-platform UI patches and glitch-foixes
131 2017-09-15 04:41:25	0|jonasschnelli|*fixes
132 2017-09-15 04:41:59	0|achow101|jonasschnelli: it shouldn't be a problem if we use the Qt sanctioned method, no?
133 2017-09-15 04:42:24	0|jonasschnelli|achow101: you mean http://doc.qt.io/qt-5/qwidget.html#restoreGeometry?
134 2017-09-15 04:42:36	0|bitcoin-git|[13bitcoin] 15MeshCollider opened pull request #11335: [WIP] Replace save|restoreWindowGeometry with Qt functions (06master...06201709_fix_offscreen_2) 02https://github.com/bitcoin/bitcoin/pull/11335
135 2017-09-15 04:42:42	0|achow101|yes
136 2017-09-15 04:42:58	0|jonasschnelli|I'm not sure if this would fix it... haven't tested it. Haven't checked the upstream by-platform code for restoreGeometry
137 2017-09-15 04:43:05	0|jonasschnelli|But we should def. use it
138 2017-09-15 04:43:21	0|jonasschnelli|Rather then implementing it in our layer
139 2017-09-15 04:43:35	0|meshcollider|Agreed
140 2017-09-15 04:43:52	0|luke-jr|https://gist.github.com/5c7afe24d6d4cb512c9d0940c91d5ac4
141 2017-09-15 04:43:53	0|achow101|well then lets try and see what it does
142 2017-09-15 04:44:04	0|jonasschnelli|meshcollider: Oh. Great, ... you already PRed #11335
143 2017-09-15 04:44:05	0|gribble|https://github.com/bitcoin/bitcoin/issues/11335 | [WIP] Replace save|restoreWindowGeometry with Qt functions by MeshCollider · Pull Request #11335 · bitcoin/bitcoin · GitHub
144 2017-09-15 04:44:07	0|jonasschnelli|I'll go a gitian build
145 2017-09-15 04:44:20	0|jonasschnelli|*do
146 2017-09-15 04:45:01	0|meshcollider|Oh that looks a lot better if it also checks for maximised and minimised right?
147 2017-09-15 04:45:15	0|meshcollider|luke-jr: is that from Qt 4
148 2017-09-15 04:45:21	0|gmaxwell|this code looks like it'll handle it.
149 2017-09-15 04:45:21	0|meshcollider|or 5
150 2017-09-15 04:45:42	0|luke-jr|meshcollider: 5.7
151 2017-09-15 04:46:04	0|gmaxwell|might want to take a quick look at 4.x just to see if it still looks sufficient there.
152 2017-09-15 04:46:19	0|luke-jr|so long as the current version works, IMO it's not our problem if older ones are broken
153 2017-09-15 04:46:48	0|achow101|luke-jr: well qt4 is still used and supported, so we want to make sure that works too
154 2017-09-15 04:46:51	0|luke-jr|(so long as we're outsourcing to Qt)
155 2017-09-15 04:47:00	0|gmaxwell|luke-jr: meh, if we build with it, it should work. at least against serious issues (and I think the program being totally inoperable is serious)
156 2017-09-15 04:47:06	0|meshcollider|speaking of Qt issues, does 5.9 fix any of the segfault crashes like #9683? (cf 10505)
157 2017-09-15 04:47:09	0|gribble|https://github.com/bitcoin/bitcoin/issues/9683 | random abort running master/HEAD · Issue #9683 · bitcoin/bitcoin · GitHub
158 2017-09-15 04:47:12	0|jonasschnelli|https://bitcoin.jonasschnelli.ch/build/305
159 2017-09-15 04:47:23	0|luke-jr|gmaxwell: disable restoring position on Qt4? :p
160 2017-09-15 04:47:57	0|achow101|luke-jr: qt4 has the same functions, we should check their behavior
161 2017-09-15 04:47:59	0|gmaxwell|luke-jr: maybe you could argue that it was sufficient if the latest QT4 had it, in any case I expect it does have similar code.  This stuff seems like GUI101.
162 2017-09-15 04:48:01	0|meshcollider|I'd imagine 4 would still have decent restoring function anyway lol
163 2017-09-15 04:48:02	0|luke-jr|achow101: sure
164 2017-09-15 04:48:15	0|jonasschnelli|Qt4 support doesn't need to support everything: https://github.com/bitcoin/bitcoin/blob/f65614726de21e116966366d6abdf025dfeb6db2/doc/build-osx.md#L27
165 2017-09-15 04:48:23	0|luke-jr|gmaxwell: the Qt5 code there suggests it changed around 5.3
166 2017-09-15 04:48:24	0|gmaxwell|thou shall not restore the window to a location which is completely inaccessible to the user.
167 2017-09-15 04:49:44	0|jonasschnelli|Can we just get rid of Qt4 support? Qt 4.8 seems unmaintaned since a couple of years... very bad for a cross-platform layer
168 2017-09-15 04:51:11	0|luke-jr|no objection from me anymore
169 2017-09-15 04:51:13	0|luke-jr|fwiw
170 2017-09-15 04:51:16	0|gmaxwell|I dont even understand why this is coming up here.
171 2017-09-15 04:51:31	0|gmaxwell|Roughly the same code is in QT4. so it should be fine too.
172 2017-09-15 04:52:58	0|jonasschnelli|Unrelated to the offscreen bug, yes.
173 2017-09-15 04:53:27	0|jonasschnelli|Switching to Qt5 would allow to remove legacy code and use Qt5 features like lambdas, etc.
174 2017-09-15 04:53:30	0|jonasschnelli|Simpler signal bindings
175 2017-09-15 04:53:46	0|gmaxwell|Well, on all my machines Qt4 is the only version installed I think. I think it's still the default even in debian testing (though QT5 is available there).
176 2017-09-15 04:57:33	0|luke-jr|oh :/
177 2017-09-15 04:57:50	0|luke-jr|Now that you mention it, I do recall seeing Qt4 on Devuan
178 2017-09-15 04:59:04	0|gmaxwell|I'm no expert and perhaps a fresh install would change some default.  And maybe we could just address by providing more install instructions.
179 2017-09-15 04:59:32	0|achow101|gmaxwell: perhaps you need a new laptop :p
180 2017-09-15 05:03:11	0|gmaxwell|your inside jokes might confuse people. :P  fwiw, on an up to date debian testing system, which was QT4 only, you can parallel install QT5, but it seems bitcoin-qt still builds using QT4 even with QT5 'installed'.
181 2017-09-15 05:49:20	0|meshcollider|hmmm it seems that its working, but if you move it right to the very very edge of the screen you literally cant see it, you can grab it again with the mouse but theres no way you'd be able to find it again lol
182 2017-09-15 05:49:35	0|meshcollider|that'd be a rare case though I'd imaging
183 2017-09-15 05:49:43	0|meshcollider|s/imaging/imagine/
184 2017-09-15 05:52:38	0|achow101|meshcollider: that's exactly the case that gmaxwell and I were testing earlier
185 2017-09-15 05:53:25	0|achow101|and that case is kind of the worst case scenario, not off screen, but not on screen enough to be usable
186 2017-09-15 06:10:19	0|meshcollider|mhm but if we rely on Qt restore and save, its not really fixable by us. I'd say its a non-issue if its exactly how other OS programs behave like notepad?
187 2017-09-15 06:10:57	0|meshcollider|As long as its possible to get it back on screen with just the mouse, which it is, as long as you can find it lol
188 2017-09-15 06:21:47	0|gmaxwell|bleh, I suppose. why do other things find that behavior tolerable.
189 2017-09-15 06:22:30	0|sipa|there is some key shortcut to move a window
190 2017-09-15 06:23:02	0|cfields_|ctrl+space gives you the min/max/move context-menu
191 2017-09-15 06:23:20	0|cfields_|er, alt+space
192 2017-09-15 06:23:33	0|meshcollider|yeah or you can shift+rightclick on the icon in the taskbar
193 2017-09-15 06:24:41	0|cfields_|and fwiw, i only know that because programs would end up out-of-view now and then
194 2017-09-15 06:27:36	0|gmaxwell|things really can get out of view and then save and restore it across restarts... how the heck have they not fixed that.
195 2017-09-15 06:31:03	0|meshcollider|huh I figured out an even easier way to get it back on screen, windows key + arrow
196 2017-09-15 06:31:06	0|cfields_|gmaxwell: well i haven't messed with it since xp, I assume they've made some progress :)
197 2017-09-15 06:33:47	0|midnightmagic|meh. it's qt. they only fix stuff for corps that have an upper-tier support contract.
198 2017-09-15 06:34:32	0|cfields_|midnightmagic: regardless of qt, the window manager is ultimately in charge of positioning
199 2017-09-15 06:34:42	0|cfields_|which i assume was gmaxwell's point
200 2017-09-15 06:37:08	0|meshcollider|^ use a tiling window manager and all your problems disappear ;)
201 2017-09-15 06:45:35	0|midnightmagic|cfields_: no, memory of a window position is not something dwm does, and I use dwm-- "we" had issues with qt windows doing stupid things, fixing it was a pointless chore.
202 2017-09-15 06:46:57	0|midnightmagic|xterm positioning for a new window is similar-- you pass in X geometry, and/or dwm guesses a top-left position for it, that's pretty much it
203 2017-09-15 06:49:20	0|midnightmagic|eh, I might be misremembering. we had a lot of qt problems. a lot. lotta lot..
204 2017-09-15 07:53:05	0|bitcoin-git|13bitcoin/06master 14dc2f737 15danra: Trivial: Fix comments for DEFAULT_WHITELIST[FORCE]RELAY
205 2017-09-15 07:53:05	0|bitcoin-git|[13bitcoin] 15MarcoFalke pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/0f399a9ff227...ae233c4ec3d1
206 2017-09-15 07:53:06	0|bitcoin-git|13bitcoin/06master 14ae233c4 15MarcoFalke: Merge #11330: Trivial: Fix comments for DEFAULT_WHITELIST[FORCE]RELAY...
207 2017-09-15 07:53:46	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #11330: Trivial: Fix comments for DEFAULT_WHITELIST[FORCE]RELAY (06master...06patch-10) 02https://github.com/bitcoin/bitcoin/pull/11330
208 2017-09-15 07:55:55	0|bitcoin-git|13bitcoin/06master 14cdaf3a1 15Matt Corallo: Fix Qt 0.14.2->0.15.0 segfault if "total at least" is selected...
209 2017-09-15 07:55:55	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 06master: 02https://github.com/bitcoin/bitcoin/compare/ae233c4ec3d1...09627b1dd41d
210 2017-09-15 07:55:56	0|bitcoin-git|13bitcoin/06master 1409627b1 15Wladimir J. van der Laan: Merge #11332: Fix possible crash with invalid nCustomFeeRadio in QSettings (achow101, TheBlueMatt)...
211 2017-09-15 07:56:36	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #11332: Fix possible crash with invalid nCustomFeeRadio in QSettings (achow101, TheBlueMatt) (06master...062017/09/qsettings_1) 02https://github.com/bitcoin/bitcoin/pull/11332
212 2017-09-15 07:57:40	0|bitcoin-git|13bitcoin/060.15 1446c8d23 15Matt Corallo: Fix Qt 0.14.2->0.15.0 segfault if "total at least" is selected...
213 2017-09-15 07:57:40	0|bitcoin-git|[13bitcoin] 15laanwj pushed 1 new commit to 060.15: 02https://github.com/bitcoin/bitcoin/commit/46c8d23dad216d990564014f4c8000b79fd36f4e
214 2017-09-15 08:18:17	0|bitcoin-git|[13bitcoin] 15danra opened pull request #11337: Fix code constness in CBlockIndex::GetAncestor() overloads (06master...06fix/const-get-ancestor) 02https://github.com/bitcoin/bitcoin/pull/11337
215 2017-09-15 08:44:40	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #11208: Fixing offscreen GUI issue (06master...06201709_offscreen_fix) 02https://github.com/bitcoin/bitcoin/pull/11208
216 2017-09-15 08:52:17	0|wumpus|meshcollider: "mhm but if we rely on Qt restore and save, its not really fixable by us" <- well, as we never managed t oget it right in years of trying, please don't be angry at me for thinking maybe the qt devs know more about handling this and can do this better? :)
217 2017-09-15 08:52:50	0|meshcollider|wumpus: yeah im not saying we should do it, I'm just saying its their problem now :)
218 2017-09-15 08:53:00	0|wumpus|we can try this, give it one try, if it's flaky as well, I'd say we remove the functionality
219 2017-09-15 08:53:17	0|meshcollider|their restore and save function have a lot more benefits than ours anyway, like the maximisation issue I mentioned in my PR
220 2017-09-15 08:53:22	0|wumpus|it's not worth spending too much cycles on
221 2017-09-15 08:53:54	0|wumpus|but I'd bet quite a lot that their function received significantly more testing than us
222 2017-09-15 08:54:00	0|wumpus|ours*
223 2017-09-15 08:55:46	0|wumpus|I don't understand why this triggers yet another qt 4 discussion though, the function has existed for about forever
224 2017-09-15 08:55:50	0|wumpus|or is there some subtlety there?
225 2017-09-15 09:00:15	0|promag|what does the "n" prefix means in nRPCConsoleWindow ?
226 2017-09-15 09:08:22	0|wumpus|"number" I guess
227 2017-09-15 09:09:27	0|wumpus|all of the qt settings have weird names, AFAIK it's based on satoshi's naming scheme for settings when they were still in the wallet
228 2017-09-15 09:10:10	0|wumpus|anyhow it doesn't really matter what the names are, as long as the names are unique within bitcoin-qt
229 2017-09-15 09:10:54	0|wumpus|for new settings I'd say use a plainer scheme
230 2017-09-15 09:11:10	0|meshcollider|its a perfect time to change the nRPCConsoleWindow  and nWindow ones if you want to because theyre both new to this PR
231 2017-09-15 09:11:22	0|meshcollider|nRPCConsoleWindowGeometry and nWindowGeometry
232 2017-09-15 09:11:27	0|wumpus|but I don't care, no one is supposed to see them
233 2017-09-15 09:11:45	0|wumpus|right
234 2017-09-15 09:12:33	0|promag|yeah, I would rename since they hold a different value/type
235 2017-09-15 09:13:05	0|promag|the suffix "Geometry" sounds enough
236 2017-09-15 09:14:04	0|wumpus|yes, it's enough
237 2017-09-15 09:14:21	0|meshcollider|so just get rid of the 'n' at the front?
238 2017-09-15 09:14:25	0|meshcollider|RPCConsoleWindowGeometry and WindowGeometry ?
239 2017-09-15 09:14:30	0|wumpus|sounds good to me
240 2017-09-15 09:14:44	0|wumpus|maybe MainWindowGeometry
241 2017-09-15 09:15:43	0|promag|was going to say the same
242 2017-09-15 09:16:59	0|meshcollider|sweet done 👍
243 2017-09-15 09:19:49	0|promag|so the old settings will stay?
244 2017-09-15 09:20:46	0|meshcollider|the old settings weren't called Geometry anyway, they saved position and size seperately in nWindowPos and nWindowSize
245 2017-09-15 09:21:36	0|meshcollider|But yeah those Pos and Size will stay in there unused
246 2017-09-15 09:22:52	0|wumpus|adding upgrade handling code just to remove them sounds like overkill
247 2017-09-15 09:35:08	0|promag|and for some reason the user can keep 2 versions installed
248 2017-09-15 09:43:12	0|wumpus|right, or some sinister altcoin that uses the same settings directory :-)
249 2017-09-15 09:57:32	0|bitcoin-git|[13bitcoin] 15laanwj opened pull request #11338: qt: Backup former GUI settings on `-resetguisettings` (06master...062017_10_backup_resetguisettings) 02https://github.com/bitcoin/bitcoin/pull/11338
250 2017-09-15 10:11:12	0|meshcollider|Are there any other goals for 0.15.1 other than the segwit wallet support sipa is working on?
251 2017-09-15 10:15:54	0|bitcoin-git|[13bitcoin] 15danra opened pull request #11339: Minor improvements to checkpoints code (06master...06refactor/checkpoints) 02https://github.com/bitcoin/bitcoin/pull/11339
252 2017-09-15 10:16:28	0|supay|i was wondering if it would be possible to create a contact system where individuals grant access to another individuals contact information on top of the blockchain
253 2017-09-15 10:16:39	0|supay|i believe a peer-to-peer ledger recording the level of access granted or revoked could enable an open social graph
254 2017-09-15 10:16:54	0|supay|once there is a clean method to grant and update contact information across users, there is also now a social graph with information tiering, allowing a user-controlled and defined connection level between individuals.
255 2017-09-15 10:17:04	0|meshcollider|supay: wrong channel, try #bitcoin
256 2017-09-15 10:17:59	0|supay|meshcollider: tried :(
257 2017-09-15 10:20:30	0|venzen|supay: Namecoin provides namespaces for ID information
258 2017-09-15 10:21:56	0|supay|venzen: but no open social graph!
259 2017-09-15 10:22:15	0|supay|on #bitcoin ali1234 mentioned that there are still analog holes
260 2017-09-15 10:22:31	0|meshcollider|supay venzen: this channel is only for discussion of bitcoin core, please take this discussion to another channel
261 2017-09-15 10:22:32	0|supay|but i see this as a powerful p2p contact exchange and/or social graph
262 2017-09-15 10:22:39	0|venzen|supay: we shouldn't really discuss that here, it's off-topic for this dev channel
263 2017-09-15 10:22:57	0|supay|venzen: can i pm?
264 2017-09-15 10:23:14	0|venzen|sure
265 2017-09-15 10:36:26	0|wumpus|meshcollider: no other features at least, bugfixes are always good of course
266 2017-09-15 10:44:20	0|bitcoin-git|[13bitcoin] 15danra opened pull request #11340: Trivial: Fix validation.cpp comment, BIP113 already deployed (06master...06patch-12) 02https://github.com/bitcoin/bitcoin/pull/11340
267 2017-09-15 11:15:30	0|bitcoin-git|[13bitcoin] 15danra closed pull request #11331: Trivial: Update outdated comment in validation.h (06master...06patch-11) 02https://github.com/bitcoin/bitcoin/pull/11331
268 2017-09-15 11:15:40	0|bitcoin-git|[13bitcoin] 15danra closed pull request #11328: Move comment about transaction/block weight calculation (06master...06fix/weight-comment) 02https://github.com/bitcoin/bitcoin/pull/11328
269 2017-09-15 11:27:17	0|aj|wumpus: hey, now 0.15 is out, should i ping back about #10996 (network.conf) ? we were worrying about qt/bitcoind initialisation diverging and generally making that stuff harder to understand, and potentially tests and such
270 2017-09-15 11:27:19	0|gribble|https://github.com/bitcoin/bitcoin/issues/10996 | [WIP] Add per-network config file network.conf by ajtowns · Pull Request #10996 · bitcoin/bitcoin · GitHub
271 2017-09-15 12:16:30	0|wumpus|aj: yes, agreed
272 2017-09-15 12:20:45	0|wumpus|so, do a 0.15.0.1 with just #11332 or merge #11335 and maybe #11338 as well?
273 2017-09-15 12:20:47	0|gribble|https://github.com/bitcoin/bitcoin/issues/11332 | Fix possible crash with invalid nCustomFeeRadio in QSettings (achow101, TheBlueMatt) by jonasschnelli · Pull Request #11332 · bitcoin/bitcoin · GitHub
274 2017-09-15 12:20:48	0|gribble|https://github.com/bitcoin/bitcoin/issues/11335 | Replace save|restoreWindowGeometry with Qt functions by MeshCollider · Pull Request #11335 · bitcoin/bitcoin · GitHub
275 2017-09-15 12:20:50	0|gribble|https://github.com/bitcoin/bitcoin/issues/11338 | qt: Backup former GUI settings on `-resetguisettings` by laanwj · Pull Request #11338 · bitcoin/bitcoin · GitHub
276 2017-09-15 12:21:40	0|wumpus|all are necessary qt gui changes, though only the former is a crash issue
277 2017-09-15 12:53:33	0|MarcoFalke|What is the deadline for 0.15.0.1?
278 2017-09-15 12:53:54	0|MarcoFalke|I might prepare some qa backports, so that people won't run into the intermittent errors we saw on travis
279 2017-09-15 12:54:34	0|wumpus|I don't know - would be nice to have something as soon as possible to recommend to people experiencing crashes, though
280 2017-09-15 12:54:53	0|wumpus|that's the motivation for it
281 2017-09-15 12:55:34	0|MarcoFalke|hmm, in which case it should be tagged today. To give some time for builders
282 2017-09-15 12:56:55	0|wumpus|if we only include 11338 do we even need a RC phase?
283 2017-09-15 12:57:25	0|wumpus|I guess it would be better to do one anyway, a very short one
284 2017-09-15 12:58:16	0|wumpus|ok, will do the version bump, update translations and tag rc1
285 2017-09-15 13:00:22	0|bitcoin-git|[13bitcoin] 15MarcoFalke opened pull request #11341: [0.15.0] Bump manpages (060.15...06Mf1709-docMan15) 02https://github.com/bitcoin/bitcoin/pull/11341
286 2017-09-15 13:00:43	0|MarcoFalke|wumpus: Someone complained about the manpages ^
287 2017-09-15 13:02:02	0|wumpus|ah yes, was just about to do that
288 2017-09-15 13:02:19	0|wumpus|but need after the 0.15.0.1 bump
289 2017-09-15 13:05:13	0|MarcoFalke|Maybe we don't need an rc phase for this 1 LOC change...
290 2017-09-15 13:05:40	0|wumpus|yes, maybe not
291 2017-09-15 13:05:57	0|MarcoFalke|If it is broken (despite testing) don't push the binaries.
292 2017-09-15 13:06:24	0|wumpus|and make a 0.15.0.2? :)
293 2017-09-15 13:07:47	0|MarcoFalke|ja :)
294 2017-09-15 13:09:16	0|promag|wumpus: do you think the backup filename should have the datetime?
295 2017-09-15 13:11:05	0|promag|also, if the file already exists it is appended, not really a backup imo
296 2017-09-15 13:11:37	0|wumpus|no need for a datetime, it's a one-time thing
297 2017-09-15 13:11:38	0|promag|*appended as in the settings are added or updated
298 2017-09-15 13:11:45	0|wumpus|oh, good point, it should be cleared first
299 2017-09-15 13:12:05	0|promag|thats why I thought about the timestamp
300 2017-09-15 13:12:54	0|wumpus|I'll just add a dst.clear()
301 2017-09-15 13:14:32	0|wumpus|I prefer a fixed name as I don't want people to end up with zillions of files in their datadir
302 2017-09-15 13:16:06	0|promag|allright
303 2017-09-15 13:16:37	0|promag|nit: backupSettings(filename, src) swap: backupSettings(const QSettings &src, const fs::path &filename)? :P
304 2017-09-15 13:16:46	0|wumpus|why?
305 2017-09-15 13:17:19	0|promag|ah I see you prefer dst, src
306 2017-09-15 13:17:22	0|wumpus|yes
307 2017-09-15 13:17:35	0|promag|why? :P
308 2017-09-15 13:18:00	0|wumpus|don't feel like arguing this
309 2017-09-15 13:19:12	0|wumpus|both options are equally valid and it really doesn't matter here, it's an internal function not an official API
310 2017-09-15 13:19:47	0|bitcoin-git|[13bitcoin] 15MarcoFalke closed pull request #11339: Minor improvements to checkpoints code (06master...06refactor/checkpoints) 02https://github.com/bitcoin/bitcoin/pull/11339
311 2017-09-15 13:22:10	0|warren|Does bitcoin-cli have any tool that will tell you the feerate of a rawtx?
312 2017-09-15 13:22:31	0|bitcoin-git|13bitcoin/060.15 148432332 15Wladimir J. van der Laan: build: Bump version to 0.15.0.1...
313 2017-09-15 13:22:31	0|bitcoin-git|[13bitcoin] 15laanwj pushed 2 new commits to 060.15: 02https://github.com/bitcoin/bitcoin/compare/46c8d23dad21...7123b95bea5b
314 2017-09-15 13:22:32	0|bitcoin-git|13bitcoin/060.15 147123b95 15Wladimir J. van der Laan: doc: Bump manpages to 0.15.0.1...
315 2017-09-15 13:24:21	0|wumpus|warren: no, as that information is not in a raw transaction, it'd need the parent transactions (which aren't available without txindex)
316 2017-09-15 13:25:20	0|promag|unless the transaction is in your wallet
317 2017-09-15 13:28:02	0|promag|wumpus: in #11338 `T &x` should be `T& x`
318 2017-09-15 13:28:04	0|gribble|https://github.com/bitcoin/bitcoin/issues/11338 | qt: Backup former GUI settings on `-resetguisettings` by laanwj · Pull Request #11338 · bitcoin/bitcoin · GitHub
319 2017-09-15 13:28:20	0|wumpus|sure, but in that case you don't need a call that works on raw transactions, but would just use the wallet api...
320 2017-09-15 13:29:02	0|wumpus|promag: can you please do minor nits the issue instead of IRC, otherwise I'm bound to forget about them
321 2017-09-15 13:30:22	0|bitcoin-git|[13bitcoin] 15laanwj closed pull request #11341: [0.15.0] Bump manpages (060.15...06Mf1709-docMan15) 02https://github.com/bitcoin/bitcoin/pull/11341
322 2017-09-15 13:37:40	0|promag|wumpus: sure, done
323 2017-09-15 13:37:53	0|wumpus|thanks
324 2017-09-15 13:49:14	0|bitcoin-git|13bitcoin/060.15 14f266f26 15Wladimir J. van der Laan: qt: Translations update pre-0.15.0.1...
325 2017-09-15 13:49:14	0|bitcoin-git|[13bitcoin] 15laanwj pushed 1 new commit to 060.15: 02https://github.com/bitcoin/bitcoin/commit/f266f2668bf8fd73015bf8265f0d00f5e858b577
326 2017-09-15 13:55:09	0|achow101|so is 0.15.0.1 just #11332?
327 2017-09-15 13:55:11	0|gribble|https://github.com/bitcoin/bitcoin/issues/11332 | Fix possible crash with invalid nCustomFeeRadio in QSettings (achow101, TheBlueMatt) by jonasschnelli · Pull Request #11332 · bitcoin/bitcoin · GitHub
328 2017-09-15 13:55:46	0|wumpus|yes
329 2017-09-15 13:57:44	0|promag|Why #11332 removes nCustomFeeRadio usage? achow101
330 2017-09-15 13:57:50	0|gribble|https://github.com/bitcoin/bitcoin/issues/11332 | Fix possible crash with invalid nCustomFeeRadio in QSettings (achow101, TheBlueMatt) by jonasschnelli · Pull Request #11332 · bitcoin/bitcoin · GitHub
331 2017-09-15 13:57:55	0|achow101|promag: no need for it
332 2017-09-15 13:58:19	0|promag|no need for the checkbox?
333 2017-09-15 13:58:19	0|wumpus|promag: see https://github.com/bitcoin/bitcoin/pull/11334
334 2017-09-15 13:58:21	0|achow101|it was only to store which of the custom fee radio buttons were selected, but now there is only one option, so no need for it
335 2017-09-15 13:59:20	0|achow101|the checkbox is unrelated to that
336 2017-09-15 14:00:04	0|promag|ops, meant radio button
337 2017-09-15 14:01:41	0|bitcoin-git|[13bitcoin] 15laanwj pushed 1 new commit to 060.15: 02https://github.com/bitcoin/bitcoin/commit/fb7b5293844ea6adc5dcf5ad0a0c5890b4495939
338 2017-09-15 14:01:42	0|bitcoin-git|13bitcoin/060.15 14fb7b529 15Wladimir J. van der Laan: doc: Release notes for 0.15.0.1...
339 2017-09-15 14:02:19	0|promag|achow101: remove ui->labelCustomPerKilobyte->setEnable?
340 2017-09-15 14:02:54	0|promag|IIRC setEnabled on labels does nothing
341 2017-09-15 14:02:56	0|achow101|promag: no, that's required to make that row grayed out if checkBoxMinimumFee is checked
342 2017-09-15 14:02:59	0|achow101|I think
343 2017-09-15 14:07:20	0|wumpus|yes the proper fix needs more review, which is why we went with the minimal fix that avoids the crash in 0.15.0.1
344 2017-09-15 14:07:51	0|wumpus|* [new tag]         v0.15.0.1 -> v0.15.0.1
345 2017-09-15 14:23:46	0|promag|wumpus: do you think Copy/BackupSettings should be in src/qt/guiutil.cpp since they are "general function"
346 2017-09-15 14:24:02	0|wumpus|not as long as they're only used in one place IMO
347 2017-09-15 14:24:25	0|wumpus|guiutil is a grab-bag for common shared stuff
348 2017-09-15 14:24:46	0|wumpus|if something is used in only one compilation unit, better to have it static and local
349 2017-09-15 14:24:58	0|wumpus|can always be moved later
350 2017-09-15 14:25:04	0|wumpus|let's not try to overdesign this anyhow
351 2017-09-15 14:25:53	0|wumpus|another case of both choices being valid, it doesn't really matter
352 2017-09-15 14:31:47	0|meshcollider|sipa: why didn't you choose a 2 character HRP for bech32 regtest addresses if testnet and mainnet are both 2 characters? 'rb' or something?
353 2017-09-15 14:36:20	0|wumpus|probably because he didn't want to 'burn' a 2 character code on regtest, which is only meant for local testing
354 2017-09-15 14:37:01	0|wumpus|unlike testnet3 and mainnet it's not an 'official' network
355 2017-09-15 15:00:44	0|morcos|achow101: while you're playing around in the QT code, i think we want to eliminate the checkbox for "Pay only the required fee ..."
356 2017-09-15 15:01:00	0|morcos|maybe separate PR in case we want to back port 11334
357 2017-09-15 15:02:22	0|achow101|why?
358 2017-09-15 15:06:34	0|morcos|sorry, why what?
359 2017-09-15 15:12:21	0|achow101|morcos: why remove that checkbox? It just means that you will be paying the minrelayfee
360 2017-09-15 15:12:32	0|achow101|(although the wording on it probably needs to be changed)
361 2017-09-15 15:13:16	0|morcos|achow101: because you can just select the minrelayfee in the custom box if you want to pay that
362 2017-09-15 15:13:37	0|morcos|it doesn't seem like selecting it as a stand alone option really sends the right message
363 2017-09-15 15:13:43	0|achow101|morcos: the user may not know what the minrelayfee is
364 2017-09-15 15:14:15	0|achow101|also, doesn't it change if the mempool is full?
365 2017-09-15 15:14:24	0|morcos|that number does not change
366 2017-09-15 15:14:33	0|morcos|but regardless of getting rid of that checkbox
367 2017-09-15 15:14:59	0|morcos|i am realizing that it's a bit annoying that you don't know what the minimum is you need to put on your transaction if you are selecting current fee
368 2017-09-15 15:15:14	0|morcos|perhaps we want to output a : minimum to currently be accepted in the mempool fee
369 2017-09-15 15:15:29	0|morcos|but i would suggest people be forced to type that into the box
370 2017-09-15 15:15:46	0|morcos|making it too easy to select that implies it could could be a good idea
371 2017-09-15 15:15:53	0|morcos|unless you're seeing that returned in fee estimation
372 2017-09-15 15:15:57	0|morcos|it's proably not a good idea
373 2017-09-15 15:16:49	0|achow101|we could change it to update the custom fee box with the fee rate that it would be using
374 2017-09-15 15:17:35	0|morcos|not sure i follow
375 2017-09-15 15:17:45	0|morcos|it uses the fee rate you give it
376 2017-09-15 15:18:00	0|morcos|if you're using custom fee (as opposed to recommended)
377 2017-09-15 15:18:07	0|morcos|even if such fee would not get you accepted to the mempool
378 2017-09-15 15:19:08	0|achow101|I meant that if you check the box to use minrelayfee, it can display the fee rate that it is using in the custom fee rate box right above it
379 2017-09-15 15:19:45	0|morcos|i just don't think we should give users an option to use the minimum
380 2017-09-15 15:20:28	0|morcos|it has very limited benefit... the options should be..  recommended (for a given target) or you specify
381 2017-09-15 15:20:42	0|morcos|we discussed in SF, and i thought there was other agreement with this idea
382 2017-09-15 15:22:02	0|bitcoin-git|[13bitcoin] 15danra opened pull request #11342: Add sanity assert in CheckSequenceLocks (06master...06patch-13) 02https://github.com/bitcoin/bitcoin/pull/11342
383 2017-09-15 15:22:09	0|achow101|I'm afraid that if someone wants to use the minimum that they will accidentally enter 1 sat/kB (not 1000 sat/kB) in the box if they don't have the option to use the minimum
384 2017-09-15 15:22:25	0|morcos|well, in that case it'll be changed to 1000 for them
385 2017-09-15 15:22:28	0|achow101|although I suppose that can be fixed by lower bounding the box to the minrelayfee
386 2017-09-15 15:23:45	0|morcos|it already lower bounds.. but it doesn't really tell you it's going to
387 2017-09-15 15:24:27	0|morcos|in any case, i suppose there are a lot of options for how to improve this area...  so maybe its not critical right now
388 2017-09-15 15:28:02	0|gmaxwell|you could also put 0 in the custom box and get clamped to min relay fee. no
389 2017-09-15 15:28:51	0|gmaxwell|oh nevermind
390 2017-09-15 15:29:21	0|luke-jr|another reason I thought of, that people might want to keep it: this way you can check the min fee checkbox, while retaining the current saved value for the manual fee
391 2017-09-15 15:29:40	0|luke-jr|but that use case might be better served by allowing users to define pre-set values
392 2017-09-15 15:30:23	0|morcos|gmaxwell: yes that does work
393 2017-09-15 15:31:42	0|gmaxwell|the nevermind was because on the line before mine you pointe dthat out. :)
394 2017-09-15 15:32:00	0|morcos|i'm not sure of the QT mojo to do this easily, but seems like the best outcome might be to replace the checkbox with a display of what the mempool min fee is, and have a way to click that and have it populated into the custom box
395 2017-09-15 15:32:52	0|morcos|then we're missing informing people of what the min relay fee is in case they actually wanted to do a tx that wouldn't yet be accepted into the mempool, but that seems pretty advanced, and maybe people will know that if they want to do it
396 2017-09-15 15:33:36	0|morcos|submitting these super low fee txs does have the use case for manually managed rbf bumping
397 2017-09-15 16:44:00	0|bitcoin-git|[13bitcoin] 15isghe opened pull request #11343: added `-walletallowsymboliclink` (default false) (060.15...06wallet-allow-symbolic-link) 02https://github.com/bitcoin/bitcoin/pull/11343
398 2017-09-15 17:30:37	0|warren|Yikes. I had been using my wallet as a symbolic link for years now.
399 2017-09-15 17:31:11	0|sipa|i wasn't even aware that could work...
400 2017-09-15 17:31:44	0|warren|It worked just fine.
401 2017-09-15 17:41:06	0|luke-jr|it shouldn't..?
402 2017-09-15 17:44:36	0|warren|It worked for me for years. #10885 sounds like it got more complicated with the possibility of the same wallet being opened multiple times after multi wallet was added
403 2017-09-15 17:44:40	0|gribble|https://github.com/bitcoin/bitcoin/issues/10885 | Reject invalid wallets by promag · Pull Request #10885 · bitcoin/bitcoin · GitHub
404 2017-09-15 17:44:57	0|warren|was there concern about single wallet being broken with a symlink? that worked for me for years
405 2017-09-15 17:47:42	0|luke-jr|warren: the concern is situations like your own: it doesn't work in a way that is useful
406 2017-09-15 17:47:49	0|gmaxwell|wallets being on different media than the /database directory will result in corruption.
407 2017-09-15 17:47:52	0|luke-jr|warren: wallet.dat being on an encrypted fs doesn't provide you any security
408 2017-09-15 17:49:45	0|BlueMatt|I was like "oh, please dont do that...I mean will usually work, but if your system crashes, you may get fucked"
409 2017-09-15 17:51:27	0|warren|OK, maybe I was lucky to never run into problems.
410 2017-09-15 17:52:31	0|BlueMatt|well we have a loop that compacts wallet like every 100ms, and (usually) if you crash post-compact pre-writing-new-stuff you're ok, but bdb makes no guarantees there, and certainly you can get screwed if you miss the time window there
411 2017-09-15 17:58:02	0|luke-jr|and I'd expect the wallet data to be written to the unencrypted database/ dir before getting into wallet.dat
412 2017-09-15 17:58:22	0|BlueMatt|i believe it does, yes
413 2017-09-15 17:58:25	0|BlueMatt|or the .log file
414 2017-09-15 17:58:49	0|gmaxwell|everything written to wallet.dat first goes through the database/ dir. IIRC
415 2017-09-15 18:03:44	0|delinquentme|running bitcoind w the option --assumevalid=<blah>  but Im not seeing an update in the   "verificationprogress" ... If i've given it a block that is only hours old ... shouldnt that verification shoot up?
416 2017-09-15 18:04:23	0|gmaxwell|Nope.
417 2017-09-15 18:05:17	0|gmaxwell|assume valid means skipping script processing, it doesn't (and cannot) avoid having to process blocks.
418 2017-09-15 18:06:58	0|gmaxwell|also, setting an assumevalid right now won't do much, as the default is a fairly recent.
419 2017-09-15 18:07:58	0|delinquentme|so if I've got a low compute node that I want this on, my best option for speeding up the verification is side loading it?
420 2017-09-15 18:09:14	0|gmaxwell|delinquentme: yes, or just be patient, if your system is fast enough to keep up it will catch up.
421 2017-09-15 18:11:37	0|BlueMatt|also -dbcache
422 2017-09-15 18:12:15	0|gmaxwell|BlueMatt: I didn't suggest that because most cpu starved things don't have ram.
423 2017-09-15 18:13:14	0|BlueMatt|ah, well sure, just saying crank it if possible
424 2017-09-15 18:17:09	0|delinquentme|yeah I've got that specd at 8 gigs on the machine doing the sideloading
425 2017-09-15 18:17:25	0|delinquentme|I've got a total of 16 ... any reason I shouldn't make it 12g?
426 2017-09-15 18:22:25	0|sipa|it won't use that much
427 2017-09-15 18:22:38	0|sipa|i believe it maxes out at around dbcache=6000
428 2017-09-15 18:22:50	0|sipa|but setting it higher won't hurt
429 2017-09-15 18:26:36	0|BlueMatt|ha! https://github.com/bitcoin/bitcoin/pull/11226#issuecomment-329289991 <-- fuck yea, I add locks
430 2017-09-15 18:29:18	0|sdaftuar|everyone needs a hobby! :)
431 2017-09-15 18:29:51	0|StopAndDecrypt_|gmaxwell, how do you measure your sync time data?
432 2017-09-15 18:30:05	0|StopAndDecrypt_|pen and pencil or do you monitor with a tool?
433 2017-09-15 18:30:28	0|BlueMatt|sdaftuar: considering my current push is refactoring the world to remove lockorder dependancies that may actually be true........
434 2017-09-15 18:30:30	0|StopAndDecrypt_|im gonna be setting up 0.15 on a rapberry pi3 in the next few weeks if you want i can try and share the info
435 2017-09-15 18:30:38	0|StopAndDecrypt_|if it would be useful
436 2017-09-15 18:35:30	0|gmaxwell|StopAndDecrypt_: debug log timestamp differences.
437 2017-09-15 18:35:42	0|gmaxwell|you can make the logs have microsecond precision too.
438 2017-09-15 18:36:15	0|achow101|StopAndDecrypt_: Look at the debug.log file and check the timestamps for when you start the sync and when IsInitialBlockDownload is finished
439 2017-09-15 18:37:24	0|StopAndDecrypt_|makes sense, just curious if that info would be useful.
440 2017-09-15 18:40:38	0|bitcoin-git|[13bitcoin] 15jnewbery opened pull request #11345: [tests] Check connectivity before sending in assumevalid.py (06master...06assume_valid_improvement) 02https://github.com/bitcoin/bitcoin/pull/11345
441 2017-09-15 18:59:51	0|BlueMatt|jonasschnelli: do you believe the latest qt only fixed the offscreen issue, or did you also intend to imply that it also fixed the setSortingEnabled thing?
442 2017-09-15 23:33:04	0|jonasschnelli|BlueMatt: I don't think setSortingEnabled is fixed in newer Qt versions