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-06-21 00:00:00Z and 2020-06-22 00:00:00Z

[ddnet-maps:master] 1 new commit

bb6aabb M Blood Moon, M Milthrid Core, M Not Adrenaline 6, M Road to Rank 1, M 4luvz, M Arcade, M Deadline 2, M Fantabob, M Halloween, M HappyValenTees, M double up, M secreT, M valentees, M Against the Current, M Bob, M Castle Mario, M Chevtzo, M DayDrive, M Everlast, M Fried, M NightSky, M NightSky2, M NightSky3, M NightSky4, M Silk Touch, M Soft, M Spooky, M Sunny Land, M The Problem, M Zaiross-Gores, M sense, M BlueChaos, M Dark Moon, M Future_Race, M Illusion, M Paranormal, M Stereotype 3, M Triple Kill, M Aufnahmetest_AE, M run_alkohol, M run_for_onkelz, M run_linkin_park, M AIB Quest 2, M Aufnahmetest 2, M Aufnahmetest, M Impetuous, M combo, M solo_castle - ddnet-maps

[ddnet/ddnet] Issue closed: #2292 Learn button blocks client

09:31

[ddnet:master] 2 new commits

70b8bf9 Open links in background (fixes #2292) - def- 715ef25 Merge #2295 - bors[bot]

09:31

[ddnet/ddnet] Pull request closed: #2295 Open links in background (fixes #2292)

09:38

[ddnet/ddnet] Issue closed: #2293 Learn button doesn't fit in 5:4 resolutions

09:38

[ddnet:master] 2 new commits

1610079 Make the learn button fit (fixes #2293) - def- 80f19b7 Merge #2294 - bors[bot]

09:38

[ddnet/ddnet] Pull request closed: #2294 Make the learn button fit (fixes #2293)

https://hastebin.com/efuzaxetoy.cpp compiled a list of possible issues that may be closed

thanks

i think ill try to do https://github.com/ddnet/ddnet/issues/2080

Feature request: "Visible" players in /spec · Issue #2080 · ddnet/d...

As said by Boci#1337 on discord (link): i have a great idea when someone goes spec, you would see his ghost, of where he went spec like if he were in freeze, but like 40% opacity 6 other people, in...

10:48

it looks rly cool

10:49

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

Current weapon crosshair in freeze · Issue #2252 · ddnet/ddnet

This is a simple idea that I think would be useful. When you are frozen (or deep-frozen) your crosshair should represent the weapon you are currently holding, instead of showing the ninja crosshair...

10:49

this would be rly nice too

cool, yeah

[build] src/game/generated/protocol.h:163:41: warning: left shift count >= width of type [-Wshift-count-overflow] [build] 163 | GAMEINFOFLAG_USE_FREEZE_PLAYER_FLAG = 1<<32, [build] | ~^~~~ [build] src/game/generated/protocol.h:163:41: error: right operand of shift expression ‘(1 << 32)’ is greater than or equal to the precision 32 of the left operand [-fpermissive]

11:13

oh rip

11:13

reached the limit in game info flags

11:14

i want to make the server not send weapon ninja when frozen and instead use the ddnet player flags we already have

11:14

but i need a game info to ensure client supports it

11:14

if i got this right

nevermind gameinfo is for server capabilities, i wanted to know client capabilities

11:37

idk if there is another way than checking client version

Image attachment

12:27

omg

12:27

isnt this cool

12:27

you can even know what frozen tees weapon have

12:27

:D

can u see where he aims

yep

12:30

u can see if he holds a grenade too

12:30

and if ur dummy u can see too

but how do u distinguish frozen tees

12:30

from normal ones

12:30

if the normal tee has black skin

?

12:30

this is the ninja skin

12:30

u cant use it

yeah

12:30

u can

cuz it has x_ prefix

12:30

no u cant

what do u mean

unless u modify the client

12:30

x_ninja is hidden from selection

yes but u can download it

12:30

as a skin

well u would have the same problem right now

and some people changed it to have normal skins on frozen tees

no one uses ninja skin

instead of that

12:31

so how would they distinguish

Im gonna use ninja skin from now on and troll ryo

frozen tees from normal ones

what about cl_show_ninja 0

12:31

i use it

i take it into account actually

12:32

Image attachment

yeah i ment cl show ninja

Make it on off and it fixes the problems

well u would have the same problem right now

@Ryozuki no u dont coz when ur frozen u have particles and katana

hmmm

12:33

i can make a option for the people who disabled ninja skin to show katana then

12:33

they will lose info

12:33

but their problem

why is it our problem

Put them im the ice block

find a solution that it works with cl_show_ninja 0

Easy

yea

troll

i will add aa solution for show ninja 0

12:34

but u wont be able to know what weapon the freeze uses

12:34

cuz u decided to disable the skin

12:36

@jao @snail

Image attachment

12:36

right now

12:36

with cl show ninja off

12:36

it shows those eyes

12:36

and particles

12:36

isnt that enough?

12:37

well i can add another option for special ppl like you to not show weapons

12:37

so needy

those eyes can be put manually

the particles not

12:38

why would u disable ninja skin is a thing i dont understand

12:38

you disable a feature which shows visibility on who is frozen and then u complain lol

lol

to see what player i am hooking

12:39

imagine 10 people in freeze next to each other

i never had a problem

12:39

dont worry ill add the option

u cant deny that u can differentiate better with no ninja skin

cl_you_are_special 1 0 justatest

justatest

Just make the tee hold sign “im frozen”

12:40

Holy fuck

12:40

Lets do it

12:40

@Ryozuki

pls...

It would solve jaos problem kek

kek

MACRO_CONFIG_INT(ClUseKatanaFreeze, cl_use_katana_freeze, 0, 0, 1, CFGFLAG_CLIENT|CFGFLAG_SAVE, "Force the use of katana when frozen.")

12:41

ill add this

@jao +++++++ same for me

What did 0.7 require fixing?

12:49

I need to finish up today and take a nice long break

[ddnet-web:master] 1 new commit

dc9e41d add #Matecito - 12pm

GitHub :O so nice i love it

@snail @jao @Lady Saavik what about showing ninja particles and showing the weapon? To be honest, code wise this is way easier, right now i dont know how to show ninja weapon with this implementation

Image attachment

ye i thought about it

13:01

could be a solution

i think this is rly good

(also should be togglable)

what should be toggleable

13:01

the particles?

ninja particles

13:01

yes

@snail why tho?

13:01

right now u cant disable them

13:02

iirc

13:02

well ill make it

it will be weird to see frozen tee moving and changing weapon

true

13:03

lol

like he's alive

but well

13:03

its an option

why I should see someones wepaon?

well it will be weird to you cuz u dont use ninja skin

13:03

its useful in some parts

...

and its a requested feature

13:03

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

Current weapon crosshair in freeze · Issue #2252 · ddnet/ddnet

This is a simple idea that I think would be useful. When you are frozen (or deep-frozen) your crosshair should represent the weapon you are currently holding, instead of showing the ninja crosshair...

Image attachment

but isnt it meant for ur own tee only

ninja skin

13:04

still weird

why do u need to know from others

yes

why wouldnt u want to know others

13:04

i want to know others

for me this image looks like

13:04

mapper forgot to put freeze there

for me it doesnt

13:04

:)

what can u do with the info what weapon someone else has

u can know if hes rdy

13:05

and aiming properly

its good to know a frozen tee who has rocket and is aiming somewhere

can u see the aim?

for example

13:05

yes

from a competitive point of view its a rly useful feature

13:05

but ye

13:05

for casual players its confusing

13:06

should be turned off by default but its good to have that option

but if the frozen tee knows what weapon he will shoot it's already much better situation

Image attachment

than when nobody knows

need to fix abit the particles

so it will be his fault if he shoots idk shotgun instead laser and fails someone

13:07

what Ravie suggested is rly subtle yet useful change

13:07

just a small detail

13:08

and it will already help much

ye maybe make it like ravie suggested

I would at least wait with more changes

this is kinda overkill

i like to know what others have

I can't think about so many situations where I NEED TO KNOW SO BADLY

still i want to know

it will be a mess with 5 tees in freeze

not rly

oh is he deep is he 3sec frozen he is on edge so he will unfreeze soon

13:09

he has shotgun

13:09

he is afk

????

he is chatting

13:10

and every tee looks different

13:11

@Ryozuki for me your idea to see others weapon is as confusing as (what some people already wanted) seeing other peoples laser colors xd

for you :)

