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-10-20 00:00:00Z and 2019-10-21 00:00:00Z
Avatar
ChillerDragon 2019-10-20 10:20:21Z
@Learath2 good idea mutes is a mess
Avatar
@TacTic comes from https://info.ddnet.tw/info so you'll have to replace info.ddnet.tw in the source code with your own server
Avatar
ChillerDragon 2019-10-20 11:09:38Z
isnt that what i just said? xd
Avatar
[quakenet] Ryozuki BOT 2019-10-20 13:39:40Z
Learath2 @Learath2
13:39
yes
13:43
Learath2 i agree some team code stuff looks rly weird, i don't get why https://github.com/ddnet/ddnet/blob/master/src/game/teamscore.h exists
DDraceNetwork, a cooperative racing mod of Teeworlds - ddnet/ddnet
13:44
solo and stuff is handled in many places its rly bad
13:44
if you want to check someone is in solo check it directly, not through this stupid class
13:45
also cankeephook and cancollide should be in the player or character
13:45
and it should also store the team info
13:45
no need for this
Avatar
[quakenet] Ryozuki BOT 2019-10-20 14:13:44Z
i think we also shouldnt care about ddrace16 at this point
14:13
ill try smth
Avatar
Ryozuki: The idea with teamscore is that it's shared with the client iirc
14:23
to make sure the client also uses the same rules i guess
Avatar
[quakenet] Ryozuki BOT 2019-10-20 14:26:06Z
im changing it
Avatar
Ryozuki: Get rid of TEAM_SUPER aswell while at it
Avatar
[quakenet] Ryozuki BOT 2019-10-20 15:11:25Z
im doing so
15:11
and also team flock
15:11
i deleted the whole file
Avatar
but team_flock is actually used, isn't it? "team 0"
Avatar
ChillerDragon 2019-10-20 20:30:03Z
We could see 0.7 as an opportunity to rewrite the server in bash 🙂
Avatar
ill need some more time for that rewrite
20:49
and weekend is over xd
Avatar
@Ryozuki can you atleast tell me along what line are you going to clean it up, so I can also get it in the 0.7 branch?
21:51
I guess I could work on mutes while you finish that one up
Avatar
i wanted to remove teamscore and save the team in cplayer and charactercore
21:52
cplayer cuz u can be in a team while ded
21:52
and u dont have a character while ded
Avatar
charactercore so we can mirror it to the client?
21:52
im not interested in 0.7 :p
Avatar
Even in 0.6 you'll need to find a way to replace the now missing teamscore
21:53
it's used for prediction
21:53
i need more time
21:53
but we dont need team super anymore
21:53
super is a character status
Avatar
yeah, i wonder who thought it was a good idea to have it as a team
Avatar
sounds good to me tbh
22:13
super as a team
22:13
at least in its current form
22:14
where you can interact with all teams
Avatar
but u still have the normal team
Avatar
do you? huh
22:17
oh, apparently
Avatar
@heinrich5991 I thought you could interact with all teams aswell, but that doesn't appear to be the case
22:38
(or I'm just not reading this correct, been to lazy to test it)
Avatar
it is the case
Avatar
@Learath2 src/game/server/ddracecommands.cpp:111 set character team
22:40
to TEAM_SUPER
22:41
CTeamsCore::CanCollide checks for either tee having TEAM_SUPER
Avatar
I stand corrected
22:41
Still, it's not a team, might aswell just check the character
22:42
I thought I was the one who loved to mix concepts just for ease of coding 😛
Avatar
if we want super to interact with all teams, I find this better than keeping the character in the old team
22:43
I dislike the concept of super complicating all our code between otherwise separate worlds though, so I'd be happy to drop that feature
Avatar
We are pretty much handling TEAM_SUPER everywhere, no?
22:43
super makes every code ugly
22:44
not everywhere
22:44
iirc
22:44
it makes no sense super for testing
22:44
only for trolling
Avatar
falae
22:45
false*
Avatar
with infinite jump and no freeze
22:45
its enough
22:45
true
22:45
*
Avatar
@heinrich5991 It could probably be neatly wrapped into if(CanCollide(cid1, cid2)) that abstracts away most if not all the complexity though, no?
Avatar
not really
22:45
you'd want to not have physics core have to concern itself with something like teams
22:45
at least I'd want that
22:46
i.e. having separate gameworlds with separate lists of characters, so that characters from a different team aren't even returned from the relevant functions
Avatar
we must check for super, teams, solo, collision on, nocollision, hook on/off and if colision is globaly enabled iirc
Avatar
@heinrich5991 Huh, isn't the physics core actually the most concerned part? Teams manipulate the physics of the game more then anything
Avatar
see above
Avatar
Having separate gameworlds while possible would complicate a couple things a whole lot
Avatar
u sure
Avatar
like spectators seeing everything and showothers
Avatar
i think it makes sense for a team to have its own "world"
Avatar
I see it complicating SUPER (maybe makign it impossible) and snapping a bit
Avatar
Ofc it makes sense, but I don't think we have the tools to "merge" worlds to get the combined views
Avatar
why not? except for deduplicated static objects like draggers etc?
22:48
just dump everything into the snap, as before
Avatar
I mean it'd all need to be handled in snap with fairly ugly code, no?
Avatar
no
Avatar
Something like for(auto t: m_pTeamsCore->m_pTeams->m_pWorld would need to be in a lot of places
Avatar
why go through teams?
22:49
just use your m_pWorld (edited)
Avatar
The teams will have the CGameWorlds, it makes sense that they own the worlds
22:50
For spectators I mean, for you it's obv just m_pWorld of your own character
Avatar
why teamscore
22:50
drop teamscore
22:50
there are worlds
22:50
and players in it
22:50
no teams as is
Avatar
Some other class name that will hold the teams stuff 😛
Avatar
m_apWorlds
Avatar
The team glue logic needs to go somewhere
Avatar
@Learath2 in the CGameContext::OnSnap function, you could iterate through the game worlds
Avatar
Hmm, I might try a rewrite like that tomorrow
22:52
Having an empty slate with 0.7 is nice as I get to try more 😛
Avatar
no uni?
Avatar
I can't convince myself to go, too much social anxiety
22:52
might aswell be useful to ddnet
Avatar
isn't it start of semester?
Avatar
It already started
Avatar
yes, a week or two ago?
Avatar
Well exactly 3 weeks ago
Avatar
then maybe go to uni ^^
Avatar
Interesting that this month started on a Monday, heh
Avatar
october the 0th was a monday, indeed 😛
Avatar
Oh no it's past midnight, I missed that
22:54
So close to a perfect month start
Avatar
the 0th?
Avatar
1st was tuesda,
22:54
y
Avatar
@heinrich5991 I tried going last tuesday, just couldn't get myself to, even had butterflies in my stomach 😛
Avatar
mh 😦
Avatar
Such a weird illogical feeling, even though I know no one gives two shits who comes and goes to a class, I still feel very anxious
22:57
Anyway, I have to go to milan for some bureaucracy anyway, it's a lost day, might aswell do some ddnet at the end to atleast feel better about myself 😄
Exported 124 message(s)