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-10-17 00:00:00Z and 2020-10-18 00:00:00Z

GUYS

05:15

Hello and sorry for disturb

05:15

Why i have closed eyes for all tees on mine personal DDNET server?

05:15

what command to able use eyeemote

05:19

I mean: all tees on server have closed eyes, what command responding for it?

close eyes may be because your tees are set as afk or you have /pause

05:45

@Люди = быдло. Discord = провал try to use command /emote

have you tried other emotes than normal?

06:02

really weird thing

https://ddnet.tw/settingscommands/

If you are on fng There is an extra command for wheel and it must be set in cfg before start

The problem was in non-working eye emotes, on server i cant do any eye emote, but i tryed to search for some functions to work it kind: svmax_afk_time; sv_input_info and another, but problem was in "sv_max_afk_vote_time" - (The time in seconds a player can be afk and his _votes still count (0 = disabled) 300) but cant imagine how VOTE VOICE TIME refers to EYE EMOTE. If somebody will have same problem, set "sv_max_afk_vote_time" to any number besides zero! I done!

kek

cant understand, sorry

07:53

@Deleted User About what extra command you told? And for what wheel: eye emote or just emote? (edited)

Is it even fng server else nvm

Yes, but i interested, please, can you explain more in details?

Just read fng.cfg

07:55

And the github page

07:55

It explains everything

oh, okay, thanks

Statistics don't look good on phone

Image attachment

I'll try to fix

[ddnet/ddnet] Pull request opened: #3115 Smooth Camera

closes #3096 Since nobody expressed any opinions, how about we get this in and see some reactions. it is off by default anyway.

@Lady Saavik better now?

yes @deen thank you

[ddnet/ddnet] Pull request opened: #3116 Show halloween skins only on Halloween

Making use of the skin downloader. Skin choice is based on your name, only active on October 31 and November 1 with events 1 in settings

oof I expected to be able to rejoin with timeout after pkill -f DDNet seems like it sends a SIGTERM and ddnet closes and disconnects cleanly :/

yes, use kill -9

yea now i know

10:18

kill sounds so strong i should keep in mind it trys to be gentle

10:20

gnome bugged on me i never had something like this beofore :/

10:20

time to switch to KDE @Patiga :D

!

[ddnet/ddnet] Pull request opened: #3117 Enable modernize-use-auto clang-tidy check

https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-auto.html And fix the code: cmake -GNinja -DWEBSOCKETS=OFF -DMYSQL=ON -DCMAKE_BUILD_TYPE=RelWithDebInfo -DVIDEORECORDER=ON -DAUTOUPDATE=OFF -DANTIBOT=OFF -DUPNP=OFF -DPREFER_BUNDLED_LIBS=OFF -DINFORM_UPDATE=OFF -DCMAKE_CXX_CLANG_TIDY="clang-tidy;--fix-errors" -DCMAKE_C_CLANG_TIDY="clang-tidy;--fix-errors" . Manually fixed sizeof(auto) in a few occurences (i.e. server.cpp:1359) First steps for #3088

be sure to enable wobbly windows in the desktop effects!

xd

10:25

omg it happend again

10:25

i blame ddnet client

10:25

ddnet is virused gnome crasher

10:27

ok it was a joke

10:27

but now im serious

10:27

when i get pinged in ddnet chat my gnome crashed

10:28

2 times in a row

I think I nkow what's causing this

cl_notifications 0 it is for now i guess

8ce6de6d8e

10:28

try reverting that

does anyone know the reasoning behind map sounds being allowed to be external btw?

reverted can someone come ingame and ping me? :)

just because it's the same in images

surived ty @Patiga

10:31

deen too slow xd

^^

ok seems ur guess was right

[ddnet/ddnet] Pull request opened: #3118 Revert "Set libnotify icon"

Seems to cause crashes in Gnome, for ChillerDragon at least This reverts commit 8ce6de6d8e520f96724d9e2c4c8da8f24e3e72a8.

i wasnt running master before but the ddnet-git aur package but it should be the commit anyways

