Guild icon
DDraceNetwork
Development / developer
Development discussion. Logged to https://ddnet.tw/irclogs/ Connected with DDNet's IRC channel, Matrix room and GitHub repositories β€” IRC: #ddnet on Quakenet | Matrix: #ddnet-developer:matrix.org GitHub: https://github.com/ddnet
Between 2019-09-30 00:00:00Z and 2019-10-01 00:00:00Z
Avatar
in vanilla this shit is only tolerated because magnet does it
Avatar
i would love to see this discussion in vanilla
Avatar
I had it
Avatar
heinrich pr to allow rapid fire hammer/pistol to cope with scrollwheel spam
00:45
did u actually?
Avatar
no
00:46
I had the discussion of banning +fire on scrollwheel
00:46
a couple of years back
Avatar
why was it turned down
Avatar
"pros use it"
00:46
I found that a non-convincing argument
Avatar
why not the other way then
Avatar
I did not propose that back then
Avatar
it’s the same argument
Avatar
yes, but I'd like it the other way for vanilla
00:48
ddnet is known for quirky, "fun" gameplay
00:48
vanilla is not so quirky
Avatar
can u do something against it server side
Avatar
we can try
00:50
but I don't think it'll be easy
Avatar
Theres way too much you want...
01:40
It is not about what you want, maybe you should ask the community more often @heinrich5991
Avatar
which is what I did there?
Avatar
You don't quite seem to understand what it means to block this server-side
06:18
the heuristic we decide to use could end up eating your input, is that preferable to losing some bragging rights?
06:22
Also you don't always know that someone is spamming the hammer at the time, what if we know after the hammer hits? Do we ban them for cheating?
06:24
Anyway if we don't merge this we might aswell disallow the hammer spam. I'd love to see how you feel about it when konsti starts cheating all the deepfreeze parts smashing records
Avatar
@fokkonaut you force-pushed to ddnet7, apparently also killing teehistorian
Avatar
DDraceNetwork server modification for Teeworlds 0.7, ported by fokkonaut - ddnet/ddnet7
07:44
DDraceNetwork server modification for Teeworlds 0.7, ported by fokkonaut - ddnet/ddnet7
07:44
no teehistorian.cpp
Avatar
It's installed basically everywhere and just increases release size. They're serious about backward compatibility and haven't had a breaking change in ages.
07:54
It's installed basically everywhere and just increases release size. They're serious about backward compatibility and haven't had a breaking change in ages.
08:01
@heinrich5991 does removing the commit fix it?
Avatar
removing what commit?
08:02
I don't know what you did
08:02
you probably did git reset followed by a forced git push
08:02
as a rule of thumb, never do forced pushes to public branches like master
Avatar
what to do now? can you fix it? :/
Avatar
what did you do after merging my PR?
08:20
you could have a look at your local git reflog (edited)
08:20
you should be able to see the last state after you reset it
08:21
what did you do, anyway? git reset? ^^
08:23
just edited without a pull i guess
Avatar
no, something worse happened there
08:26
it might also be caused by someone else with write access to the repo, but I doubt anyone else did something(?)
Avatar
Bojidar Marinov 2019-09-30 10:26:44Z
@heinrich5991 if you have the (a?) previous state on your machine, you could restore it as well πŸ™‚
Avatar
I have a previous version of it
10:27
I don't know if it's the newest one
Avatar
Bojidar Marinov 2019-09-30 10:28:45Z
also, according to GitHub, https://github.com/ddnet/ddnet7/commit/95a4b6579433ca3205c31aab79ba009985455cb6 was pushed by its author, as were the previous and next commits (AFAIK) (edited)
10:29
are you sure that teehistorian.cpp was actually commited?
Avatar
This uses version 2 of the teehistorian file format which has support for extra chunks, i.e. rarely used chunks that need not be standardized. teehistorian records all network messages and inputs f...
10:30
how do you know what commit was pushed by whom?
Avatar
Bojidar Marinov 2019-09-30 10:31:05Z
ah, that PR was somehow nuked, indeed
10:31
I think GH would sometimes show "xx committed and yy pushed"
10:31
not sure if it does it on branches or only on PRs though
Avatar
no, that's just "authored and committed"
10:31
not pushed
10:32
authored = written the code, commited = cherry-picked or rebased the commits, pushes aren't seen in git
10:32
(they might be seen somewhere on github, I don't know)
Avatar
Bojidar Marinov 2019-09-30 10:32:25Z
your commit is in the staging branch though
Avatar
[quakenet] Ryozuki BOT 2019-09-30 11:01:50Z
Konsti what
Avatar
[quakenet] Konsti BOT 2019-09-30 12:13:37Z
Ryozoozki
Avatar
@heinrich5991 idk why but your commits arent there anymore
Avatar
yes
12:33
try using git reflog on your local repo to find the last version before the break
Avatar
@heinrich5991 lets run a test server with your patch
Avatar
sure
Avatar
[quakenet] Ryozuki BOT 2019-09-30 17:24:27Z
Konsti: hi
17:24
Konsti: why u slap me
Avatar
[quakenet] Konsti BOT 2019-09-30 17:43:38Z
isnt it ur fetish ryozoozki
Avatar
Is/was there a reason why we have this awkward combination of record_race and record_teamrace in our database with all the redundancy but still no way to clearly match stuff from record_race with a teamrace?
Avatar
no one did it properly I guess?
19:38
there's still lots of race conditions in there as well πŸ˜›
Avatar
I got a little time the last days and in light of ddnet7 did some experiments to implement an rpc-service that is used to communicate with a database. Things are mostly working already but now i am creating a python script to do the actual sql-queries and it is not so nice. To clarify how it works right now: DDNet-Server calls my pythonscript or whatever implements the rpc-service which is then responsible to store and retrieve ranks etc.
19:50
About the db-scheme: I was thinking of a table for finishes storing map, time, gameuuid etc. but not the playernames or checkpoints with an uuid and another table linked via this uuid with the players and their checkpoints.
19:51
Maybe checkpoints should even be another table, so we keep only the best.
Exported 80 message(s)