IRC logs for #openttd on OFTC at 2018-03-17
            
00:03:46 *** Stimrol has quit IRC
00:10:00 *** supermop has joined #openttd
00:15:05 *** Wormnest has quit IRC
00:26:55 *** iSoSyS has quit IRC
00:27:06 *** nielsm has quit IRC
00:32:32 *** HerzogDeXtEr has quit IRC
00:39:41 *** iSoSyS has joined #openttd
00:42:00 *** debdog has quit IRC
00:43:07 *** debdog has joined #openttd
00:46:32 *** Progman has quit IRC
00:47:38 *** snail_UES_ has joined #openttd
00:54:46 *** keoz has quit IRC
00:55:20 *** FLHerne has quit IRC
01:02:19 *** glx has quit IRC
01:06:15 *** glx has joined #openttd
01:06:15 *** ChanServ sets mode: +v glx
01:19:25 *** Flygon has joined #openttd
01:48:01 *** SeraphAngel has joined #openttd
02:07:24 *** Thedarkb has quit IRC
02:32:15 *** Thedarkb has joined #openttd
02:56:11 *** iSoSyS has quit IRC
04:21:21 *** SeraphAngel has quit IRC
04:38:35 *** glx has quit IRC
05:30:52 *** snail_UES_ has quit IRC
07:14:35 *** Mahjong1 has quit IRC
07:15:20 *** Mahjong1 has joined #openttd
07:24:59 *** sla_ro|master has joined #openttd
07:30:49 *** supermop has quit IRC
08:02:39 *** nielsm has joined #openttd
08:02:45 *** keoz has joined #openttd
08:07:21 *** andythenorth has joined #openttd
08:15:14 *** Alberth has joined #openttd
08:15:14 *** ChanServ sets mode: +o Alberth
08:15:21 <Alberth> moin
08:17:03 <andythenorth> hi
08:17:25 <Alberth> it's snowing here :)
08:25:15 *** Cubey has quit IRC
08:30:12 *** Progman has joined #openttd
08:39:37 *** [1]sim-al2 has joined #openttd
08:42:36 *** sim-al2 has quit IRC
08:42:37 *** [1]sim-al2 is now known as sim-al2
08:48:44 *** keoz has quit IRC
09:05:09 <andythenorth> sulfur is so brightly coloured https://dddusmma.files.wordpress.com/2012/09/sulfer-barge-1-sm.jpg
09:07:48 <Alberth> :D
09:13:28 <__ln__> which newgrf is that
09:15:15 <andythenorth> it's future Sam
09:15:28 <andythenorth> screenshot used a time machine
09:15:33 <__ln__> i see
09:15:47 <andythenorth> we also introduced arbitrary map rotation in future
09:16:00 <Alberth> including upside down
09:16:06 <Alberth> (not shown in this picture)
09:16:48 <Alberth> I started a new steel game at 256x256, it feels very small :)
09:17:26 <andythenorth> you should try my ships :P
09:17:46 <Alberth> :o indeed
09:17:54 <andythenorth> I like 256x256
09:18:39 <Alberth> I usually play 512x512 or 256x512
09:19:04 <Alberth> hmm, shippy steel game at 128x128 ?
09:19:35 <andythenorth> http://bundles.openttdcoop.org/unsinkable-sam/releases/
09:19:50 <andythenorth> it's not a complete roster in the alpha
09:21:30 <Alberth> let's try
09:36:24 *** gelignite has joined #openttd
09:50:14 *** [1]sim-al2 has joined #openttd
09:50:14 *** sim-al2 is now known as Guest814
09:50:14 *** [1]sim-al2 is now known as sim-al2
09:51:41 *** Guest814 has quit IRC
09:57:35 <TrueBrain> right .. lets see how GitHub auto-import from SubVersion works :D
09:57:43 <TrueBrain> Detecting ..... well, that takes for ever :P
09:59:08 <TrueBrain> okay, this is going to take a while I guess .. lol ..
10:00:05 <TrueBrain> haha: "There was an error importing commits"
10:00:09 <TrueBrain> I am not surprised :D
10:00:42 <TrueBrain> no details nothing
10:05:26 <LordAro> yay
10:13:14 *** iSoSyS has joined #openttd
10:19:18 <andythenorth> I imported to github, but maybe from git :P
10:19:21 <TrueBrain> funny, we used to modify git-svn to do what we want to do :)
10:19:31 <andythenorth> https://github.com/andythenorth/NotRoadTypes
10:19:45 <TrueBrain> do commits start with (svn r..) ?
10:19:56 <andythenorth> yes
10:20:03 <TrueBrain> then you forked from the git import :)
10:20:08 <TrueBrain> (that is very non-default behaviour)
10:20:12 <andythenorth> :P
10:21:01 <TrueBrain> but I really like that, over leaving a git-svn-id to a svn that will be shut down over time ..
10:21:09 <TrueBrain> but it turns out that the patch we used for that, no longer applies :D
10:27:35 <TrueBrain> and nobody understands Perl, so this is difficult :)
10:33:49 <Alberth> quite :(
10:34:14 <TrueBrain> okay .. so that import is running ... that will take a very very very very very very very long time :D
10:35:07 <TrueBrain> most curious if it picks up branches etc correctly
10:35:24 <TrueBrain> do we still have branches on svn .. hmm
10:35:32 <TrueBrain> (except for the release branches)
10:35:47 <TrueBrain> owh, yes, the 'extra' stuff :D
10:35:51 <TrueBrain> they need to go in different repos, I guess
10:36:09 <TrueBrain> no other unexpected stuff anymore, good
10:40:15 <Alberth> old airport stuff perhaps?
10:40:36 <TrueBrain> not in subversion anymore
10:40:50 <Alberth> someone has cleaned up :)
10:40:56 <TrueBrain> yeah, a while back I think
10:41:06 <TrueBrain> hmm .. git svn doesn't tag our tags :D
10:41:26 <TrueBrain> but that is fixable
10:41:41 <TrueBrain> in subversion you make a commit to make a tag, which is a bit odd :D
10:41:59 <Alberth> git likes that too, afaik
10:42:06 <Alberth> but you can do without
10:42:07 <TrueBrain> a tag is not a commit in git
10:42:13 <TrueBrain> just a marker
10:42:47 <Alberth> git tag -s I think?
10:43:03 <Alberth> but makes sense if it's a public tag, like a release
10:43:14 <Alberth> much less if it's an internal tag
10:43:36 <TrueBrain> I guess what I will do is make git-tags out of the tags branches, and remove the branches
10:43:42 <Alberth> oh, git tag -a is the one
10:43:43 <TrueBrain> even if there are some commits on them, the tag will show that
10:44:00 <TrueBrain> should be sufficient information I guess
10:44:21 <Alberth> yeah, it just has to be findable
10:46:29 *** Biolunar has joined #openttd
10:48:04 <TrueBrain> so now, FlySpray import .. lets see :)
10:52:51 <TrueBrain> hmm .. can I make the svn import skip any branches no following certain names .. as .. they will be removed anyway .. wasting time .. I wonder ..
10:53:18 <TrueBrain> seems you cant
10:53:20 <TrueBrain> owh well
10:53:38 <TrueBrain> guess they are part of the history too :P
10:54:44 <Alberth> :)
10:55:40 <TrueBrain> import is at commit 500
10:55:47 <TrueBrain> this is going to take a long while :P
10:56:08 <LordAro> and fail at commit 19k
10:56:09 <nielsm> so why are you making your own anyway?
10:56:22 <TrueBrain> "my own"
10:56:31 <TrueBrain> LordAro: why do you make me sad :(
10:56:42 <LordAro> TrueBrain: i try
10:56:56 <TrueBrain> nielsm: if I am not going to do it, who is going to?
10:57:50 <nielsm> well http://git.openttd.org/openttd/trunk.git/ is an automatically maintained gitsvn, syncing up to new svn adds?
10:59:06 <TrueBrain> well, it is automatic, but it didn't came into existance "automatically" :D
10:59:18 <LordAro> nielsm: if you check http://www.openttd.org/en/contact you might get an answer
10:59:23 <nielsm> true
10:59:30 <TrueBrain> LordAro: don't spoil it :P
10:59:58 <TrueBrain> nielsm: but for git.openttd.org we made the desicion to have all branches etc not follow their parent; this means that we have alot of small git repos
11:00:04 <TrueBrain> I now need one single repo with everything in there
11:00:10 <nielsm> oh
11:00:13 <TrueBrain> I can of course merge those gits together .. but that is more effort ;)
11:00:33 *** Stimrol has joined #openttd
11:00:55 * nielsm feels slightly silly
11:01:03 <TrueBrain> basically, I am trying to prepare a full migration to GitHub, where we shut down svn/git/hg on openttd.org itself
11:01:12 <LordAro> TB has a tendency to do that, don't worry :p
11:01:15 <TrueBrain> LordAro: now look what you did to that poor man!
11:01:26 <TrueBrain> or girl .. but niels suggests man :P
11:01:40 <TrueBrain> well .. girl vs man is a bit insulting .. not what I meant :P
11:01:57 * LordAro hands TB a shovel
11:02:05 <nielsm> so development would move to using a git repos as the primary source of truth?
11:02:05 <TrueBrain> nah, I did that all by myself, no worries
11:02:12 <TrueBrain> yes
11:02:22 <TrueBrain> and it makes me write PHP .. not sure what I think about this
11:02:30 <LordAro> ono
11:03:03 <TrueBrain> 1000 commits done ..
11:03:12 <TrueBrain> that that makes 10 minutes for 500 commits ..
11:03:17 <TrueBrain> so 50 per minute . .1 per second ..
11:03:25 <TrueBrain> @calc 27993 / 3600
11:03:25 <DorpsGek> TrueBrain: 7.77583333333
11:03:27 <TrueBrain> 7 hours
11:03:28 <TrueBrain> lol
11:03:31 <LordAro> aye, git-svn is not fast
11:03:48 <TrueBrain> I dont mind if it is slow, as long as it does the right thing :D
11:03:53 <LordAro> i tried running it on our svn repo at work once
11:03:56 *** sla_ro|master has quit IRC
11:03:57 <LordAro> with 90k commits
11:03:59 <TrueBrain> we removed commits from subversion .. and some tools really really do not fancy that
11:04:12 <LordAro> luckily it died quickly and i gave up
11:04:16 <TrueBrain> (you cannot find commit r10000, for example)
11:04:20 <andythenorth> could just reset history :P
11:04:23 <LordAro> haha, yes
11:04:28 <andythenorth> we could 'lose' 19k commits
11:04:33 <andythenorth> there is precedent :P
11:04:41 <TrueBrain> it was not 19k commits!
11:04:43 <TrueBrain> just 600!
11:04:47 <TrueBrain> (which was a big deal back then)
11:04:54 <TrueBrain> also the moment I took over hosting :P
11:05:15 <TrueBrain> we could get those commits back if we paid some weird dude money ..
11:05:18 <TrueBrain> it was that bad :P
11:05:24 <TrueBrain> (basically the data was being kept hostage)
11:05:28 <LordAro> r1 implies 975 commits
11:05:36 <TrueBrain> you are correct
11:05:39 <TrueBrain> still not 19k :P
11:05:42 <LordAro> true
11:05:49 <nielsm> so just going to ask again, I'd like someone to review my new music decoding/etc code, mainly for stylistic issues but anything goes really: https://gist.github.com/nielsmh/b0ebe5338a86d0dd0e7b6fdbabd9e230
11:06:09 <TrueBrain> was also the moment btw that we started to backup subversion to 3 locations .. every night a full copy of the full subversion is being sent to other locations
11:06:13 <nielsm> any pointers for things to improve or watch out for would be appreciated :)
11:06:15 <TrueBrain> both geo-locations as owners :P
11:06:59 <LordAro> nielsm: https://wiki.openttd.org/Coding_style i noticed function brace style immediately
11:07:41 <LordAro> no if-not-all-on-one-line without braces
11:08:16 <nielsm> huh and I've looked through that page several times and never noticed my function braces were wrong
11:08:25 <TrueBrain> I am very happy I no longer have a clue how to write PHP .. but this is annoying too :D
11:08:50 <LordAro> nielsm: if-elseif-else braces too
11:08:57 <LordAro> code itself looks alright
11:09:16 <LordAro> not that i've looked in detail
11:09:33 <nielsm> I've exercised the code itself quite a lot so I'm confident it works ;)
11:10:48 <TrueBrain> that is always a good thing :D
11:12:38 <nielsm> I wonder if I can configured VS2015 to not automatically reformat "} else {" into }\nelse {"
11:12:56 <__ln__> nielsm: actually you can
11:15:31 <nielsm> just found it yes
11:17:18 <TrueBrain> too bad you cannot create issues with older dates in GitHub, not even as admin
11:17:22 <TrueBrain> that would have been awesome ...
11:19:40 <andythenorth> found a grain barge :o http://www.bluefish.org/TidewaterBarge.jpg
11:19:52 <andythenorth> life imitates my newgrf :P
11:21:28 <nielsm> now, is there also an option to make VS2015 understand ///< doxygen comments? by default it just parses them as "invalid xml"
11:21:34 <nielsm> because of the <
11:21:45 <andythenorth> :o box wagon barges also https://static.wixstatic.com/media/0642f9_e8ae99e02547467fb5a44a71e3e239ec.jpg/v1/fill/w_572,h_336,al_c,lg_1,q_80/0642f9_e8ae99e02547467fb5a44a71e3e239ec.webp
11:21:49 <LordAro> i'd imagine google will tell you
11:21:56 <LordAro> probably just disable xml comment parsing
11:23:25 <Alberth> xml as comment format is insane anyway
11:23:48 <LordAro> ^
11:24:12 *** synchris has joined #openttd
11:26:10 <nielsm> at the bottom of midifile.cpp I do some hacky things defining two debugging/easter-egg console commands: https://gist.github.com/nielsmh/b0ebe5338a86d0dd0e7b6fdbabd9e230#file-midifile-cpp-L1004
11:26:14 <nielsm> is that acceptable at all?
11:27:21 <nielsm> and, does midifile.cpp belong in the src/music/ directory or somewhere else? it's only related to music but it's not a music driver in itself
11:28:19 <LordAro> it'd be more normal to just move the function above where it's called
11:28:36 <LordAro> the includes should be at the top of the file regardless
11:28:47 <Alberth> I am getting a little lost between all the islands, andy :) http://devs.openttd.org/~alberth/Prenfingfield_transport_1902-10-13.png
11:29:03 <LordAro> i'm not qualified to answer about where the file should actually go
11:29:34 <andythenorth> Alberth: that is some ugly map gen :)
11:29:39 <andythenorth> boggles my eyes
11:30:04 <Alberth> what's worse, there is no room for ships to pass between the islands :)
11:31:08 <Alberth> needs digging
11:34:38 <Alberth> I take it that ship prices are still wip?
11:41:09 <TrueBrain> r3333! Almost there or something :D
11:42:43 <andythenorth> Alberth: very WIP
11:43:02 <Alberth> andy, bow of small tanker seems to change shape from - to /
11:43:12 <Alberth> latter looks much more round
11:44:54 <Alberth> (going to the right, ie -> to /^
11:48:58 <andythenorth> let's test
11:50:13 <andythenorth> it looks like it is sharp in – then blunt in / ?
11:54:37 <Alberth> yes
11:54:42 *** frosch123 has joined #openttd
11:54:49 <Alberth> hola
11:55:55 <Alberth> blunt version is not bad, it's different from many other ships :)
11:56:25 <andythenorth> the – sprite is modfiied from old
11:56:30 <andythenorth> the / is drawn new
11:56:42 <andythenorth> the – should be changed
11:56:54 * andythenorth makes note
11:57:46 <andythenorth> also covered hopper barge http://dev.openttdcoop.org/attachments/download/8913/covered_hopper_barge.png
11:57:51 <andythenorth> v0.0.1
11:58:27 <Alberth> ha :)
11:58:54 <Alberth> drop the last stair to the top floor?
11:58:59 <andythenorth> follows http://dev.openttdcoop.org/attachments/download/8682/horsey_hoppers_more_such.png
12:00:16 <Alberth> not sure the barge works, I can't quite decide where the top is
12:00:33 <Alberth> of the tank, that is
12:00:48 <frosch123> moi
12:01:13 <andythenorth> Alberth: yeah it sucks at the moment
12:01:22 <Alberth> the big deck at the pushing boat throws it off
12:02:15 <andythenorth> the RL edition is quite definitively triangular https://www.snakeriverwaterkeeper.org/wp-content/uploads/2018/01/Barge-Photo.jpg
12:02:24 <andythenorth> and has big walkway along top
12:03:14 <Alberth> oh, you had another one :)
12:03:37 <Alberth> pushing boat looks like an medieval castle tower :)
12:04:51 <nielsm> which of these is preferable? https://gist.github.com/nielsmh/841ec60c99289741a44392d6627644c8
12:06:09 <andythenorth> in – angle, triangle roof is very hard to show :P
12:06:10 <frosch123> the first one
12:06:18 <andythenorth> and in / \ views, triangle is PITA :)
12:06:56 <Alberth> I can imagine
12:07:08 <andythenorth> maybe I flat-top it
12:07:29 <andythenorth> two rows of cargo filling hatches
12:07:35 <nielsm> frosch123 thanks
12:08:29 <Alberth> I'd write it as a for-loop, I think
12:08:52 <frosch123> also sounds good
12:09:04 <nielsm> Alberth well yes but it gets annoying since it's big endian so you have to write it "in reverse"
12:09:17 <TrueBrain> almost r5000 .. that is not too bad .. 4000 per hour
12:09:28 <nielsm> big endian 7 bit per byte, MIDI is wonderful
12:09:41 <andythenorth> hmm
12:09:44 <andythenorth> new ship newgrf
12:09:48 <andythenorth> NewWaterFeatures? o_O
12:10:06 <Alberth> always decode 4 bytes, and keep track of the first non-0 byte
12:10:48 <Alberth> writing in reverse is perhaps simpler
12:11:18 * andythenorth looks for the water pony list
12:11:18 <Alberth> hmm, keeping track of the last non-0 perhaps
12:11:36 <andythenorth> https://github.com/andythenorth/NotWater/issues/1
12:15:54 <TrueBrain> FlySpray allows a "reported version" of another project to exist ....
12:16:02 <TrueBrain> serious, FlySpray database schema is a piece of .....
12:16:44 <Alberth> we should split openttd in lots of small modules!
12:20:59 <nielsm> okay time to get the baseset file handling hacks I've made cleaned up
12:21:35 <nielsm> need some advice on what to do about the .obm files that will need to specify either straight .mid files, or point into .cat files
12:21:42 <nielsm> what I have for the .obm format right now: https://gist.github.com/nielsmh/f03f9ba93bea4a9b558d714a5f9aebf4
12:22:12 <nielsm> orig_win.obm is slightly modified from the one in the game right now, only added the [timingtrim] section
12:22:31 <nielsm> orig_dos.obm is my suggestion for how to handle the .cat file music without needing a wholly different format
12:23:25 <Alberth> https://paste.openttdcoop.org/pyqy4enmf <-- big endian writing example, very untested :)
12:24:03 <Alberth> why not make a section for each file?
12:25:07 <Alberth> would be simpler to extend to more files
12:25:35 <nielsm> I'm not entirely sure what you mean, but this was what I could come up with that doesn't break the entire baseset infrastructure already in place
12:26:20 <TrueBrain> cool .. FlySpray has tasks that are opened by nobody! :D
12:27:06 <Alberth> no idea what infrastructure that is, but you're writing code to load the file, right? couldn't it just load it in the existing infra structure?
12:27:20 <Alberth> secret FS tasks!
12:27:32 <Alberth> could be easter eggs :p
12:27:45 <nielsm> Alberth the problem is that the existing code assumes that each music track is a separate physical file, which is true for the original music from the Windows version of TTD
12:27:54 <nielsm> but the DOS version has all the music packed into a single .CAT file
12:29:14 <Alberth> and? how does that prevent ordering data about one entry in one section?
12:29:16 <Eddi|zuHause> can't you add a layer that treats the .cat file like a directory, like .tar files?
12:30:14 <Alberth> [old1] name=...; md5=...; title=... much simpler and more extensible imho
12:30:38 <nielsm> Alberth yeah then every single other baseset file (defining graphics etc) would also have to be changed ;)
12:30:54 <Alberth> why?
12:30:54 <nielsm> and invalidate existing basesets on bananas
12:31:13 <Alberth> I am just moving the same information around
12:31:17 <nielsm> unless there were two separate code paths reading old format or new format
12:32:03 <Alberth> separate section for the information does not automatically imply change in data format of the storage, imho
12:32:57 <Alberth> If I split a shopping list to 2 pieces of paper, you don't have to go to different shops by definition
12:34:52 <TrueBrain> lol, currently the import is busywith something called MiniIN .. that is a long long time ago, holy crap :D
12:35:14 <Eddi|zuHause> those were the good times :p
12:35:39 <LordAro> haha
12:36:06 <Eddi|zuHause> MiniIN was born when the old PBS code was removed from trunk
12:37:29 <TrueBrain> and I can now fetch all the needed data from FlySpray .. next step, uploading it to GitHub :D
12:37:34 <TrueBrain> first some lunch ..
12:38:01 <Eddi|zuHause> and i need to go outside and drive through the snow... but i really don't want to... :/
12:38:23 <Alberth> game stops me from breaking the "ships only" rule, by not having trucks or trains at the current year :)
12:43:58 <andythenorth> Eddi|zuHause: deliveroo? :P
12:44:07 <andythenorth> or is it more than just getting food?
12:44:30 <Eddi|zuHause> no, extended family gathering
12:45:25 *** gelignite has quit IRC
12:53:05 *** Lejving_ has quit IRC
12:53:26 *** keoz has joined #openttd
13:03:22 * andythenorth might learn how to pipenv https://docs.pipenv.org/basics/
13:03:40 <andythenorth> if it's good, we could maybe wrap nml in it
13:09:38 *** synchris has quit IRC
13:10:50 *** synchris has joined #openttd
13:33:19 <TrueBrain> r9000 .. 1/3rd or so there .. and it is still running :D w00p!
13:35:03 <Rubidium> but will it fail around r10k?
13:35:57 <TrueBrain> I wonder that myself too
13:36:18 <TrueBrain> the GitHub import for sure fails on our svn layout .. I am guessing the missing commits really fucked them over :P
13:36:29 *** Wormnest has joined #openttd
13:40:46 *** Gja has joined #openttd
13:45:09 <TrueBrain> awh, because OpenTTD is now an Organization, I cannot use it as Users to create the tickets via
13:45:23 <TrueBrain> so I need to create a new account I guess to do so ..
13:45:52 <TrueBrain> I guess DorpsGek is expanding his horizon :P
13:45:58 <LordAro> :D
13:54:40 <Alberth> dorpsgek goes github!
13:55:20 *** HerzogDeXtEr has joined #openttd
13:55:51 <TrueBrain> I called it OpenTTD-Bot .. I am getting old :(
13:57:30 <LordAro> butbut.. dorpsgek
13:57:37 <LordAro> doesn't appear to be taken
13:57:50 <TrueBrain> having tickets created by "DorpsGek" might not be very verbose for anyone, I guess
13:58:05 <LordAro> eh
13:58:16 <TrueBrain> then again ...
13:58:22 <TrueBrain> BaNaNaS is neither :D
13:58:29 <LordAro> the google code importer did it by just tagging the issues as imported
13:59:25 *** dvim has quit IRC
13:59:36 <TrueBrain> its done! DorpsGek his name is
13:59:43 <LordAro> \o/
14:00:12 <TrueBrain> it is possible with that I broke the SVN -> GitHub link we currently have, but fuck that :P
14:00:22 <TrueBrain> that "should" work via a deploy-key
14:00:47 <TrueBrain> oeh, r10150 .. so the import doesnt care about missing revisions :D
14:01:04 <LordAro> yay
14:01:31 <TrueBrain> this might not be the most bandwidth efficient method .. I am doing the import on my home machine :D
14:03:09 <LordAro> there can't be that much data
14:07:20 <TrueBrain> subversion? 30k commits?
14:07:43 <LordAro> or rather, it can't be that much all at once that it's bandwidth limited
14:07:50 <TrueBrain> hmm .. when creating an issue on GitHub I get a 404 back, with the message: Not Found, and the link to the documentation ...
14:08:14 <TrueBrain> lot of IO I notice
14:08:18 <TrueBrain> it is just plainfully slow :P
14:15:57 *** tokai|noir has joined #openttd
14:15:57 *** ChanServ sets mode: +v tokai|noir
14:17:13 <TrueBrain> totally puzzled why I cannot create an issue on github ..
14:18:31 <TrueBrain> ah, token didnt have enough permissions
14:19:16 <andythenorth> http://dev.openttdcoop.org/attachments/download/8914/covered_hopper_barge_2.png
14:20:04 <TrueBrain> For 'links/0/schema', nil is not an object
14:20:05 <TrueBrain> lol @ error
14:20:21 <LordAro> well it's not wrong
14:21:00 <Alberth> much better andy
14:21:12 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/1
14:21:15 <TrueBrain> and now the fiddle starts :)
14:21:34 <TrueBrain> owh, and it emails me when DorpsGek makes an issue .. ugh .. that is a lot of spam :P
14:21:40 <LordAro> haha
14:21:50 <LordAro> you can probably disable the emails somewhere
14:21:59 <Alberth> disable notification by email :)
14:22:07 <LordAro> also, presumably that issue should be closed
14:22:39 <TrueBrain> yes, comments and closing is yet-to-be-done
14:22:41 *** tokai has quit IRC
14:24:29 <TrueBrain> reload if you had it loaded; suggestions to make the content more clear?
14:24:57 <nielsm> okay think I've settled on doing the baseset music with catfile support like this: https://gist.github.com/nielsmh/a8b0d5457c9422cecc7a0c12a330c69e
14:25:43 <nielsm> doesn't affect the MD5File type or any other handling related to that at all then
14:26:03 <nielsm> only MusicSet and other directly music-related code needs to change now
14:29:44 <andythenorth> TrueBrain: do another comment, more complicated content? o_O
14:30:13 <TrueBrain> I want to get the basics done first, but I will do in a bit
14:36:37 <TrueBrain> I wonder if I should make the 'category' a label instead .. not sure how consistent they are used in FlySpray
14:37:22 *** supermop has joined #openttd
14:39:29 <andythenorth> I went through 800 bugs in 2017
14:39:37 <andythenorth> I'd say the category doesn't add much
14:40:06 <andythenorth> to make sense of issues, I rewrote titles :P
14:40:10 <andythenorth> seemed easier
14:49:48 *** SeraphAngel has joined #openttd
14:50:45 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/5
14:50:49 <TrueBrain> better?
14:53:55 <LordAro> not sure the "Information" header is needed
14:54:18 <TrueBrain> agreed, will remove
14:55:04 <TrueBrain> I wonder if this is also triggering emails in info@ mailbox :D
14:56:09 <frosch123> maybe put the author of the message above their comments
14:56:47 <frosch123> like "Darkvater [2015-12-21 14:41]:\nCH_LAST tells..."
14:58:14 *** Wacko1976 has quit IRC
14:59:02 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/6
14:59:11 <Alberth> suggestion label would be useful, imho
14:59:52 <TrueBrain> suggestion label?
15:00:02 <LordAro> i think i preferred the comment by/at headers
15:00:07 <frosch123> currently it only has "flyspray" label
15:00:19 <LordAro> maybe "Comment by: Bjarni, 08 Decem..."
15:00:27 <frosch123> category and issue type could be labels
15:00:47 <LordAro> ^ can't hurt, even if they are a bit useless
15:01:02 <Alberth> oh "Feature Request" it's called
15:01:10 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/7
15:01:53 <LordAro> still needs something separating the "header"
15:01:58 <LordAro> maybe just bolding it would do?
15:02:07 <TrueBrain> "header" ?
15:02:15 <Alberth> s/@/at/ but seems ok
15:02:17 <TrueBrain> I think this way of author @ date is nice
15:02:36 <TrueBrain> @->at, sure
15:02:46 <TrueBrain> and labels .. hmm .. will check in a bit
15:02:47 <LordAro> the author bit
15:03:54 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/8
15:03:57 <TrueBrain> also with labels
15:04:18 <Alberth> Reason to close: Implemented ?
15:04:29 <LordAro> i like it :)
15:04:44 <TrueBrain> at -> on
15:05:50 <TrueBrain> Alberth: reload, like this?
15:06:15 <Alberth> looks good
15:06:18 <TrueBrain> I like I could keep the links to attachments :D
15:06:42 <LordAro> http://195.112.37.102/ottd/replacepool6.diff shame it wasn't actually an attachment :p
15:06:55 <TrueBrain> ?
15:07:07 <TrueBrain> owh
15:07:08 <TrueBrain> there
15:07:10 <TrueBrain> yeah
15:07:12 <TrueBrain> stupid people being stupid :)
15:07:22 <TrueBrain> I can only keep things working for which I am in control :D
15:07:33 <Alberth> :D
15:07:38 <TrueBrain> Q: I use the username here now; should I add the full name of people?
15:07:41 <TrueBrain> (like FlySpray does)
15:07:59 <Alberth> github username would be another option
15:08:09 <TrueBrain> I don't have them for all our users, sorry :)
15:08:39 <Alberth> I'd leave it at a nick
15:08:50 <frosch123> i only ever use the nicks
15:09:19 <TrueBrain> FlySpray label good enough, or should it read: Imported from FlySpray or something?
15:09:38 <TrueBrain> I need to wait for glx to get here before I import more .. he needs to unwatch the project :P
15:09:39 <frosch123> it's only for filtering, so good enough
15:09:40 <LordAro> it'd be good if they were labels that can be reused later
15:09:53 <TrueBrain> FlySpray should never be reused silly :P
15:10:06 <TrueBrain> the other labels .. *shrug* .. that will work itself out
15:10:11 <LordAro> well, yes :p
15:10:26 <TrueBrain> okay, some more tweaks ..
15:10:33 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/9
15:10:46 <TrueBrain> please also do a grammer check etc :D
15:10:55 <TrueBrain> I am not retroactively going to fix 7000 tickets :P
15:11:13 <LordAro> "Reason for closing"
15:11:59 <TrueBrain> now I need to find a way to waste issue-number, to fill the holes that exist :D
15:12:07 <TrueBrain> (to keep the FS numbers in sequence)
15:12:12 <LordAro> "Category" is a bit redundant if using labels
15:12:31 <TrueBrain> so is the first line of the ticket, in that case
15:12:38 <TrueBrain> reload
15:12:55 <TrueBrain> anything else?
15:13:26 <frosch123> https://github.com/OpenTTD/OpentTD-Testing/labels <- should be use some of the default labels?
15:13:38 <TrueBrain> how?
15:13:38 <frosch123> like "feature request" -> "nhancement"?
15:13:49 <TrueBrain> I can make a mapping for the task types
15:14:00 <Alberth> Bjarni opened FlySpray ticket #3 on 28 November ... ?
15:14:27 <LordAro> probably no way to automatically apply "invalid" to the relevant ones, right?
15:14:37 <LordAro> or duplicate
15:14:46 <frosch123> there are close reasons "invalid" and "duplicate"
15:14:47 <Alberth> add empty line before the first line of text
15:14:52 <TrueBrain> Alberth: good one
15:15:29 <frosch123> also possibly "xxx closed ticket on" and "reason of closing" should be in this order
15:15:53 <Alberth> could move the import message also to the top then
15:16:10 <TrueBrain> would that really be better?
15:16:15 <frosch123> that does not work if people add new comments
15:16:32 <frosch123> not all tasks are readonly archives
15:16:44 <Alberth> I mean <FlySpray ticket #3> a link
15:17:02 <TrueBrain> I quiet like the link at the bottom, just so everyone knows it was imported
15:17:09 <Alberth> ok
15:17:10 <TrueBrain> (well, not the link, more the text that comes with it)
15:17:37 <frosch123> yes, bottom is fine for that
15:17:52 <Alberth> comment 48 messes up the + and -
15:17:57 <TrueBrain> and I think if the numbers are in sync, we dont even have to mention it was original opened in FlySpray :D
15:18:06 <TrueBrain> yeah .. it was already a bit silly in FS :D
15:18:16 <TrueBrain> I can put a code block around all comments, but that looks bad in 90% of the cases
15:18:29 <TrueBrain> I guess the goal should not be to have every comment perfectly correct :D
15:19:19 <Alberth> I agree, doesn't look feasible to solve this in a simple manner
15:20:31 <TrueBrain> I also not going to make PRs out of patches btw :P
15:21:00 <frosch123> you can escape stuff
15:21:05 <frosch123> \- and \+
15:21:13 <Alberth> how do we keep patches available?
15:21:13 <frosch123> \*
15:21:36 <Alberth> link to FS means it can't die :p
15:21:41 <TrueBrain> frosch123: but in some cases people wanted a list :)
15:21:51 <frosch123> :)
15:21:53 <TrueBrain> Alberth: FS will remain available as a read-only archive, I guess
15:22:30 <TrueBrain> does GitHub allow file-upload btw?
15:22:30 <Alberth> bunch of files available from http, sounds good enough
15:22:35 <TrueBrain> or how are we going to do core-dumps?
15:22:51 <TrueBrain> Alberth: yup; I am going to crawl it when it is migrated, and publish that :)
15:23:56 <LordAro> github has very limited file uploads, last i checked
15:24:14 <LordAro> https://help.github.com/articles/file-attachments-on-issues-and-pull-requests/
15:24:35 <TrueBrain> so we might need something for people to upload their crash dumps etc
15:28:46 <TrueBrain> okay, made a lookup that maps the FS stuff to GH labels as much as possible
15:28:50 <TrueBrain> lets see how the latest now looks :D
15:30:34 <Alberth> empty line after the header?
15:32:18 <TrueBrain> still unsure if Category should be a label, but we will see how that goes :)
15:32:39 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/10
15:33:58 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/11 <- lol .. I did something worng I think :D
15:34:01 <Alberth> more likely, our categories are a bit obsolete
15:34:18 <LordAro> finotwx
15:34:44 <TrueBrain> check bottom LordAro, much more fun :)
15:34:58 <LordAro> :p
15:35:04 <TrueBrain> let me fix this :D
15:35:19 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/12
15:35:44 <TrueBrain> example how Out of date maps to "wontfix"
15:36:08 <TrueBrain> lets try something more recent now ..
15:36:26 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/issues/13
15:36:42 <TrueBrain> any other suggestions/comments/changes?
15:37:13 <LordAro> be nice if FS# could be converted to links
15:37:26 <LordAro> not sure if that should be GH links or FS links
15:37:27 <TrueBrain> I think it will
15:37:39 <TrueBrain> once I import them in the right order, I think it will be fine :)
15:38:16 <TrueBrain> hmm, no, FS#something never resolves to the GH
15:38:35 <TrueBrain> I am a bit reluctant to rewrite comment bodies
15:38:41 <TrueBrain> that is a bit of an endless pit to fall into
15:39:13 <LordAro> yeah
15:39:19 <Alberth> FS #123 should work
15:39:44 <Alberth> unless you mean the number itself
15:39:51 <TrueBrain> but most of us write FS#123
15:39:56 <TrueBrain> which won't work
15:40:25 <Alberth> it won't indeed
15:40:37 <TrueBrain> also not from commit messages, which is a bit sad
15:40:48 <TrueBrain> but rewriting those things ... is .. well .. slippery slope :)
15:41:00 <Alberth> ah well, if #123 is the right destination, you have most of the battle done
15:41:16 <TrueBrain> indeed; I think we just have to accept it :)
15:41:27 <TrueBrain> r14433 ... 50% there?
15:42:13 <Alberth> should be :)
15:52:07 <TrueBrain> out of the 6684 FS issues, only 180 are not in scope of the migration
15:52:10 <TrueBrain> that is not bad :)
15:52:20 <TrueBrain> (means I just have to make 180 dummy issues)
15:53:22 <LordAro> why are they not in scope?
15:53:28 <LordAro> oh, website issues and such?
15:53:31 <TrueBrain> yes
15:53:33 <TrueBrain> WT2
15:53:34 <TrueBrain> WT3
15:53:36 <TrueBrain> those
15:53:45 <TrueBrain> even a few private ones
15:53:49 <TrueBrain> (read: spam)
15:54:13 <TrueBrain> okay, I just have to wait for glx to unwatch the project, and then I can give this a spin
15:54:22 <TrueBrain> I wonder if this will hit a protection at GitHub
15:54:30 <TrueBrain> as creating 7000 tickets might be a bit too much
15:54:32 <TrueBrain> but we will see :P
15:55:00 *** Flygon has quit IRC
15:56:06 <TrueBrain> in the meantime, lets see if I can push my partial import
16:01:39 <frosch123> why do only some labels show in the column on the right?
16:01:56 <frosch123> TrueBrain: what is actually the goal of the new svn import?
16:02:07 <frosch123> i consider it rather important to keep the hashes of the existing git repo
16:02:49 <LordAro> why so? anything that's based off the existing one can be easily rebased
16:03:38 <nielsm> it might be a problem for existing forks hosted on github
16:04:04 <LordAro> they have to be rebased whenever trunk gets updated anyway
16:04:26 <frosch123> well, i never tried a rebase with skipping 30k commits :p
16:05:03 <LordAro> skipping 30k commits? rebase doesn't care for anything except the top
16:05:04 <TrueBrain> frosch123: some labels? What do you mean?
16:05:20 <TrueBrain> frosch123: the current gitrepo on github is ONLY trunk; it does not contain any of the release branches
16:05:20 <nielsm> if the destinations look identical when checked out, a rebase should work flawlessly
16:05:28 <nielsm> regardless of history leading up to it
16:05:31 <frosch123> https://github.com/OpenTTD/OpentTD-Testing/issues/12 <- i only see 3 labels on the right, for exmaple "wontfix" is missing
16:05:44 <TrueBrain> frosch123: yes; I fucked that one up :D See 13
16:05:54 <TrueBrain> it used to be 7 labels, w, o, n, t, f, i, x
16:05:55 <TrueBrain> :P
16:05:59 <TrueBrain> but I removed the labels by now
16:06:02 <TrueBrain> you are just too slow :D
16:06:09 <frosch123> ok :)
16:07:51 <TrueBrain> uploading the first test now .. hopefully that makes a bit more clear why this reimport is needed
16:08:51 <TrueBrain> I love my new internet connection .. it makes these things very quick :D
16:09:30 <TrueBrain> there we go
16:09:40 <TrueBrain> https://github.com/OpenTTD/OpentTD-Testing/
16:09:49 <TrueBrain> frosch123: on this repo now everything works .. tags, release branches, etc
16:10:01 <TrueBrain> please check if any of you spot anything weird
16:10:20 <TrueBrain> it still has 8 years of data to process, but meh :)
16:11:08 <TrueBrain> I love how the "releases" tab works too :D
16:11:15 <andythenorth> bonus points: map usernames in FS tasks to github usernames :P
16:11:20 <andythenorth> via a mapping of known names :PP
16:11:42 <TrueBrain> let me answer that in very clear terms: no
16:11:44 <TrueBrain> :)
16:11:59 <TrueBrain> as you cannot change it once imported, that is just confusing :)
16:13:11 <frosch123> well, new branches is fine
16:13:13 <Alberth> piece good carrier seems fine for rubber transport :)
16:13:17 <frosch123> but why make master different?
16:13:28 <frosch123> i see no reason why to use new hashes for master
16:13:32 <TrueBrain> frosch123: because there is no way to add branches after-the-fact
16:13:32 <andythenorth> Alberth: doesn't show much cargo ;)
16:13:44 <andythenorth> I might put visible cargo on top of the piece goods ships
16:13:49 <TrueBrain> frosch123: feel free to try an import yourself where you get both tags and branches, while keeping the hashes of master
16:13:55 <TrueBrain> I do not see a viable solution
16:13:58 <Alberth> some vehicles and such :)
16:14:02 <TrueBrain> and I strongly doubt a rebase is going to give any issues :)
16:14:09 <TrueBrain> (as the content is identical)
16:14:22 <frosch123> why is it not possible? it would be like taking current git, and then rebasing the new branches on whatever revision they were forked from
16:14:28 <Alberth> isn't the first hash based on the uuid of the repo?
16:14:34 <TrueBrain> svn works a bit different and annoying
16:14:40 <TrueBrain> tools completely fail on it
16:14:45 <TrueBrain> mostly because we abused svn over the years :D
16:14:49 <LordAro> https://i.imgur.com/Vge5t7B.png heh
16:15:13 <LordAro> C++ merge?
16:15:14 <frosch123> noai merge?
16:15:26 <frosch123> oh, indeed c++ switch
16:15:30 <frosch123> noai was a year later
16:15:49 <TrueBrain> frosch123: basically, the current git on GitHub was never meant for this ..
16:16:05 <frosch123> i visually see no difference between the two masters
16:16:08 <frosch123> except the hashes
16:16:12 <Alberth> removal of the old windows system?
16:16:38 <TrueBrain> frosch123: when the import is done, we can try rebasing one to the other
16:16:46 <TrueBrain> git is smart enough, that should be of little issue
16:17:36 <TrueBrain> but I am really happy GitHub picks up this git repo correctly now
16:17:50 <TrueBrain> I am sure someone will be bored sooner or later, and add all the binaries on GH too
16:17:52 <LordAro> https://github.com/OpenTTD/OpenTTD-Testing/commit/f35ed4bbc2b05f1b83476b60948d64375f77f1b4 hmm, not C++ branch
16:17:53 <TrueBrain> for older releases :P
16:19:16 <LordAro> TrueBrain: i notice lack of github links for KUDr, miham, Darkvater, celestar, orudge
16:19:31 <TrueBrain> LordAro: and what do you want ME to do about that?
16:19:38 <LordAro> ¯\_(ツ)_/¯
16:19:58 <TrueBrain> (GitHub links accounts based on email address ;))
16:21:25 <LordAro> oh yeah, you just used foo@openttd.org as email, didn't you?
16:21:33 <TrueBrain> of course
16:21:45 <frosch123> LordAro: same as before
16:21:45 <LordAro> ok, carry on
16:22:10 <TrueBrain> it is up to every developer to claim it at GH :) Not going to do that for them ;)
16:24:27 <TrueBrain> btw, frosch123, most likely the best thing to do, is rename the current repo to something else, and create a new one with everything in it
16:24:33 <TrueBrain> that way we break no forks etc
16:26:57 <TrueBrain> btw, just to be clear: DO NOT USE OpenTTD-Testing! It will be removed :D
16:26:59 <frosch123> i still have no idea why the hashes are even different
16:27:18 <TrueBrain> no clue
16:27:31 <TrueBrain> data is now being converted differently
16:27:34 <TrueBrain> might be related?
16:27:40 <TrueBrain> (it now follows parents)
16:27:41 <nielsm> because the hashes also include ancestor information, likely
16:27:52 <nielsm> and if you're hooking up ancestors correctly then yes they will be different
16:28:37 <TrueBrain> frosch123: I think merges from branches back into trunk are different
16:28:39 <frosch123> oh, it's from the branch merges
16:28:42 <TrueBrain> :D
16:28:52 <frosch123> ok, that makes sense at least :)
16:29:24 <TrueBrain> the history should be more correct with this new import, in relation to branches
16:29:36 <TrueBrain> the old one I think acted like the commits were on the trunk
16:29:39 <TrueBrain> where they were on a branch
16:36:43 <nielsm> by the way I read somewhere on the wiki that svn had been kept as the primary repos because it provides the simple monotonous revision number, which is useful for various versioning things
16:36:51 <nielsm> how are those issues going to be handled now?
16:37:09 <frosch123> they are dropped because of no longer relevant
16:38:11 <frosch123> i think i'll change the version detection script to include a date into the version information
16:38:11 <TrueBrain> we made up tons of reasons why we wanted a single number going upwards for every new version
16:38:14 <TrueBrain> but that is so 2008
16:38:54 <TrueBrain> I cannot think of a single thing that needs a linear number these days ..
16:38:59 <TrueBrain> network just wants the same value
16:39:03 <TrueBrain> savegames have their own revision
16:39:07 <TrueBrain> newgrf?
16:39:07 <frosch123> nielsm: for example bananas content and newgrf can say they require a certain version, like 15.x, but they can also say they need unreleased version > r12345
16:39:17 <frosch123> but noone really used the latter correctly
16:39:25 <TrueBrain> for good reason :P
16:39:47 <TrueBrain> 20180317-g<hash> sounds like a good value tbh :P
16:39:49 <nielsm> you'd really need a list of "features" a newgrf requires instead
16:40:04 <nielsm> (possibly versioned features)
16:40:06 <frosch123> yes, something like that
16:40:16 <frosch123> jgr has just that for savegames
16:40:28 <frosch123> LordAro: btw. are you involved in jgr's c++11 branch?
16:40:45 <LordAro> nop
16:41:41 <nielsm> okay snow-filled clouds seem to have left the sky for a moment, I should go outside and get a bit of sunlight
16:45:27 <Alberth> /me likes a number incrementing on each commit :p
16:46:35 <frosch123> suggest a like button to github?
16:46:47 <TrueBrain> Alberth: git describe does that fine for you :D
16:49:51 <nielsm> actually _never_mind_, too windy outside to enjoy a walk
16:50:11 <andythenorth> can we move provide git on devzone next? o_O
16:50:24 <andythenorth> - move
16:53:41 <frosch123> make a docker compile farm for it :p
17:04:02 <frosch123> would eints users want to put in their email to be linked from github?
17:07:46 *** supermop has quit IRC
17:14:29 *** iSoSyS has quit IRC
17:16:32 *** sla_ro|master has joined #openttd
17:22:34 <Alberth> no way to verify such an address
17:29:43 *** Wacko1976 has joined #openttd
17:30:57 * andythenorth cleans house
17:39:59 *** Wormnest_ has joined #openttd
17:45:37 *** Wormnest__ has joined #openttd
17:46:21 *** Wormnest has quit IRC
17:52:26 *** Wormnest_ has quit IRC
17:56:06 *** Wormnest_ has joined #openttd
18:02:56 *** Wormnest__ has quit IRC
18:03:15 <andythenorth> hmm
18:03:21 <andythenorth> I broke Hog on bundles :| https://jenkins.openttdcoop.org/job/road-hog/1435/console
18:03:46 <andythenorth> is rhodecode stuck again or something?
18:05:14 <frosch123> i just restarted it 2 minutes ago
18:05:46 <andythenorth> ok builds have triggered again
18:05:50 <andythenorth> see if they complete
18:06:25 <frosch123> sam is happy
18:06:49 <andythenorth> thanks
18:07:23 <andythenorth> so what will new OpenTTD compile farm look like LordAro ?
18:07:41 <andythenorth> and will it compile python jobs?
18:07:43 <andythenorth> :P
18:08:44 *** Cubey has joined #openttd
18:17:05 <LordAro> andythenorth: idk, ask TB
18:17:14 <LordAro> magical, i hope
18:17:38 *** Mahjong2 has joined #openttd
18:18:14 <LordAro> given it's just a load of docker instances, i don't see why not
18:19:30 <andythenorth> I think coop devzone is probably time to declare end-of-life
18:19:43 <andythenorth> it's not interesting for anyone to maintain
18:22:04 *** keoz has quit IRC
18:25:06 *** Mahjong1 has quit IRC
18:26:33 <andythenorth> hmm
18:28:42 <andythenorth> so my newgrfs support randomising sprites per vehicle
18:28:52 <andythenorth> it's only actually used for:
18:29:10 <andythenorth> 1) swapping 1CC and 2CC randomly
18:29:18 <andythenorth> 2) reversing tank engines randomly
18:29:29 *** Gja has quit IRC
18:29:53 <andythenorth> it was used to make, e.g. quite different looking graphics for same ship model
18:30:01 <andythenorth> e.g. house at back, house in middle etc
18:30:11 <andythenorth> was BAD FEATURE I think
18:30:18 <andythenorth> might delete
18:33:59 <frosch123> randomising is for when you have more graphics than models
18:34:06 <frosch123> which is likely best for train wagons
18:34:16 <frosch123> but less useful for engines
18:34:23 <frosch123> ships may count as engines
18:35:20 <andythenorth> it even had date-specific code :P
18:35:34 <andythenorth> so could randomise 6 different appearances for the lifetime of a vehicle
18:35:47 <andythenorth> I keep some random, just don't need all this complication
18:38:13 *** Biolunar has quit IRC
18:40:48 <Alberth> some sets do change appearance which can be nice
18:40:57 <Alberth> opengfx+ does, iirc
18:41:40 <Alberth> but likely should be used sparingly :)
18:47:24 <andythenorth> livery can randomise
18:47:30 <andythenorth> visible cargo can randomise
18:47:37 <andythenorth> and asymmetric engines can be reversed
18:47:42 <andythenorth> rest...nah :)
18:59:59 *** Gja has joined #openttd
19:00:28 <TrueBrain> r22500! ALMOST! (well, not really, but well)
19:05:33 <nielsm> looking forward to trying out a rebase
19:20:20 *** glx has joined #openttd
19:20:20 *** ChanServ sets mode: +v glx
19:20:50 <TrueBrain> hi glx; sorry for the spam :(
19:21:50 <andythenorth> oops, I annoyed pyflakes
19:23:07 *** roidal has joined #openttd
19:23:46 <glx> TrueBrain: it's ok, my github is correctly configured ;)
19:24:22 <glx> so I just received the repo creation email
19:24:27 <TrueBrain> I am about to create 7000 issues :) but I was waiting for you, to see if you either want to unwatch OpenTTD-Testing, or disable notifications :D
19:24:31 <TrueBrain> good
19:25:53 <andythenorth> I have unused imports, that I think are valid
19:26:00 <andythenorth> but pyflakes doesn't :(
19:26:09 <andythenorth> now I have to rewrite my codebase to please the validator :(
19:32:43 *** Lejving has joined #openttd
19:34:09 <TrueBrain> okay ... lets import a lot of issues then :D Curious how that goes ...
19:36:08 *** JetFly has joined #openttd
19:36:18 <JetFly> hello
19:36:21 <andythenorth> GL TrueBrain
19:36:51 <LordAro> seems placeholder issues don't get closed :p
19:37:53 <andythenorth> Alberth: which of these is least stupid? (both are stupid)
19:37:55 <andythenorth> 1. http://dev.openttdcoop.org/projects/road-hog/repository/revisions/8a76ad0817be/entry/src/global_constants.py#L33
19:38:01 <andythenorth> 2. http://dev.openttdcoop.org/projects/road-hog/repository/revisions/9c1285699694/entry/src/global_constants.py#L33
19:38:36 <Alberth> let's see
19:38:38 <LordAro> i'd probably say the latter
19:38:53 <LordAro> implicitly adding stuff to scope gets scary quickly
19:39:04 <andythenorth> it scales badly
19:39:16 <andythenorth> I know why pyflakes is tripping on it
19:39:54 <Alberth> yeah, explicit is better than implicit
19:40:19 <Alberth> why don't you collect things here?
19:40:40 <TrueBrain> haha, I hit the rate limiter :D
19:40:57 <frosch123> what rate does it limit to?
19:42:20 <Alberth> or why is dropping the import bad?
19:42:39 <andythenorth> it's used as a convenience
19:42:47 <andythenorth> Polar Fox is a shared library across 3 newgrfs
19:42:49 <TrueBrain> and nothing explains to me what the rate limit is :(
19:43:07 <TrueBrain> it just says that I hit the "abuse rate limits", but I did stay below the "rate limit"
19:43:09 <TrueBrain> wtffff
19:43:09 *** iSoSyS has joined #openttd
19:43:12 <andythenorth> Polar Fox unwisely but conveniently extends 'global_constants'
19:43:23 <andythenorth> so there's no need to change the existing grf codebases
19:43:30 <LordAro> https://developer.github.com/v3/rate_limit/ ?
19:43:59 <TrueBrain> yes, I did not hit those limits
19:44:03 <TrueBrain> (5000 per hour, I mean ..)
19:44:09 <TrueBrain> but there is another "abuse rate limit"
19:45:16 <TrueBrain> lets add a sleep(1)
19:45:31 <Alberth> polar_fox.py does no such thing, it seems
19:45:55 <Alberth> also, did you ever consider writing that data as a yaml file?
19:45:58 <andythenorth> I did
19:46:04 <andythenorth> but I went down this road before
19:46:08 <andythenorth> with .cfg files
19:46:12 <andythenorth> and I considered json
19:46:17 <andythenorth> but pure python is just easier
19:46:38 <TrueBrain> lol @ issue 20 .. WHY YOU SO BOLD :D
19:46:51 <Alberth> more open to random hacking :p
19:47:02 <LordAro> TrueBrain: RRAHHH
19:47:59 <Alberth> but yeah, I've seen other pick python for that reason too
19:49:04 <TrueBrain> even with sleeps, I hit the abuse rate limit .. how can I do 5000 hits per hour, if you dont allow me 1 hit per second :(
19:49:10 <Alberth> another option is to ignore fyflakes
19:50:13 <Alberth> or add magic to say to pyflakes "I know unused imports are bad" :p
19:50:24 <andythenorth> those are also options
19:50:30 <andythenorth> I could pick a better validator :P
19:50:34 <Alberth> if such magic exists
19:50:36 <TrueBrain> frosch123: I created some more closed tickets; can you give it a looksy if that is what you expect?
19:50:39 <andythenorth> I can't make flake8 work for some reason
19:50:42 <TrueBrain> (and others too, if you like :D)
19:51:25 <TrueBrain> aligning the numbers seem to work fine :)
19:51:56 <frosch123> did you already set fs as readonly?
19:52:04 <TrueBrain> no
19:52:06 <TrueBrain> again, this is only testing
19:52:08 <TrueBrain> just dry-runs
19:52:12 <frosch123> ah, ok :)
19:52:24 <TrueBrain> going to wipe everything and start over once we agree this is all as we expect
19:53:36 <TrueBrain> still hitting the abuse rate limit .. seriously .. lol
19:54:50 <TrueBrain> please create this content "at a reasonable pace" ..
19:54:51 <TrueBrain> I WAS!
19:54:53 <TrueBrain> ffs :P
19:57:46 <TrueBrain> now takes ~10 seconds per ticket ... we have 7k ..
19:57:50 <TrueBrain> lol
19:58:25 *** Gja has quit IRC
20:04:01 <TrueBrain> and still hitting rate limits .. this is silly :P
20:10:19 *** Thedarkb has quit IRC
20:14:19 <TrueBrain> and the documentation is a lie .. ugh .. I understand why GitHub has these protections, but bah, it is annoying
20:16:14 <TrueBrain> nothing in the reply giving any hints when I can retry ... owh boy ..
20:17:48 *** supermop has joined #openttd
20:27:49 <TrueBrain> I send GitHub an email asking them what they want me to do :)
20:28:45 <TrueBrain> so we will have to wait for a reponse there
20:28:52 <TrueBrain> otherwise, from what I can tell, the import is looking very good
20:28:58 <TrueBrain> happy with the level of details we can import
20:30:20 <frosch123> is dorpsgek just a random to the project? or is it assigned to openttd somehow?
20:30:32 <TrueBrain> it is info@openttd.org
20:30:45 <TrueBrain> (and owner of OpenTTD organization)
20:30:51 <frosch123> yes, but is there a setting in github to give users permissions
20:30:58 <frosch123> ok, "owner" sounds good
20:31:04 <TrueBrain> I assumed as much
20:31:11 <TrueBrain> it has no explicit rights into OpenTTD-Testing
20:31:23 <TrueBrain> don't expect it matters much tbh
20:31:30 <TrueBrain> I think this abuse rate limit is to prevent spam
20:32:22 <TrueBrain> yeah, GitHub sees DorpsGek as "owner"
20:32:28 <TrueBrain> (check any issue; it says right there :D)
20:33:55 <TrueBrain> good question btw; happy it is how I expected it to be :D
20:34:03 <frosch123> hmm, is the close reason missing?
20:34:12 <TrueBrain> if you see TrueBrain closing an issue
20:34:14 <TrueBrain> there is
20:34:19 <TrueBrain> as then the script halted
20:34:25 <frosch123> i see stuff like "fixed", but usually there is an add-on "in r12345"
20:34:40 <TrueBrain> if there is a close reason, it is added
20:34:47 <TrueBrain> so there is a resolution and a close reason
20:34:57 <TrueBrain> resolution is *almost* always there, close reason .. sometimes
20:35:03 <frosch123> oh, i see
20:35:10 <frosch123> it's added like a comment
20:35:11 <TrueBrain> depending on who did it, I guess
20:35:14 <TrueBrain> yes
20:35:25 <TrueBrain> is there a better way?
20:36:02 <frosch123> it's fine, i was just looking at an unusual case
20:36:13 <TrueBrain> k :)
20:36:58 <frosch123> 119 is the earlies issue i know properly :)
20:37:39 <frosch123> (like still knowing the number after 10 years)
20:38:13 *** Gja has joined #openttd
20:38:14 <LordAro> even i know what issue that is
20:38:36 <TrueBrain> you are getting old dude :P
20:38:42 <LordAro> :<
20:44:49 *** SeraphAngel has quit IRC
20:49:56 *** Thedarkb has joined #openttd
20:50:52 <nielsm> bah, trying to neatify my code history is so troublesome
20:51:04 <nielsm> should have made even more, smaller commits
20:51:16 <LordAro> :)
21:03:56 *** Thedarkb has quit IRC
21:19:17 *** gelignite has joined #openttd
21:20:13 *** Supercheese has joined #openttd
21:22:57 *** FLHerne has joined #openttd
21:36:32 *** SeraphAngel has joined #openttd
21:36:54 *** Supercheese has quit IRC
21:48:27 <Alberth> nielsm: my strategy is to have a fresh repo and your current work in two directories, and copy changes over from the current to the fresh
21:48:46 <Alberth> some diff/merge util usually works quite good
21:49:08 <Alberth> I use gvimdiff, but ymmv with that :)
21:50:42 <andythenorth> my strategy is to not worry about history :P
21:50:52 <andythenorth> my commits reveal the true carnage :P
21:50:54 <nielsm> yes that's also what I'm about to end up doing, manually playing back changes in a more sensible order
21:50:57 <LordAro> Alberth: ah, but some people can use git :p
21:51:48 <Alberth> some people are just too addicted :p
22:00:11 *** Alberth has left #openttd
22:21:24 *** sla_ro|master has quit IRC
22:27:42 <supermop> andythenorth: have you drawn cars as vehicle sprites?
22:28:03 <supermop> hoping i can skip drawing them
22:28:40 <andythenorth> nope
22:28:47 <andythenorth> snail has
22:28:50 <andythenorth> and some other set
22:29:03 <andythenorth> maybe one of the truck sets, Hungarian
22:29:17 <TrueBrain> oeh, git to svn is done .. let's publish this stuff :D
22:29:23 <LordAro> :o
22:30:00 <TrueBrain> 222k objects; lol
22:30:11 *** synchris has quit IRC
22:30:20 <LordAro> even with a fsck or something?
22:30:32 <TrueBrain> 91 MiB
22:30:34 <TrueBrain> not bad
22:30:40 <TrueBrain> 18k delta files
22:31:52 <supermop> can probably kitbash by trucks down to someting pick-up sized
22:31:57 <LordAro> https://blog.github.com/2018-03-05-measuring-the-many-sizes-of-a-git-repository/
22:31:58 <supermop> *my
22:35:17 <TrueBrain> pushing now ... curious :)
22:36:38 <TrueBrain> it is done
22:36:46 <glx> that was fast
22:36:56 <debdog> https://www.youtube.com/watch?v=3XqyGoE2Q4Y Steppenwolf - The Pusher
22:37:22 <glx> 14K commits
22:37:37 <TrueBrain> my connection is not the bottleneck in this process :P
22:38:35 <TrueBrain> okay, everything seems to be how it should, as far as I can tell
22:38:40 <TrueBrain> I would like to ask you guys kindly to validate :)
22:38:43 <peter1138> what devious plan is truebrain up to?
22:39:00 <TrueBrain> https://github.com/OpenTTD/OpenTTD-Testing <- AGAIN, ONLY FOR TESTING; REPO WILL BE REMOVED!!!
22:39:39 * andythenorth clones
22:40:45 <andythenorth> classic https://github.com/OpenTTD/OpenTTD-Testing/issues/82
22:41:23 <peter1138> 2006 ;(
22:42:23 <TrueBrain> minus the Abuse Rate Limiter, I think this should be it-ish
22:42:44 <TrueBrain> frosch123: I will give it a week or so to give people the time to test and validate
22:42:52 <TrueBrain> I will run some tests myself too over the week
22:43:02 <TrueBrain> and I hope github gets back to me soon with a proper solution :D
22:43:33 <TrueBrain> for now I am running the import even slower
22:44:00 <TrueBrain> so please let me know any issues/problems/weird thing, or something you just would like to see different
22:44:28 <frosch123> thanks, will look :)
22:44:31 <andythenorth> :)
22:46:15 <TrueBrain> lol, SourceForge bug-numbers are coming by too :D
22:46:20 <TrueBrain> I forgot we once had a bug tracker there :P
22:47:10 <frosch123> at least we were not on geocities
22:47:12 <frosch123> were we? :p
22:47:38 <supermop> https://imgur.com/a/IMXXS
22:47:53 <supermop> think that will fit on a truck
22:48:19 <supermop> still kind of tall
22:49:02 <TrueBrain> hmm .. maybe I should just map WIP to patch
22:49:07 <TrueBrain> as most seem to be just that .. patches
22:53:25 <andythenorth> supermop: truck on a truck
22:53:32 <frosch123> i think we discussed that last year
22:53:42 <frosch123> and made andy turn all patches into wips
22:54:00 <supermop> andy now to draw a little subaru
22:54:18 <TrueBrain> your #119 is imported btw
22:54:32 <TrueBrain> frosch123: so if I map them all to label 'patch', that is fine by you?
22:54:36 <TrueBrain> or you have a better label suggestion?
22:54:40 <TrueBrain> they should of course now become PRs
22:54:42 <frosch123> yes, "wip" is a stupid label :)
22:54:43 <TrueBrain> but .. that is tricky :)
22:55:14 <frosch123> in the early issues i also saw a lot of "wontfix" which were rather "invalid" ones
22:55:15 <glx> indeed patches should be PR
22:55:27 <TrueBrain> https://github.com/OpenTTD/OpenTTD-Testing/issues/137 <- no spelling errors or anything? :D
22:55:34 <glx> but hard to do in the migration I guess
22:55:42 <TrueBrain> glx: near impossible :)
22:55:52 <TrueBrain> frosch123: you have examples? I can update the mapping :)
22:55:56 <supermop> the spoiler is hard
22:55:57 <frosch123> TrueBrain: also, there are some spam/scam tasks on fs which were maked private
22:56:09 <TrueBrain> frosch123: yup; I will skip those
22:56:17 <TrueBrain> already in there
22:57:37 <frosch123> huh what?... i did not know michi was the author of 119
22:57:55 <frosch123> the world is so small
22:58:32 <TrueBrain> and I hate the abuse filter again :)
22:59:17 <TrueBrain> let me see .. I map the following to wontfix: Won't fix, Deferred and Out of date
22:59:19 <supermop> https://imgur.com/a/PdWM2 these fit on carries easier
22:59:32 <supermop> can't fit 555 on side
22:59:34 <TrueBrain> invalid: Not a bug, Won't implement, Invalid, and Unreproducible
22:59:36 <frosch123> anyway, "out of date" and "deferred" seem to be mapped to "wontfix"
22:59:40 <frosch123> i would just give them no label
22:59:49 *** Gja has quit IRC
23:00:18 <TrueBrain> done; no longer will get a label
23:00:38 <supermop> i think the cars are too noisy
23:01:14 <TrueBrain> there is also the label 'external', basically all bugs closed because of unsupported custom version, custom AIs, NewGRFs, ..
23:01:37 <TrueBrain> okay .. at this rate I have the issues imported in 7 days
23:01:42 <TrueBrain> so I hope GitHub gets back to me :D
23:03:01 <frosch123> https://github.com/orgs/OpenTTD/people <- dorpsgek is not listed there
23:03:31 <TrueBrain> he is for me :)
23:03:35 <TrueBrain> (marked as Private)
23:03:47 <frosch123> ok, i did not bother logging in
23:04:02 <glx> I see it too ;)
23:04:19 <TrueBrain> frosch123: logging in wont help I think :P
23:04:28 <TrueBrain> glx, me and DorpsGek are owner of OpenTTD organization
23:04:33 <TrueBrain> I just put myself on Public
23:05:08 <TrueBrain> the whole Owners group is secret :D
23:06:26 <frosch123> yeah, they are busy eating dates
23:08:02 <TrueBrain> lets add frosch123 as owner too :P
23:08:17 <TrueBrain> sounds like the right thing to do :D
23:08:31 *** Wacko1976 has quit IRC
23:08:43 <TrueBrain> there is a team Developers, with write access to OpenTTD-Testing
23:08:50 <frosch123> i am still impressed that i got a green avatar
23:08:56 <TrueBrain> not sure that is needed .. I guess all devs can be owner of OpenTTD itself
23:09:14 <TrueBrain> then again .. they might do unexpected stuff ..
23:10:46 <LordAro> :o
23:11:17 <frosch123> yeah, imagine someone would give access to lordaro
23:11:17 <TrueBrain> owh well, enough fiddling around for today :) Pretty happy with the result so far :)
23:11:35 <LordAro> :o
23:11:41 <frosch123> suddenly everything would be c++20
23:11:48 <TrueBrain> no, we told everyone what would happen if they tried ...
23:11:55 <LordAro> haha
23:12:04 <LordAro> i don't even know what's going into C++20
23:12:13 <TrueBrain> well, a few have tried ... I mean ..
23:12:17 <TrueBrain> @seen Bjarni
23:12:17 <DorpsGek> TrueBrain: Bjarni was last seen in #openttd 6 years, 23 weeks, 2 days, 21 hours, 53 minutes, and 10 seconds ago: <Bjarni> heh
23:12:21 <TrueBrain> nuff said :P
23:15:10 <frosch123> oh, my contributions doubled today
23:16:02 *** roidal has quit IRC
23:16:07 <frosch123> nrt is out of sync, then it would count it 3 times
23:20:55 <andythenorth> :P
23:21:06 <andythenorth> NRT won't merge cleanly currently
23:21:14 <andythenorth> I need to fix openttd.grf
23:21:23 <andythenorth> and saveload is also conflicting
23:22:18 <andythenorth> also very sleep
23:22:20 <andythenorth> bye
23:22:24 *** andythenorth has left #openttd
23:23:04 <TrueBrain> frosch123: changing to this new repo goes like this, at least, I found it easiest:
23:23:13 <TrueBrain> git remote add neworigin https:/...
23:23:19 <TrueBrain> git fetch neworigin
23:23:28 <TrueBrain> git rebase --onto neworigin/master origin/master <branchname>
23:23:52 <TrueBrain> this basically picks up the diff between origin/master and <branchname> and replays it on neworigin/master
23:24:06 <TrueBrain> without --onto, it will try to find the common parent .. which will take a while :D
23:29:32 <frosch123> ah, --onto :)
23:30:21 <TrueBrain> Hidden gems :)
23:30:37 <TrueBrain> hmm .. it is suggested that if you make a private repo, the abuse rate limits don't apply as they do currently
23:30:40 <TrueBrain> something to test tomorrow
23:33:10 <TrueBrain> it seems I either have to make the import script resumable, or we have to close both bug-trackers for a day or so .. hmmm
23:33:32 <TrueBrain> should not be too difficult to add resume
23:37:11 <TrueBrain> something to toy with tomorrow! nn!
23:37:46 *** iSoSyS has quit IRC
23:55:45 *** Wacko1976 has joined #openttd