ah ye i also dont really understand why the images are not automatically sorted into external/embedded

oh cmn i lost map progress again due to CTRL-C xd

10:32

i suck

10:32

guess now i know

10:33

oh wait guys the test was flawed xd

10:33

i had cl_notifications 0

10:34

rip

10:35

ok @deen seems like reverting 8ce6de6d8e520f96724d9e2c4c8da8f24e3e72a8. did not help

10:36

i can try to bisect somewhen else but im currently in the middle of something

10:36

ill open a issue for now i guess

[ddnet/ddnet] Pull request closed: #3118 Revert "Set libnotify icon"

10:39

[ddnet/ddnet] Issue opened: #3119 gnome crashes when tabbed out and pinged in chat

cl_notifications 1 and fullscreen using gnome or arch linux. If pinged in chat while being tabbed out gnome acts broken until relogging. I can do further testing if I find time. Reverting 8ce6de6d8e520f96724d9e2c4c8da8f24e3e72a8 did not help.

lmao i manged to get pinged in chat in a message without my name xd https://zillyhuhn.com/cs/.1602931225.png

Embedded image

https://en.wikipedia.org/wiki/GNOME_Shell Written in JavaScript and C it cannot be taken seriously

GNOME Shell is the graphical shell of the GNOME desktop environment starting with version 3, which was released on April 6, 2011. It provides basic functions like launching applications, switching between windows and is also a widget engine. GNOME Shell replaced GNOME Panel an...

both fine and fun languages

you think using things like JS for core system components is OK?

10:55

a lot of people were making fun of Windows when it had some GUI things integrated in the kernel, and now core of main Linux desktop environment is written in JS....

is the core written in js? tbh idk i thought its more for the extensions and stuff

11:40

btw coming from the dude who runs a electron app to chat xd

11:40

[ddnet/ddnet] Pull request opened: #3120 Embed map sounds by default

Not one ddnet map has a embedded map sound since there are none which would work embedded.

Discord's web version works well

12:51

no reason to use that Electron junk

Not much difference between running it in a browser or in electron. Might aswell get the better integration of electron

when you run it in web browser, it can't mess with your system much (edited)

12:57

but with Electron it can

12:58

sneaking what apps are running etc

[ddnet/ddnet] Pull request closed: #3120 Embed map sounds by default

13:25

[ddnet:master] 2 new commits

17e5313 Embed sounds by default - Patiga 996270a Merge #3120 - bors[bot]

[ddnet/ddnet] Issue opened: #3121 Add a HUD notification to tell players how to unpause.

Multiple people have asked me why are their dummy/main tees sleeping when switched. Although hilarious, we should probably have a hint to tell players how to unpause, since they are most likely accidentally hit p before entering chat while chatting.

I finished the grass_doodads svg version btw https://github.com/TeeworldsDB/mapres/blob/master/svg/grass_doodads_chiller.svg Idk how to export pngs from it properly and its untested but at least its finished xd

TeeworldsDB/mapres

A sorted collection of teeworlds mapres. Contribute to TeeworldsDB/mapres development by creating an account on GitHub.

nice looks good

:)

[ddnet/ddnet] Issue closed: #3110 Cursor position in text field overlaps text

15:08

[ddnet/ddnet] Pull request closed: #3112 Fix and change text cursor

15:08

[ddnet:master] 2 new commits

186206d Fix and change text cursor - Jupeyy b93896b Merge #3112 - bors[bot]

@heinrich5991 a2b3f45f8b105ebbd6e391c48a2c529ddb1ed09c broke our configurations

15:13

Cannot execute game command 'sv_team', put it into the map config or start the server with 'sv_test_cmds 1' to enable it most critically sv_solo_server

15:14

@deen is ger4 not compiling it's own server?

15:19

How is every single one of our servers running a different version? :/

justatest

1

ChillerDragon: Want to add ultra hd option in client?

15:22

@Learath2 they are all built on ddnet.tw and scped from there

15:22

@Learath2 so solo server is currently broken? Should we quickly revert and deploy?

15:23

