DDraceNetwork - developer

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 2020-12-22 00:00:00Z and 2020-12-23 00:00:00Z

if I output a list of (texture, texture coords, screen coords), then that could theoretically be rendered by a graphics card, right?

09:59

@Learath2 ^ (thinking about letting render_map use the graphics card)

@heinrich5991 render to a framebuffer

10:01

http://www.opengl-tutorial.org/intermediate-tutorials/tutorial-14-render-to-texture/

Tutorial 14 : Render To Texture

Free tutorials for modern Opengl (3.3 and later) in C/C++

@heinrich5991 could you help adding support for F-Client to ddnet server?

10:02

I dont know how to translate extended msgs in the ddnet code

if seen.contains(&(deck1, deck2)) {
    ...
}

seen is of type HashSet<(VecDeque<i32>,VecDeque<i32>) Why does this move deck1 and deck2? Any way around this? (edited)

any way around this:

10:05

let decks = (deck1, deck2);
if seen.contains(&decks) {
  …
}

:o

this transfers ownership to decks?

yes, you can get it back out

10:05

decks.0, decks.1 or let (deck1, deck2) = decks;

Silly :D

10:06

Thanks, you've been a huge help in my rust journey @heinrich5991

you could manage it in the tuple the whole time, then you won't have to do that juggling

10:06

you're welcome

sadSnail

10:07

@timakro soon u will know more than me

10:07

pepe_straight

@fokkonaut wdym by translate?

10:08