ignorant af

guess why you still don't see what laser color other people have

13:12

because it wasn't only for me

???

13:13

this circlejerk is awesome

13:13

keep it

I mean sure you can add it but let's test it with a bigger group of tees

13:14

u don't accept the fact some people can have other opinion?

i do, yet i prefer to give people more options

don't start being aggressive

you are the aggressive ones here

13:14

gatekeeping features

he doesnt want feedback if its negative

this is not negative feedback lol, its just "if i dont like it you dont add it"

i want to know others

@Ryozuki

13:16

for me it doesnt

@Ryozuki

i even adapted this

13:16

to ur requests

i like to know what others have

@Ryozuki

with on of stuff

I think it is something to get used too and then it has benefits

still i want to know

@Ryozuki

can you stop ping spamming me thanks

for now I only see 1 person that wants it

Lol

ah believe that

"if i dont like it you dont add i" and u said it

13:17

while repeating "I like it so I add it"

Good way to motivate developers @Lady Saavik

@n000b u get used to it here, no wonder only 3ppl still working in this game

[2:33 PM] Ryozuki: they will lose info [2:33 PM] Ryozuki: but their problem

13:17

best attitude

@jao i fixed what u said later

13:18

so

13:18

be grateful

lmao

im sure 95% if not 99% use ninja skin