so all servers should run the same version, depending on when they last restarted of course

I'm just going to revert 7e84e1ef43516beeab579a850d664242e91d2843 @heinrich5991 can re PR when he is around. We basically need some sort of unsafe-exec or atleast a time when we can unsafe-exec

ok, thanks

How do I get it to deploy to all servers btw?

yeah, before map start would be great

15:23

ssh ddnet.tw "cd src; ./build.sh

15:23

"

15:24

~/servers/all-restart.sh afterwards to start running a bit earlier

15:24

also on ddnet.tw, no need to connect to the other servers manually (they are too many for that now anyway)

@deen is this a question or a request?

A joke/request/question combo

15:27

mostly just an idea of what we could do with the svgs

Did you know that it's not trivial to revert merge commits?

yes

15:28

revert all changes in the PR in reverse order

15:28

and squash that with git rebase -i master

15:28

is what I would do

Odd that git doesn't have that as default

@deen well i just made the individual doodads for my movie and website and did the whole tileset on request on Jupstar. I play in high res I would love to see better graphics but I currently do not plan to thinker with client code reagarding this

I assume git revert -m with the correct parent would do that

but apparently that has the weird behaviour of still leaving A and B in the repository, so after @heinrich5991 fixes it, re merging it would apparently only merge the new commits he makes

15:32

https://github.com/git/git/blob/master/Documentation/howto/revert-a-faulty-merge.txt

Git Source Code Mirror - This is a publish-only repository and all pull requests are ignored. Please follow Documentation/SubmittingPatches procedure for any of your improvements. - git/git

ok, never used that

[ddnet:master] 1 new commit

ba7b95f Revert #3066 - Learath2

@deen are you perhaps editing build.sh on ddnet.tw?

I opened it in vim, but only read

15:35

because you asked me about it

I still think we should add the git hash of the server on the finishes

yes

yes

would help with bugs

15:38

is it in the teehistorian files?

15:39

then we could extract it retroactively from all existing teehistorian files

Yep it is

15:42

"server_version":"0.6.4, 15.0.5 (d3691bf221c2d6d2)"

so basically we have the info already, it's just not so easy to access

@deen Copying to servers seems to hang

@Learath2 it's done slowly and some servers (CHN) can do 5 KB/s

Ah

yeah

Hm, that might be a good reason to compile on those servers

ps aux|grep scp shows the CHN servers are the ones

15:43

but git pull can also take 5-10 min or just fail

Amazing

no one plays solo on CHN anyway, you could still restart the rest already

is CHN last on the list though?

they are copied at the same time

Oh, great

all others are done

thonk

hi

@deen btw cat: all-locations: No such file or directory we should probably make all our scripts cd to ~/servers

oops, yeah

15:46

hi @TsFreddie

Btw you could have put all-restart.sh in terms of execute-all.sh instead of restart-ddnet-on-empty.sh

I didn't write that script, but feel free to rename

saw you guys talking about CHN servers, and I realize I have no idea what you guys are doing

15:47

so carry on

Solo servers are broken, Learath2 is reverting the change and deploying to serv ers

15:48

and CHN has slow internet to ddnet.tw main server

oof. ye, you can probably close CHN solo servers, nobody would even notice

There are no dedicated solo servers anyway

15:49

Okay that should be all the servers

15:49

now I'll join the servers they are cheating on and kill those by hand

thonk

wait so what is broken

see #bugs, solo mode

15:51

so you can touch others

@deen since we'll have to delete the solo ranks from today anyway, I'll just restart the solo servers that are running the old version even if people are playing on them

how will you enforce that people don't change server type to solo on other running servers?

15:54

I think you have to pw-protect all, or alternatively remove solo option from votes (but that is super hacky)

15:54

(and disable /map)

@deen hm, ok Ill pw-protect old servers

Or remove all votes?

15:57

and disable sv_map

15:57

I mean most people never change map anyway

15:57

would be less painful

so sv_map 0, clearvotes

yes

what about servers that are already solo?

hard restart

15:59

or get in with gdb and set them to solo

I feel bad for the people playing