(I'm not well-versed in 0.7 compatibility code btw)

Mh, well, 0.7 for example uses an extended netmsg for team state, so i have to translate the msgid to the 0.7 msgid in RepackMsg()

10:09

I think

10:09

Thats how I do it in my mod at least but I cant find RepackMsg

yes, sounds good

its generated or so?

idk ^^

Mh

I haven't worked with that code yet

10:10

I think Learath2 and timakro worked with it, but I don't know if they want to answer questions about it

I don't think I remember

10:11

wdym by adding support for F-Client to ddnet server? @fokkonaut

@timakro Learath did a lot things different than in your initial build

Yep I read some of his code

Doubt you could help, but anyways; I am trying to send and receive extended net msgs to and from my 0.7 client

10:12

For example, in ddnet sv_teamstate is a normal net msg

10:13

and in 0.7 i made it a netmsgEx, so now I have to translate it between the versions

10:13

That should happen in CServer::RepackMsg

10:13

But I cant find it, and if its generated it will be a problem I think

Ah, I was just confused, sounded like you were working on the ddnet server code

I am

Oh then you want extended net msgs for 0.7

10:15

And the server doesn't even send them to your client?

10:15

Because it's 0.7

I just need to add the if (msgid == 30) msgid = Netmsg_sv_teamstate (but the extended one, to be sent to 0.7 clients)

10:16

msgid 30 is the normal sv_teamstate in ddnet

well does it break for vanilla tho.

i think learath did a fix for vanilla 0.7 clients, not sure

10:18

Ah no

10:18

Someone else did

@fokkonaut I think you need to fix the RepackMsg function in server.cpp to also work correctly with extended net messages

I would use @ddnet7, would that be correct?

wdym, do you want to introduce new messages?

yes

ah, I thought you had an existing one

Doesnt work without it

10:20

Yes, I do, but I am doing a new update for fclient so i will reset it

10:20

bcs i fucked up naming before

ah

10:20

ideally you'd use a domain that you contrl

10:20

if you don't have any, you could maybe use @fokkonaut.github.io

i just thought why not using ddnet7 or so because that would be more general

10:20

And can be used by other clients

yours can also be used by other clients

Mh

but I guess if you add it to the ddnet source code, it's also fine to use 7.ddnet.tw or something

ok

or just ddnet.tw

okay, one last question

10:21

xd

(look into the messages, the first ones I added were @heinrich5991.de ^^) (edited)

10:22

and they're still supported by everyone supporting the protocol

I am adding a cl_explayerflags, because that is sent by my client, it only contains whether my Aimline is on or not right now, but for the future it might get more, so does it need a 7.ddnet.tw or just ddnet.tw?

just use fokkonaut.github.io there I guess

10:23

I don't think ddnet will switch to use playerflags via a different message

mh

for teamstate I can see that ddnet will switch to it, eventually

Isn't that already covered by the ddnet protocol? I think there was a feature to display other players aimlines

[ddnet/ddnet] Pull request opened: #3435 Don't sort on entry adding

Don't sort the whole server browser everytime an entry is added, instead only sort it once per Update call. I hope this will help a bit with wrong Ping display, bcs of the br_max_request thing. However, it won't fix it completly, since new request will still alter its correctness for testing: br_max_request 500 makes the effect more visible, then its highty noticable

Checklist

[ ] Tested the change ingame
[ ] Provided screenshots if it is a visual change
[ ] Tested in ...

@timakro thats not what I need xD my client is 0.7 and in 0.7 the playerflags are clamped, so an unknown playerflag will invalidate the packet

10:25

its not possible to add the aim flag there

Sorry, I'm a bit out of touch with the network code, now I finally got what you want. Thought the whole time you wanted to support extended net msgs with your 0.7 client

10:27

Wait or do you, ok just forget it xD

10:27

I'm confused

xD

10:28

I am working on ddnet server to support fclient a bit more

10:28

fclient already has ex netmsgs etc

10:28

But communicates only with my server basically

nouis

it's more confusing now

[ddnet:master] 2 new commits

fb74bcb Don't sort on entry adding - Jupeyy ca03030 Merge #3435 - bors[bot]

10:55

[ddnet/ddnet] Pull request closed: #3435 Don't sort on entry adding

btw what is stopping vanilla from getting ex messages. did someone veto it already.

no, I guess it could be done

10:56

I can see them vetoing a compatible implementation though

10:57

e.g. because it contains strings like @heinrich5991.de

i see

10:58

is your one acts like a identifier currently?

there are a couple of messages that I expect everyone implementing the protocol to implement

10:59

ah no, they're even @ddnet.tw

sixup currently doesn't have anything extended right?

what-is@ddnet.tw, it-is@ddnet.tw and i-dont-know@ddnet.tw

10:59

correct

10:59

AFIAK

if we do it in vanilla i would imagine all extension would be teeworlds.com since they don't have anything else. and ddnet can maybe some how do the same with sixup, since it currently doesn't have any

that's exactly what I fear

11:01

protocol incompatibility just because there are a couple of strings mentioning ddnet

11:01

which demotivated me to not do a PR against teeworlds

why not rename it to teeworlds then?

because it's an existing network protocol

doesn't sixup need seperate extensions from ddnet's

no, not necessarily

if we update to 0.7 we might as well break other protocols

i see

it'll just create complexity for vanilla's pride's sake

11:03

not good

tee_thinking

[ddnet:master] 5 new commits

567f0aa Notify about vote when authed - def- 6586ab2 Play highlight sound on vote too when authed - def- 4d3d9d7 Only assign aOldName when we need it - def- 57b1311 Fix localization of version/game type/ping in server details - def- 8a15bc1 Merge #3417 #3419 #3432 - bors[bot]

11:03

[ddnet/ddnet] Pull request closed: #3432 Fix localization of version/game type/ping in server details

11:03

[ddnet/ddnet] Pull request closed: #3419 Only assign aOldName when we need it

11:03

[ddnet/ddnet] Pull request closed: #3417 Notify about vote when authed

tee_thinking

but if fok gets sixup extensions now, then vanilla someday decided to do something similar it will just create more complex stuff i think (edited)

11:05

and the way the race protocol works in vanilla is kinda dirty.

how does it work?

they just appended it and expect them to be moved to proper offset in 0.8 i think?

11:06

i don't know if offset is the right term.

yea, I understand

[ddnet/ddnet] Pull request closed: #3397 Ignore unknown commands from map config in client

vanilla also need a way to somewhat freely extend their own protocol within major version (like the player name change thing and spectating cursor thing which oy said could be done in 0.7).

11:08

and i think a map vote issue oy said might be good for 0.7 as well

11:09

ex messages kinda does that. we just need a cleaner identifier (maybe)

for a single entity updating the protocol, simply adding one each time they add a message suffices

11:09

but I guess it'd also be nicer if vnailla had an easy way for mods to add messages

right

11:10

i think i'm still thinking about mods.

11:10

vanilla can just add stuff if no mods are using those "offsets"

ye

@heinrich5991 fixed cmakelist

tough choice

@fokkonaut I still see a cross

its the clang style

11:12

idk about it

can't be merged before it's not fixed

not dilated xD lol

11:12

wtf

you can run clang-format -i $file

[ddnet/ddnet] Issue opened: #3436 Map config format

Maps shouldn't require a config interpreter to get gameplay settings. The config should be evaluated by the editor so it doesn't depend on never changing the console syntax. This would also make changes like https://github.com/ddnet/ddnet/commit/5ec702ad2fd06f2e6816192ce04f485d4cf0c80a forward and backward compatible. That would also fix the issue #3397 tried to fix.

@deen i think i got it now

[ddnet-web:master] 1 new commit

80e3b0e Add Tournament 53 results - def-

11:23

[ddnet:master] 2 new commits

ea70c34 Keep veto right after joining dummy with lower ID (fixes #3404) - def- e708f3a Merge #3405 - bors[bot]

11:23

[ddnet/ddnet] Issue closed: #3404 Veto doesn't work after joining dummy

11:23

[ddnet/ddnet] Pull request closed: #3405 Keep veto right after joining dummy with lower ID (fixes #3404)

it is possible that f-client gets enough 0.7 extensions to the point that ddnet client could just talk 0.7 protocols but still keeping most of the codes.

11:23

ye

brilliant

kek

@TsFreddie mind if I close https://github.com/ddnet/ddnet/pull/3262 ?

Spectator / Demo Cursor by TsFreddie · Pull Request #3262 · ddnet/d...

Display player's cursor when spectating and in demo. Checklist Tested the change ingame Provided screenshots if it is a visual change Tested in combination with possibly related configu...

11:25

doesn't look like it's getting updated soon

sure.

11:27

i'll wait a bit and see how vanilla deal with this.

[ddnet/ddnet] Pull request closed: #3293 Clang 11

11:28

[ddnet/ddnet] Pull request opened: #3437 Update .clang-tidy file to 11

Checklist

[ ] Tested the change ingame
[ ] Provided screenshots if it is a visual change
[ ] Tested in combination with possibly related configuration options
[ ] Written a unit test if it works standalone, system.c especially
[ ] Considered possible null pointers and out of bounds array indexing
[ ] Changed no physics that affect existing maps
[ ] Tested the change with [ASan+UBSan or valgrind's memcheck](https://github.com/ddnet/ddnet/#using-addresssanitizer--u...

11:30

[ddnet/ddnet] Pull request closed: #3262 Spectator / Demo Cursor

11:31

[ddnet:master] 3 new commits

e805e42 Alternative alternative fix for malloc of size 0 - heinrich5991 bd1c133 Work around clang-analyzer-unix.Malloc false positive - heinrich5991 e9b174a Merge #3389 - bors[bot]

11:31

[ddnet/ddnet] Pull request closed: #3389 Alternative alternative fix for malloc of size 0

[ddnet:master] 2 new commits

d4c4031 Add F-DDrace entities - fokkonaut 0b2eda2 Merge #3434 - bors[bot]

11:40

[ddnet/ddnet] Pull request closed: #3434 Add F-DDrace entities

11:42

[ddnet/ddnet-libs] Pull request reopened: #15 Add libpng for Jupeyy (untested)

11:49

[ddnet:master] 4 new commits

2544c75 Add w as default bind for dummy hammer - def- 1ed74af Update .clang-tidy file to 11 - def- 93a0dc3 Add more commonly used binds - def- 440337e Merge #3384 #3437 - bors[bot]

11:49

[ddnet/ddnet] Pull request closed: #3437 Update .clang-tidy file to 11

11:49

[ddnet/ddnet] Pull request closed: #3384 Add w as default bind for dummy hammer

Add w as default bind for dummy hammer - def- oof

i think its good

I jump on w

The message is wrong, it's on h

wait isn't w the default jump

11:58

oh

11:58

ghosty

TsFreddie

wait isn't w the default jump

space is default jump

i think both were default in ddnet at least

anyway, those defaults would never overwrite your existing settings

oh, so this commit is just regular auto-hammerfly

12:03

I thought that's actual built in deepfly

that one too

ah nice! Did you figure out whats causing the random weapon switch while deepflying or is this still unfixed

no idea

i never seen random weapon switch

12:10

i always think it was only on the first shot

Ama

i never seen random weapon switch

example here: https://discordapp.com/channels/252358080522747904/295908390956433410/782951875741483009

12:10

you'd have to manually switch to hammer again to fix it (even if you're already having the hammer out)

seems only on first shot

12:11

probably you had pistol out and after that it just looks like panic

I never panic

12:12

let me upload another example

The original deepfly bind is such a hack I'm not quite sure where to even look to find what went wrong

Learath2

The original deepfly bind is such a hack I'm not quite sure where to even look to find what went wrong

#3258

https://github.com/ddnet/ddnet/issues/3258

Make scroll (weapon switch) non-sticky. · Issue #3258 · ddnet/ddnet

In the clip i quickly scrolled+switched to main tee, This makes the deepfly bind (bind mouse1 "+fire; +toggle cl_dummy_hammer 1 0") unusable. I see no harm in making the scroll wh...

unsure if thats true, idk

@Patiga https://gitlab.com/ddnet-wiki/ddnet-wiki.gitlab.io/-/issues/3#note_472218708

Evaluate software choice and workflow (#3) · Issues · DDNet Wiki / ...

@Ryozuki Currently we make merge requests and let someone else merge it, but I am open to suggestion to...

12:44

Zwelf ^

@heinrich5991 I think you wanted to ping trml instead of me https://github.com/ddnet/ddnet/pull/3368#issuecomment-749493030

Only render visible entities by trml · Pull Request #3368 · ddnet/d...

Some entities like pickups are always sent to the client, and the client will currently try to render them even if they are off screen. I'm not sure if using hardcoded values for the maximu...

@timakro good call, ty

https://www.reddit.com/r/pcgaming/comments/ki3wxg/directx_creator_dies_aged_55_leaving_behind_a/

r/pcgaming - DirectX creator dies aged 55, leaving behind a game-ch...

6,098 votes and 322 comments so far on Reddit

16:58

oh noes, the man who made it impossible to game on linux died

16:58

im so sad

@Ryozuki Don't be an ass, especially about someone having died

directx isnt even the problem

17:07

its more about drm that makes game not run on linux nowadays

is the antiping source on github?

Yes: https://github.com/ddnet/ddnet/tree/master/src/game/client/prediction

DDraceNetwork, a cooperative racing mod of Teeworlds - ddnet/ddnet

is this: 194.67.208.6:8203 a real ddnet server, cuz it's shit af

17:55

since when u can teleport to players

@Deleted User yes, official. Teleport how?

there's a telegun at the end of the race lol

17:59

wth is this shit

haha, intentional addition or just an old tile that got a new meaning?

idk, pepe map is not like that by default, and seeing people teleporting everywhere is kinda annoying, idk how rus can play like that, prob cuz there's not much servers for them

@deen ty

justatest

@Pepe fck, u there

18:02

*run*

18:02

well I hope ddnet will remove that, it's not pepe anymore

@Deleted User yeah, tele gun might be a bad idea, @Skrill also noticed that you can do weird stuff with it

murpi

ah nice! Did you figure out whats causing the random weapon switch while deepflying or is this still unfixed

its because of cl_dummy_restore_weapon

[ddnet/ddnet] Pull request opened: #3438 Replace unused chains in chat

Probs we forgot it when we swapped from two configs to one

Checklist

[ ] Tested the change ingame
[ ] Provided screenshots if it is a visual change
[ ] Tested in combination with possibly related configuration options
[ ] Written a unit test if it works standalone, system.c especially
[ ] Considered possible null pointers and out of bounds array indexing
[ ] Changed no physics that affect existing maps
[ ] Tested the change with [ASan+UBSan or valgrind's memcheck]...

This confirms heinrich5991 always saying we have too many options

the biggest issue is that most stuff is coded to work, not designed to work xd

[ddnet:master] 2 new commits

f21687a remove unused chains - Jupeyy e6cf5bc Merge #3438 - bors[bot]

18:51

[ddnet/ddnet] Pull request closed: #3438 Replace unused chains in chat

hey

19:12

any dev there is blocker

19:17

Image attachment

19:17

here is it

19:17

cunt is blocker

19:17

and raven is my mate

19:17

he dont give him

19:17

and blocked much people

19:19

@snail @Akari

-/team

u like boomer but ıdk what boomer meanin

19:23

but u like boomer

19:23

its exatly true

19:23

c

19:28

@Swarfey racoon is better than you mate

im a raccoon

no u are looser of loosers test with finishing it 2nd

19:30

is it good mate

19:30

its not

19:30

then dont be weird like tEaM bRo!

if you want help report it in #reports not here

19:31

this is for development not for moderation (edited)

oke racoon is right

19:32

you won

raccoon is always right

can anyone get me setup with compiling teeworlds 0.6 using vscode? (edited)

vscode isn't a compiler

20:32

you need to pick a compiler first

xxd grep bamboozeld me when searching for the string --version xd https://zillyhuhn.com/cs/.1608670923.png

Embedded image

21:05

thanks to deen the hackerman i know that -- solves it :D

21:06

i spend so much time in the shell and still am not seeing how useless quotes sometimes are xd

[ddnet-web:master] 1 new commit

30053c5 Add donation by Gignr - def-

Exported 261 message(s)