|
Notes |
|
|
(0003583)
|
|
MP2E
|
|
2012-05-07 10:08
|
|
|
|
|
|
I found another bug: You're not awarded Impressive / Most Impressive medals if your target is invulnerable through invulnerabilitysphere or respawn protection.
Quote from unknownna
Accuracy / Precision medals are never given with the rocket launcher / grenade launcher / BFG10K.
Perhaps it's the intended behavior, but since the wiki states that all weapons except the railgun is supposed to award these medals, I included it in the report. How come the railgun doesn't award those medals?
'http://wiki.skulltag.net/Medals [^]' |
|
|
|
(0003590)
|
|
MP2E
|
|
2012-05-08 00:05
|
|
Railgun gives you the "Impressive!" and "Most Impressive!" medals, and Accuracy is actually one hit above that. Seeing as it's more difficult to hit someone with a railgun consecutively they put it in it's own consecutive hit "tier" so to speak.
Rockets, grenades, and BFG10K shots are definitely supposed to give you accuracy medals from a few references I've seen in the code, I'm not sure where it's breaking yet. |
|
|
|
|
Quote from Wiki
Perfect
Defeating your opponent in a duel without being fragged once [...]
The medal is awarded if your opponent has zero/negative frags regardless of whether you've died or not. Should I create a new ticket for this?
BTW: Can somebody put MP2E into the developer group on the tracker?
|
|
|
|
(0003611)
|
|
MP2E
|
|
2012-05-10 08:43
|
|
|
|
|
(0003612)
|
|
MP2E
|
2012-05-10 10:49
(edited on: 2012-05-10 11:34) |
|
|
|
|
(0003613)
|
|
MP2E
|
|
2012-05-10 12:25
|
|
'https://bitbucket.org/MP2E/skulltag/changeset/eb6897266073 [^]'
Accuracy and precision now work as intended, and I updated Skulltag Version History.txt to reflect my changes. All of the changesets are in the bug_329 branch, it may be easier to pull that branch and merge at this point(after review of course :P).
This bug is considered closed by me unless any problems with the revisions are found. |
|
|
|
|
Regarding the perfect medal: According to the wiki, you're supposed to be awarded the perfect medal in LMS if you don't lose any health before winning a round. But if you have your handicap set to a value higher than 0, you're awarded the victory medal instead.
And even if you lose health and are supposed to be awarded the victory medal, you can still earn the perfect medal if you somehow manage to pick up some health, e.g., a soulsphere that isn't spawned by the map. |
|
|
|
(0003615)
|
|
legion
|
|
2012-05-12 02:38
|
|
Earning a medal in ctf while carrying the flag will obscure the flag from view if medals are on.
Since this said "medal issues" I thought I might as well throw this out there. |
|
|
|
(0003621)
|
|
MP2E
|
|
2012-05-14 02:13
|
|
This last perfect medal bug is tricky, I found the bug quickly but I'm not exactly sure what the best method of fixing it is as the server doesn't seem to store any knowledge of the player being damaged in the past or not. Not sure where this should go to avoid cluttering up the code more. I'll post something as soon as I come up with a good fix, though.
Hmm from the quick glance I gave the medal drawing code it looks like carrier icons(aka flag carrier) should not be obscured, I'll look into it. |
|
|
|
|
>'https://bitbucket.org/MP2E/skulltag/changeset/eb6897266073 [^]' [^]
> Accuracy and precision now work as intended,
This fix didn't work as intended. A single explosion could increase ulConsecutiveHits multiple times and missing with an explosion didn't reset ulConsecutiveHits. I adapted the fix to fix this issues and added it. Couldn't pull since I would have to pull all the fixes at once and the fix I used actually reverts your fix and puts the code you added at a different position. I'm still planning to move to pulling instead of manually porting the fixes though. We'll just need to discuss the details on how fixes should be prepared so that I can pull them.
> BTW: Can somebody put MP2E into the developer group on the tracker?
Done :). |
|
|
|
|
|
|
|
|
|
|
|
(0003628)
|
|
MP2E
|
2012-05-14 19:32
(edited on: 2012-05-14 19:42) |
|
Thanks, I actually wondered if ulConsecutiveHits would break in some way but in the testing I did(admittedly only with the railgun) it worked.
2nd commit: That was on accident. It was part of a debugging function I thought I'd deleted all of. Deleted in the commit after it.
3rd commit: Very true, I don't think we have to worry about monsters in duels but perhaps any ACS scripts affecting deaths without affecting frags could break this logic. I'm going to have to rethink this..
|
|
|
|
(0003631)
|
|
Torr Samaho
|
2012-05-14 19:54
(edited on: 2012-05-14 19:55) |
|
Quote from MP2E
2nd commit: That was on accident. It was part of a debugging function I thought I'd deleted all of. Deleted in the commit after it.
You removed the ZDoom code block
// Invulnerable things completely block the shot
if (res.Actor->flags2 & MF2_INVULNERABLE)
{
return false;
}
and as far as I can see you didn't add it again.
|
|
|
|
(0003632)
|
|
MP2E
|
|
2012-05-14 19:57
|
|
|
Ah reason being was that it was blocking the handled logic in Skulltag to deal with invulnerable players. P_RailAttack was never called in the instance of shooting an invulnerable player otherwise. |
|
|
|
|
|
I didn't check this in detail, but I guess if you remove it, you change the behavior of the rail. The comment in the removed block seems to say that the rail stops at an invulnerable target, i.e. anything behind an invulnerable target is not hit. |
|
|
|
(0003638)
|
|
MP2E
|
|
2012-05-16 02:31
|
|
|
I'm going to create a new branch and look through these changes a lot more carefully :P |
|
|
|
|
>https://bitbucket.org/MP2E/skulltag/changeset/b27855378823
Note: When you introduce new members in player_t, you have to properly initialize, reset and copy them (see the implementation of the other members).
For this fix no new member in player_t was necessary though. When reviewing your patch I had a slightly different and since I didn't hear from you about this for a while, I went ahead and implemented a different fix:'https://bitbucket.org/Torr_Samaho/zandronum/changeset/5167b204d18d [^]' |
|
|
|
(0003751)
|
|
MP2E
|
|
2012-06-13 02:29
|
|
|
That works much better than my fix, sorry for the lateness of any responses and the lack of activity. I am currently in-between homes(moving) and in-between careers. I'll make it a priority to polish off the rest of these bugs soon |
|
|
|
(0005194)
|
|
unknownna
|
2012-10-24 22:40
(edited on: 2012-10-24 22:41) |
|
I'm not too sure if rocket explosions should award you accuracy and precision medals. BFG rays don't award you medals. And I noticed that you're not awarded medals with rocket/grenade/plasma/BFG shots when you have a spread rune unless all the 3 projectiles hit the player. And you're never awarded medals with the BFG10K.
|
|