15:59

I guess it has to be done

Tell them you're sorry?

Oh, version.h is broken again

16:07

@deen somehow servers running 282508 show up as 15.0.5. 282508 is 15.1.3 :/

16:10

Well apparently all servers show up as 15.0.5 bow

16:10

now*

Ah, I know why

16:10

that's my fault

16:11

cmake caches the version and overwrites the one in version.h with a define

16:11

I have the same problem locally. A clean build should help. @heinrich5991 as the cmake expert might have a better idea

Uh cmake gets it from version.h but caches it?

i have no eye on the ddnet proj so idk baut current issues so maybe this is known already see kill feed https://zillyhuhn.com/cs/.1602950996.png

Embedded image

yes

16:11

so that I can overwrite it for RCs and nightlies

theoretically cmake automatically detects when the version changes by looking at the git dir

16:12

but we have consistently managed to break it

ChillerDragon: how to reproduce? Not known to me

as a precatuion, maybe it would be good to just delete version.h (or wherever that thing is) before the build on ddnet.tw

16:12

that information is kind of crucial and hard to see when it is wrong

idk saw it first time @deen its probably the ppl reconnecting I can also provide the demo if it helps

ChillerDragon: if the demo reproduces it, would be great

k hope i dont forget after the run

@heinrich5991 but version.h is not auto-generated, it just reads the define. I guess a touch version.h should work

As a precaution we should always rm -rf the build dir and create a new one

I do that for the releases

I mean in build.sh

16:15

I was thinking maybe we should get rid of version.h

16:16

We could add a game/generated/version.h instead which cmake can generate

16:17

This reading from version.h sounds redundant to me

true

[ddnet/ddnet] Pull request opened: #3122 Fix arithmentic problems

The first three commits are safe The ones i'll add later will be inside the gamecore, so might alter behaviour. I'll see which ones are critical, but takes me some time i fixed most in #3081

timeout code 123 last stage at Patchwork 6h time tho xd timeout any minute

Okay, cmake seems to have no way to distinguish between a variable set in the cache and a variable set on the command line

[ddnet/ddnet] Pull request opened: #3123 Add workaround for CMake 2.8. Fix version caching

17:19

nervous

really can't beat real dyncam

17:19

me playing fng with my dyncam without changing habits. still suck at aim with that thing on.

17:20

but at least i won't feel sick i guess?

@Learath2 did you delete ranks and teamranks on solo maps already?

Not yet

[ddnet/ddnet] Pull request opened: #3124 Remove option to make map sounds external

!2020-10-17-180507_401x333_scrot Right now it is impossible to have a working external sound. External sounds render unusable the moment the map is saved and given to another player. For the mapper it seems like everything is working since the mapper has the sound file. For anyone else it simply won't work. Since 1. there is no use for it and 2. it adds unnecessary complexity and conf...

^ can i somehow view the 4 surpressed clang-tidy errors?

18:12

the first one is easy to fix, are the 4 surpressed ones irrelevant or do i first get to see them when i fixed the first one?

18:15

ah

18:15

'1 warning treated as error'

18:15

maybe shouldve read the whole message

ChillerDragon: you around?

whats dis

19:41

oh

19:41

dev

@Дядя Женя what gpu u have btw

did you know that ssh reads from standard input?....

19:56

CHL really is one potato server...

what's wrong with Chile?

it's having trouble connecting to the debian mirror took like a minute

oh well, not too bad

recently they shut down the server every day for 4 days because they thought I didn't pay, bu tI did

19:57

and then they broke the system clock

19:58

and we noticed because people had finishes before maps were released

anyway, I set up a little script to get the versions of finishes from teehistorian files on the servers

@Deleted User gigabyte gtx 1660 ti

ty

it doesn't have a fallback to our actual teehistorian database so it can only check however much we have on the servers, but I think that should be enough

@Learath2 not sure I would do that straight on servers, might cause lags

20:00

maybe do it from our teehistorian mirrors

it's rather trivial to parse the header

20:00

it's in json