13:18

too

13:18

i fixed ur special cas

13:18

e

13:18

by adding particles

13:19

Image attachment

lol

13:19

many ppl dont use ninja skins

13:20

i see it in a lot of screenshots

many ppl use it ninja skins

99% use entities still we have maps with designs

the image i sent is perfectly fine

ryo claming stuff without knowing it once again

with ur special case

13:20

its safe to asume most ppl use ninja skin because its behind a console setting, i think its common sense

like that more than 50% live in big cities?

I kinda need to agree with ryo, besides he fixed it so why are we discussing it xD

like that more than 50% live in big cities?

@jao yeah bring a joke convo from a stupid dc server here in the developer channel

13:22

this discussion is no longer useful

just pointing out that u always do this kinda stuff

nope

didn't know you are that unkind

"unkind"

13:25

gtfo

justatest

here i am developing in my free time

that doesnt mean stuff should be implemented without being thought through firstly tho (edited)

here we are discussing it

yea

13:26

discussing

13:26

I feel attacked

ye and u getting mad

oh you feel attacked

coz they are disagreeing xd

we just wanted to talk about some situations

13:27

what if there are more tees

13:27

what if someone doesn't use ninja skin

ill just add a option

13:27

to disable this

13:27

whole feature

and you are like yea you think it's problem I want it that way

ill just add a option to disable this whole feature

13:28

if u disagree even with this then ur gatekeeping features from others

you didn't have to repeat that

well i did

13:28

i felt offended

13:28

greenthing

so to be fair

but promise me

let's also add seeing other peoples laser feature

u wont use this feature

13:29

never

Have you all tried to chill? It's just a visual feature with a switch

ever

It doesn't even have to be on by default

it will be on by default tho

I'm not tlaking about this feature but his behaviuor

ill add a tick in the settings

13:29

visual checkbox

Yeah ryozuki does that, I've left for a month or so in the past

15:29] Ryozuki: but promise me [15:29] Ryozuki: u wont use this feature [15:29] Ryozuki: never [15:29] Ryozuki: ever I feel offended (edited)

look at it from my pov, 2 people trying to say this is shit just because they dont like it but they dont even think about if other people may like it

@Lady Saavik what do u mean with seeing other ppl lazers? collision lines?

@snail laser color

@snail you can set laser color in settings

ah

13:31

k

it has nothing to do with this

there were people believing it will be fun to see other ppls colors xd

XD

and whats wrong with that

omg with what

those people are stupid?

Could be cute, could be distracting

u make it sound like that

13:31

"xd"

WTF

it clutters ur screen

13:31

way too much

some people may be ok with that

objectively its not good gameplay wise

its like ppl who play with minimum graphics

13:32

cuz they want to be competitive

WTF @Ryozuki I FIND IT FUNNY WHEN I THINK ABOUT IT

other simply enjoy the game

satop attacking me

when ur frozen and u switch weapon just have a little indicator pop up for half a second right above the tee showing the weapon

