Heres the first patch for hitscan accuracy: http://www.pastie.org/342960

please give feedback.

for which SVN revision is that patch?

The patch consists mostly of re-indenting existing source I really cannot judge it in that state.

http://paste.debian.net/24035/ is a cleaned up version. Haven't had time to look at it yet.

Awesome, you got it finished!

I'll test it during the weekend

Cheers

It'd be nice if u explain us how to show the stats.

sorry about that: bind key "+showstats"

Dib, your patch is hard to apply for me, I get errors coz of the wrong paths of files (I use tortoiseSVN in WinXP) but i guess it causes troubles to other ppl on other systems too. U should find a way to generate the patch from the NexuizSVN directory, or from NexuizSVN/nexuiz, as u can see in the patch modified by Div.

Div...
u accidentally removed the code to modify main.qc and with your patch there was not the command +showstats defined :/
And i was using this one in the beginning...

Here is the link to the corrected version http://www.pastie.org/343797, it works fine in my SVN.


Talking about the content, I guess that maybe is better to place the stats on the bottom rather than on the top of the screen. After a game is ended, the scoreboard is always showed and when I active your stats, they are just over the scoreboard. The end of the game is the perfect moment to see the stats but all is confused and it isn't nice to see.

Why this patch has code for the crylink, hagar, rocket launcher, etc, etc if it is only for hitscan stats?

Do you have plans for including non-hitscan stats anytime soon?

If this is only for hitscan weapons, shouldn't be the client command something like "+showstats_hitscan" ?

Are we missing the minstanex weapon ?

I have the same concerns as terencehill about how to show these stats. I consider the following options:

* Show them in a new right or left bar showing only the weapon image and hit ratio, without text.
* Show the hit ratio over the weapon image in the hud
* Leave the board as is, but put it right below the scoreboard. This might require making the stats board a little smaller

I've patched the code and it works perfectly. But I'm very concerned about modularity. With this patch adding a new weapon would require changing code in many places, and it can be avoided. I will try to explain this concept just a little :

You #define MAX_WEAPONSTAT 16 in data/qcsrc/client/main.qh . Why not use WEP_COUNT ? (data/qcsrc/common/items.qh)

In data/qcsrc/client/View.qc you declare

Code: Select all

float hitscan[6], acc[3], missed[3];

I think these arrays should be able to support all the available weapons, perhaps filling it with a value like "-1" for unsupported (or non hitscan) weapons. So, having something like this:

Code: Select all

float hitscan[WEP_COUNT], acc[WEP_COUNT], missed[WEP_COUNT];


You can be able to know how many hits you made with the minstanex, reading acc[WEP_MINSTANEX - WEP_FIRST]

In short, you have constants for identifying weapons, how many weapons you have in the system and you provided a method for identifying which weapons are hitscan. This should be enough for tracking and displaying stats in the client dynamically. What I mean with "dynamic" is: the minstanex or any new hitscan weapon should be automatically displayed in the client.

Anyway I think you made an awesome work, this patch is quite huge for being your first contribution.

Cheers

here is the latest patch: http://www.pastie.org/346376
i believe you have to patch in the qcsrc folder.
now the the scoreboard goes away when you do +showhitscanstats
its been changed to +showhitscanstats

I just can't use this patch, now I've spent a considerable time trying to fix it. Please try

Code: Select all

cd X/nexuiz/data/qcsrc/ && svn diff > stats.patch

If you are using cygwin you should install the svn package manually. Don't worry if you've modified another files in your working copy, thats easy to fix.

patch: **** malformed patch at line 433: float tempdb;

The patch is indeed broken. Looks like that pastebin site inserts empty lines somewhere. Try using a different site, like paste.debian.net.

I think svn has an ability to supress empty lines. Need to check again though.

This is a syntactically broken diff file, not a bug/feature in svn. I am sure that pastebin did it.

Doesnt matter if svn fixes it or not. That diff that i posted is broken. for some reason i cant get a proper diff with all the file changes

How are you making the diff anyway?

by doing svn diff < patch.diff

what does -u do?

Shows -3,+3 lines (by default) above,below the patched place. Makes it easier to identify which places were changed.

I got it working on my machine and created a patch

http://z.nexuizninjaz.com/mods/hitscanstats_r5311.patch

can someone test?


Edit: mkzelda and I both get:

patch: **** malformed patch at line 447: @@ -292,6 +294,14 @@

I can not understand all this trouble with patching anyway.

Invalid patch files and such. Well, first of all I would probably just upload the .patch file, like -z- did it (instead of using pastebins). And when I used to create or apply patches with tortoise svn, I never had problems either.

That's so weird.

http://pastebin.com/m653717ff

thats my newest update.
Green: i dont have a webserver and most file host servers are very spammy so i avoid them.

EDIT: sorry this patch isnt working.

Icarus wrote:http://pastebin.com/m653717ff

thats my newest update.
Green: i dont have a webserver and most file host servers are very spammy so i avoid them.

EDIT: sorry this patch isnt working.

use omploader, drop.io or filedropper...

Icarus wrote:Green: i dont have a webserver and most file host servers are very spammy so i avoid them.

I'll give you a user-account on planetnexuiz.de where you can put your files. I'll talk to you in IRC about it.
WIll be something like http://esteel.planetnexuiz.de/
e.g. http://icarus.planetnexuiz.de/ (when it's done)

Newest patch: http://paste.debian.net/26232/
hopefully this one isnt malformed

Icarus wrote:Newest patch: http://paste.debian.net/26232/
hopefully this one isnt malformed

It is not valid again, sorry.
PLEASE, get a proper program to create patches. Have a look here: http://ouns.nexuizninjaz.com/dev:progra ... troduction

Icarus, if you need some webspace to host your patches hit me up on pm or irc.

I guess hosting shouldn't be the problem in this case, because http://paste.debian.net/ allows you to upload the patch file as a file (no copy/paste needed), and also allows me to download it as a file again, so this pastebin is practically a text file hoster already. Plus, I've had no problems with other patches posted here on the board that were hosted on http://paste.debian.net/, they applied just fine.

I've fixed the patch changing the paths of modified files (from nexuiz directory now) and deleting blank lines.

http://pastebin.com/f3bd5c2f

For the next time Dib, again, u should generate the patch from the directory NexuizSVN/nexuiz.
Those blank lines... why are they in the patch?? Strange. They are present only after ~340 lines... maybe u add them by fault editing the patch in a text editor...

i dont know why i cant make a patch from nexuiz folder so heres another one from qcsrc folder.
http://sites.google.com/site/dibnexuiz/ ... racy-patch

this includes splash accuracy. you need the autoexec.cfg included with it for it to work.