I'm still syncing back my teehistorian backup, so my server is not complete yet

[ddnet/ddnet] Issue opened: #3125 Kill message names disappear when switching between fullscreen and windowed

On Arch Linux.

@deen hm, /root/.ssh/authorized_keys doesn't seem to be the same on all servers

20:05

rus's seem to be missing ddnet.tw's

20:06

chl too

20:08

rus, chl, usa, jap, sgp, zaf all seem to be missing root@ddnet.tw's key on their root so the main server can't ssh to them

20:09

With this many servers we really should look into something like ansible to deploy these so they end up identical

CHN3 can't even resolve it's debian mirror

20:18

even though it's mirror is located in china

20:22

idk who hosts http://mirrors.cloud.aliyuncs.com but it seems to be out of commision and chn6 and chn3 use it

oof

fix it

looks like Alibaba's

The apt settings are generated with some cloud magic I'm not familiar with, not sure if I'm allowed to even touch it

There should be other mirrors. I remember both Peking university and tsinghua university host mirrors for a lot of things (edited)

20:27

https://mirrors.tuna.tsinghua.edu.cn/help/debian/

debian | 镜像站使用帮助 | 清华大学开源软件镜像站 | Tsinghua Open Source Mirror

清华大学开源软件镜像站，致力于为国内和校内用户提供高质量的开源软件镜像、Linux镜像源服务，帮助用户更方便地获取开源软件。本镜像站由清华大学TUNA团队负责维护。

20:27

They do

debian has a load balancer mirror in cn too, the problem is idk where to change it

tee_thinking

2

20:28

usually the mirror would just be used in /etc/apt/sources.list

learath using a linux with systemd monkalaugh

monkalaugh

apt is not part of systemd yet thank god

dabbing

though it's the first time I've touched a system using cloud-init]

Windows guy here, I'm out

20:30

Oh right can I get a role

Which role would you like?

20:32

You are the music bot now

@Learath2 https://guix.gnu.org/

GNU's advanced distro and transactional package manager — GNU Guix

Guix is an advanced distribution of the GNU operating system. Guix is technology that respects the freedom of computer users. You are free to run the system for any purpose, study how it works, improve it, and share it with the whole world.

20:32

stallman approved distro

20:32

monkalaugh

20:33

@Learath2 t it uses GNU Shepherd instead of the now ubiquitous systemd.

20:33

https://www.gnu.org/software/shepherd/

Coded in scheme, lmao

made with https://www.gnu.org/software/guile/

GNU's programming and extension language — GNU Guile

Guile is the GNU Ubiquitous Intelligent Language for Extensions, the official extension language for the GNU operating system.

This is why we can't have nice things, people who can't stop creating their own languages

???

20:38

i dont see how this prevent nice things

Person A wants to create nice thing A-thing, A instead creates awesome-A-lang then codes A-thing in said lang. No one can read thus contribute to A-thing. So A-thing will never be nice

wrong

20:40

u assume people wont learn A lang to do said thing

That's a rather fair assumption. When is the last time you learned an entire language to contribute to something or use something?

i love learning new languages

20:41

but i have yet to find a project in a language unknown to me

C++ for ddnet

I personally just use an alternative that is in/uses the 7 or so languages that I (and most of the programmers out there) are familiar with

atleast it looks cute

Image attachment

I wouldn't learn Guile to contribute to Shepherd, I'd use/contribute to OpenRC, minix, runit, busybox

wow discord ruins it

20:42

Image attachment

Heh, transparencygang

@Learath2 i would learn guile to contribute to GNU shepherd so gnu stallman recognizes me as a gnu propheyt

20:43

gnu guile*

How can alibabas debian mirror die? :/

Okay, tsinghua uni's mirror works fine

20:54

hope I didn't break anything

../src/engine/external/json-parser/json.c:402:65: runtime error: applying non-zero offset 5 to null pointer
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior ../src/engine/external/json-parser/json.c:402:65 in

@Deleted User Should we exclude external?

but still sounds bad?

(we should probably update json.c)

i just ran it with all

maybe we pass in some bad json object?