The better part of the community just disable all the visual features anyway

I didn't say it's stupid or these people are stupid

you implied it

or that anything is wrong with that

with ur offensive behaviour

don't assume that

Image attachment

13:33

it rly looks like u imply that

um

13:33

please stop

why?

@Ryozuki the issue with adding a lot of optional features for "'people who want to" is that u then end up cluttering the settings menu aswell

13:33

making it a fucking labyrinth

13:33

for any new player

not if they are in the console settings

13:33

not rly tho

ye then ye

Features for a handful of people are just not a good idea

look at it from my pov, 2 people trying to say this is shit just because they dont like it but they dont even think about if other people may like it

@Ryozuki please stop you make the same to us!!!

It leads unmaintained features and thus bitrot

ur a bit cringy

13:35

@Learath2 do you consider this feature that

fine

Well I actually wanted this for a while, so I'm a little biased

I'm out then

we have a unused ddnet character info, we can relly on it instead of having it unused

It is used pretty extensively for prediction

and not used in rendering

13:37

which is wrong

I see their point aswell, it could get confusing with a lot of people

13:40

This is why I didn't make this myself, even though it's useful to have most people just don't like any change to the way things look

13:41

Remember when we messed around with the eyes?

Image attachment

13:42

so confusing lmao

13:42

btw dont use the names as an excuse because thats how it is rn

13:42

Image attachment

13:42

Image attachment

13:42

so many options

13:43

"looks confusing" nope, not any more than the actual

how will u know they are frozen

What's wrong with the stars btw?

whats wrong

13:44

?

then dont use max size nameplates

the names are not the problem here..

On the last one they look bunched up in the middle

i just took the pic when they spawned

13:44

whats wrong

wtff don't use nameplates at all

ic

i dont see how this is confusing

13:45

Image attachment

@Lady Saavik && @jao ^^ that doesn't really look that bad to me, what were you concerned about?

you can use ninja without particles too

13:45

@Learath2 they will never accept it

@Learath2 with cl_show_ninja 0

lookthe last

13:46

with cl show ninja 1

13:46

u blind

im talking about 0 not 1

i showed 0 too

13:46

did u even scroll upo

13:46

these ppl have no intention to accept this or even look the pics

I've never used cl_show_ninja 0 what did it do? show the katana but not change skin?

@Learath2 i think yes

13:47

no one uses it

13:47

only those 2

13:47

im p sure lot of ppl will like this

How about a cl_show_freeze_weapons 0 for the 1337 players?

im trying to do it

13:48

but its kinda hard

13:48

maybe u can help me

13:48

i kinda achieved it

13:48

but with the hammer equiped

13:48

it bugs a bit

13:48

well

13:48

with the weapons too

13:48

Image attachment

13:48

like they appear

13:48

in 1 frame

13:48

every x time

13:49

let me pr

hm, idk how you did the original patch, but whatever you did, just do it if cl_show_freeze_weapons 1?

[ddnet/ddnet] Pull request opened: #2307 add client capabilites and show weapon on freeze

@Learath2 ^

13:50

@Learath2 https://github.com/ddnet/ddnet/pull/2307/files#diff-2630e62d07aa9fee946b9100f48d0cf9R659

add client capabilites and show weapon on freeze by edg-l · Pull Re...

13:50

this is my try at hiding weapons

13:50

but its bugged

13:51

i think i have to set the weapon to ninja

13:51

somewhere else

@Learath2 maybe its not bad but his attitude is not nice

good

/r is reply in 0.7 what should I rename our /r to?

i found the fix

14:21

@jao do you see the ninja skin when you get the actual ninja with show ninja 0?

14:21

when u get the ninja weapon

14:22

well thats how the code before did it

idk

Image attachment

14:23

there is now a option to do it ur way

[ddnet/ddnet] Pull request opened: #2308 Fix command overlaps in 0.7

This also renames /r to /rescue for them, but it's not that much of an issue as they have completion for it.

@Learath2 if you can review my pr when you have time would be awesome

Let me finish this one too then I'll review yours, but I have a feeling @heinrich5991 should take a look at yours instead

does the 0.7 compatiblity work well?

14:34

ugh

14:34

infinite downlaod time

@redix apparently you changed how m_RecordType works. How is it backwards compatible?