oh or that

i dunno, but doesnt sound healthy xd, but could ofc be false positive (edited)

I wonder what is null at L402

CServerBrowser::LoadDDNetInfoJson()

20:59

that is the caller

(*(json_char **) &top->u.object.values) += string_length + 1;

20:59

@Deleted User how did you figure out the caller btw?

call stack from ubsan

21:00

UBSAN_OPTIONS=print_stacktrace=1

fwiw east added the json parser 6 years ago, last commit on the json parser is from a year ago, an update might not be the worst idea

I updated it to 1.1.0 3 years ago

21:04

And there has been no new release since then

21:04

https://github.com/udp/json-parser/tags

udp/json-parser

Very low footprint JSON parser written in portable ANSI C - udp/json-parser

@Deleted User can you print out the json we are passing json_parse_ex, see if it's malformed maybe?

@Deleted User https://forum.ddnet.tw/viewtopic.php?t=7003&p=68629#p68629

21:06

(but even if a string is malformatted json the parser should not run into ub)

@deen #3108

https://github.com/ddnet/ddnet/pull/3108

Make GL 1.5 default by Jupeyy · Pull Request #3108 · ddnet/ddnet

If ppl see white textures its related to #2125 and you need to revert it, bcs thats not fixable, there is no extension for GL_GENERATE_MIPMAP or smth

thats the fix

Yes, let's merge that?

Should we look into a more robust json parser?

yes, then i am finally free again

and should I downport it into a new instant release?

21:07

because next release will take a few weeks at least

idc, just do what u want

21:07

i'll remove the gl code that is not 3.3 some day

21:08

and in 17 years we just default to 3.3

21:08

when teeworlds gets 30 years old

[ddnet:master] 2 new commits

ceefe30 Add workaround for CMake 2.8. Fix version caching - Learath2 fa0bdee Merge #3123 - bors[bot]

21:09

[ddnet/ddnet] Pull request closed: #3123 Add workaround for CMake 2.8. Fix version caching

I'd guess teeworlds will be staying with gl1 for a loong while

[ddnet:15.1-devel] 2 new commits

fadd3e5 Make GL 1.5 default - Jupeyy ed42a44 Version 15.1.4 - def-

@Deleted User u think anyone would be around in 17 years

21:09

will*

i rq before

Of course we'll all be around and discussing about whether we should finally add accounts

btw @deen someone with intel had a crash even in gl 1.x ;D

but dont tag me then

btw @deen someone with intel had a crash even in gl 1.x ;D

@Deleted User then whats the point

21:10

of the downgrade

How come you tell me this after I prepare the release

i dont get tagged or blamed xD

But you changed most of the graphics code

@deen ???? i tagged u

I wonder if an older client helps them not crash

yes

https://discord.com/channels/252358080522747904/757720336274948198/766714731188191282

so there is something wrong in our code, but we can't reproduce it

there is something wrong in intel xD

but we had no such crash reports for years before

i had no crash ever

21:11

atleast not one i provocated

That's the "we can't reproduce it" part

21:12

I'd kill to see a crashdump :/

@deen maybe the issue exist since forever but we didnt have enough players with x setups to get that

Does it happen to anyone tech savvy enough to run it in a debugger?

no

21:12

its all normies

@Ryozuki no chance

21:12

@Learath2 enough to integrate the crash reporter?

@deen heh, I wish I had the time, maybe I can slowly work on it at nights

https://cdn.discordapp.com/attachments/295908390956433410/767003311290843136/unknown.png this for example would be a proof for a bug OUTSIDE the gl code, bcs i checked and we never render such a huge quad, nor are we buffering chat text in not gl 3.3 in VRAM (edited)

Embedded image

Maybe we should ask someone who had the crash to give us teamviewer access?

21:13

and we install a debugger?

lmao

@deen I'd be willing to help someone with it but it needs to be someone patient

the worst is most of times they dont even speak english

Oh and someone that has good internet because Visual Studio is like 20G now

justatest

Gotta love windows

isnt mingw smaller

21:14

cant u debug there

Does gdb work on Windows?

Oh we actually cross-compile with mingw

21:14

I should be able to use gdb yeah

yes

but we don't have symbols, so I'll have to pray the crash still happens when I build on the guys computer :d

21:15

Anyway, find me someone and I'll make it happen

Then let's make a special build with symbols for that

21:16

If we get 1 guy reporting this issue per day, there must 100 more who don't report it

[ddnet:master] 2 new commits

537f47d Make GL 1.5 default - Jupeyy 54c7a5b Merge #3108 - bors[bot]

21:16

[ddnet/ddnet] Pull request closed: #3108 Make GL 1.5 default

Oh, btw it's probably not the driver itself crashing, driver crashes lead to bsods on windows. It's probably either our code or the userspace part of the driver

21:17

it should be evident on the crash screen btw, if the userspace part of the driver crashes windows will tell them

btw we did debug the crash

21:17

the one guy with the intel gpu and me

21:17

for like 1h

21:17

and found nothing

21:17

and when we updated his driver it instant worked

It doesn't crash under debugging?

so do you have some backtrace, where did it crash exactly?

it random places

21:18

well it just didnt continue processing

21:18

so not a real crash, true

it freezes?

it doesnt show anything anymore

21:18

but that was different from intel to intel

if you interrupt the program with a signal, where do you land? does it appear the program is still running?

yes all threads were working

21:19

no infinite loops

21:19

nothing

all threads are working as expected but nothing on the screen?

yes

21:20

it internally died

21:20

and on other intel platform probs the same behaviour caused crashes

21:20

instead of, whatever u want to call this

mh, graphics debugging I'm not really good at

its not really possible

21:20

on linux maybe with open source drivers

21:21

but never heard of any bug or crash in linux the last 2-3 years

Is there a way to make opengl verbose?

yes

21:21

already did that, but nothing really interesting

It'd be nice to have as a setting btw, something like gfx_debug 1 we can tell people to enable so we can see it in the logs

it requires opengl 4.3 tho

21:22

xd

21:23

well atleast the driver needs to support it

I seriously hate all the graphics people

I F3

How did people debug their damn programs before 4.3 then???

well amd and nvidia probs have tools

21:24

but no standard

@Deleted User there are extensions for opengl 1.0 tho

U fix my problem @Deleted User

21:24

happy

i fixed nothing XD

lmao

u did

21:24

:c

intel fixed it for u

D:

they successfully support gl 1.x

21:25

clap

brownbear

@Learath2 btw its very hard to crash opengl

21:25

bascailly only when uploading or deleting buffers

21:26

everything else is defined somewhere in the specs

21:26

all drawing and shit is not allowed to crash

how would you crash opengl anyway? it's just an api

but it says what the impl has todo

21:26

so its more a standard

21:27

anyway, i gave tw a try for modernization, failed, doesn't matter whom to blame, me or intel

21:27

we'll stick with borderfix for the next 10 years

Wait, that means we need borderfix again?

well we need all gl 1.x stuff again

oh no

that was the idea of the new renderer xD

so I need to also revert all the fixed graphics?

nvidia and amd have like 90% marketshare.. so they all could easily get opengl 4.5 running

just for 0.0001% of ppl

21:29

monkalaugh

@Deleted User that is only for dgpu

21:29

Intel has 60-70% total gpu market share

steam charts

21:29

i doubt

everyone has intel cpus and they come with integrated gpus

yes

21:30

wow

21:30

doesnt mean its used

I have 4 computers in my household, all only with Intel

Image attachment

@deen let's revert #3108 for now and try to debug instead?

https://github.com/ddnet/ddnet/pull/3108

Make GL 1.5 default by Jupeyy · Pull Request #3108 · ddnet/ddnet

If ppl see white textures its related to #2125 and you need to revert it, bcs thats not fixable, there is no extension for GL_GENERATE_MIPMAP or smth

Reverting all the graphics changes will be a whole mess

yeah

yeah but if it doesnt work, it doesnt work xD

and borderfix being necessary again is another mess