14:44

I don't see why downloads are so slow

Hey! does someone ever experimented a "tee swap" on some DDRaceNetwork? xD I made a post of it x) https://forum.ddnet.tw/viewtopic.php?f=68&t=6941

@Learath2 heinrich changed that again. See https://github.com/teeworlds/teeworlds/commit/73f7182e50ddd9a9b6fac2d7ee93089e3d056673#diff-4c1480841d96bf73e07b29e10fbd394b

Make the race record message change backward compatible · teeworlds...

Image attachment

15:25

Image attachment

too afraid to suggest something to it

justatest

1

you can disable it

15:31

but ye go ahead and tell

15:31

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

Feature request: "Visible" players in /spec · Issue #2080 · ddnet/d...

As said by Boci#1337 on discord (link): i have a great idea when someone goes spec, you would see his ghost, of where he went spec like if he were in freeze, but like 40% opacity 6 other people, in...

ooh nice did someone make that yet

wdym

15:32

i'm implementing it

nice

@Soreu

@Ryozuki ?

look a bit up

@Ryozuki Wait a bit before merging the visible-spec, I will play around more with the skin. I.e. whiteish border around that black fill of dashed skin, like the nicks are displayed. Or the other way around - since deep is black due to ninja, maybe spec should use white dashed skin

sure

[ddnet/ddnet] Pull request opened: #2309 Show /spec tees with a dashed skin

Adds the option cl_show_spec_tee !image Don't merge until @Ekidoxx confirms it:

Wait a bit before merging the visible-spec, I will play around more with the skin. I.e. whiteish border around that black fill of dashed skin, like the nicks are displayed. Or the other way around - since deep is black due to ninja, maybe spec should use white dashed skin

@deen practice mode doesnt work if u get killed by spike

16:33

and it can be annoying with some maps spammed with spikes

Known bug

16:35

There were some problems, but not sure

when do we add switchable unfreeze to switch layer

switchable tele plz

you can do all of these by adding laser door to prevent people going in

too buggy

no, switchable unfreezes gives more ways to map

18:34

now u have to do it with deep which is more work, looks ugly and not always work

18:35

and a door needs a deactivator switch to let people go through, a switchable unfreeze an activator switch and maybe there are reasons i want exactly them

18:35

+switchable teles could be nice

all switchable blocks suck without animation triggers, they aren't really usable except for ways which the player doesn't really need to know about

you can visualize it with a door for example

you might as well just use laser freeze instead of switchable

19:58

optimally you would place a layer or quad that gets hidden or changes appearance once you hit a switch

@Learath2 only /rescue I guess

[ddnet:master] 2 new commits

ea3772c Don't send redundant commands. Remove /team on clientside. - Learath2 17f7836 Merge #2308 - bors[bot]

20:36

[ddnet/ddnet] Pull request closed: #2308 Fix command overlaps in 0.7

yeah it would be kinda dope to add switches to activate certain envelopes

21:00

like each envelope would have "activate on start" y/n and also "activate on switch n"

[ddnet:master] 1 new commit

9fccd5e Fix team rank for long map names - def-

[ddnet/ddnet] Issue opened: #2310 Crash in shared_ptr

@Zwelf ``` Program terminated with signal SIGSEGV, Segmentation fault. #0 std::shared_count::operator= (r=..., this=) at /usr/include/c++/8/bi 1078 class shared_ptr [Current thread is 1 (Thread 0x7f164c6afdc0 (LWP 22368))] (gdb) bt #0 std::shared_count::operator= (r=..., this=) at /usr/include/c++/8/bi #1 std::shared_ptr::operator= (this=0x3b0) at /usr/include/c++/8/bi #2 std::shared_ptr::operator= (this=0x3b0) at /usr/include/c++/8/bits/shared_ptr.h:103 #3 ...

[ddnet/ddnet] Pull request opened: #2311 Handle invalid packets more gracefully

22:13

[ddnet/ddnet] Pull request closed: #2311 Handle invalid packets more gracefully

22:13

[ddnet:master] 1 new commit

b1ff7d3 Handle invalid packets more gracefully - def-

[ddnet/ddnet] Pull request opened: #2312 Fix crash when player voting random_map leaves and vote passes

Closes #2310

Exported 452 message(s)