Is the 0.7 client crashing for these people too?

I thought borderfix was independent since Teeworlds 0.7 also doesn't need borderfix and has opengl 1?

the problem is 0.7 still uses mostly 1.x code

21:32

@deen i'm aware, they use 3d textures without mipmapping

21:33

@deen go in 0.7 load multimap into editor

21:33

zoom out

21:33

0.7 try is completly useless

21:33

if you dont create mipmaps you can also NOT use 3d textures

[ddnet/ddnet] Pull request opened: #3126 Revert "Make GL 1.5 default"

As discussed with Lerath2, we should try to debug This reverts commit 537f47d11d8c637c36cb698f51ee654abff76e26.

Are we linking to GLEW correctly?

r we using glew?

yes

@Learath2 there are actually long error messages for glew linking specifically: https://ddnet.tw/downloads/win32.log

@Deleted User are there things like validation layers for opengl? xd

Image attachment

there is validateprogram

21:45

u can basically only crash opengl when creating binary data

u may not crash opengl but u may be doing things wrong

21:46

and not know it

or drawing too much

21:46

but we need a crash xd

eh? what is even compiling glew.c??

[ddnet/ddnet] Pull request opened: #3127 Also compile glew.c with GLEW_STATIC

This should fix the warnings when linking to glew on windows

@Deleted User does checking in order for KHR_debug, ARB_debug_output and AMD_debug_output sound about right?

if it can output invalid buffers and draw calls yes

[ddnet:master] 2 new commits

ad3bc98 Revert "Make GL 1.5 default" - def- 2edec49 Merge #3126 - bors[bot]

22:01

[ddnet/ddnet] Pull request closed: #3126 Revert "Make GL 1.5 default"

I just want to see what kind of output I can get out of it, not sure if it'll be helpful at all

if its something different to invalid operations or smth yes

[ddnet/ddnet] Pull request opened: #3128 Add add_map_votes command for easier local map testing

No need to tell new players how to manually add a vote or load a map from rcon

22:09

[ddnet-web:master] 1 new commit

a68347c Add countryflags - def-

[ddnet/ddnet] Pull request closed: #3127 Also compile glew.c with GLEW_STATIC

22:19

[ddnet:master] 2 new commits

be834b5 Also compile glew.c with GLEW_STATIC - Learath2 3fb24f7 Merge #3127 - bors[bot]

wtf

22:27

was glew_static removed

hi

22:29

how do i disable antibot on lan

by not connecting bot

[ddnet/ddnet] Pull request opened: #3129 Editor: Fix selection of None value in auto rules (fixes #3013)

by starting the indexing at 0

In good news all our shaders are completely valid

22:52

in bad news I spent an hour trying to preprocess them in every configuration and write a script to get them out of tw

Thank you for doing that

22:54

@heinrich5991 are ghost files easier to create from teehistorian than demos btw?

@deen you could create limited ghost files in the same way as teehistorian2demo creates limited demos

22:59

ghost files also contain velocity, and hook position in addition to current position

Hm, too bad. So need the full server to recreate ghost too

yes, it was the intention to require the full server

23:00

to save less info

Huh, surprisingly only 2 finishes on versions with broken solo

23:21

good thing I didn't nuke every solo/race run from today

justatest

2

did you see the teamrank by N9?

@louis lol wdym

he wants kclient on lan server

23:34

the sv_ban_kclient option wasnt showing up in console

does he want many tees

23:36

@louis just download the newest botless version of kclient, its not detected by that check and u can still use the 64 tees

no botless doesnt work either

23:39

i might have an outdated kclient but there should be a way to disable anticheat on lan

u do have outdated then

23:41

cus it should work

ok

23:41

sv setting would be nice tho

i guess

so i can test my fng bots

banhammer

4

23:45

troll

...

23:46

ban nouis

[ddnet/ddnet] Issue opened: #3130 Check list for PRs

I think we should introduce some kind of check list for PRs to help us remember what to look for. Some ideas:

Described the motivation of the change

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

No physics changes affecting existing maps

Exported 547 message(s)