Xonotic Forums
[SOLVED] Cannot get cl_simpleitems_postfix to work - Printable Version

+- Xonotic Forums (https://forums.xonotic.org)
+-- Forum: Support (https://forums.xonotic.org/forumdisplay.php?fid=3)
+--- Forum: Xonotic - Help & Troubleshooting (https://forums.xonotic.org/forumdisplay.php?fid=4)
+--- Thread: [SOLVED] Cannot get cl_simpleitems_postfix to work (/showthread.php?tid=5069)



Cannot get cl_simpleitems_postfix to work - sev - 09-07-2014

As far as I understand, cl_simpleitems_postfix allows me to load different models in place of the defaults.
For example, considering the following files:

model.md3 (default)
model_simple.iqm (included in release)
model_foo.iqm

Then cl_simpleitems_postfix "_foo" should cause model_foo.iqm to be loaded in place of model.md3.
However, Xonotic seemingly ignores the cvar and always loads the models with the "_simple" postfix.

I've attached a test case, which should replace the health models with red squares when you set
Code:
cl_simple_items 1
cl_simpleitems_postfix "_foo"

Does the test case work correctly for anyone?
Is my setup incorrect?
Is this a bug?

Any help is appreciated.


RE: Cannot get cl_simpleitems_postfix to work - Pendulla - 09-08-2014

Looking at deox's simple items pack, it comes to my mind that you should name your file: model_simple_foo. The game may be looking at the second sufix.


RE: Cannot get cl_simpleitems_postfix to work - sev - 09-08-2014

(09-08-2014, 01:32 AM)Pendulla Wrote: Looking at deox's simple items pack, it comes to my mind that you should name your file: model_simple_foo. The game may be looking at the second sufix.
I gave it a try, to no avail. According to the code, it should only add the given postfix to the name, without a "_simple" in the middle.
But I cannot find the cause of the failure to load the "_foo" models.

deox's pack rewrites the existing "_simple" skin files, which is the only method I know to work.
However, that causes a competition of resources, forcing players to handle file conflicts of multiple icon sets.


RE: Cannot get cl_simpleitems_postfix to work - Mr. Bougo - 09-08-2014

Yeah, the code is broken due to a typo. Commit 900a3c0 introduced the postfix cvar, but notice that it is wrongly named:
Code:
var string autocvr_cl_simpleitems_postfix = "_simple";
This should read "autocvar". This means that the QC variable doesn't point to the right cvar, hence the constant value.

I'm reporting this to the bug tracker, that's a good excuse to try and use gitlab Smile

EDIT: There it is, https://gitlab.com/xonotic/xonotic/issues/86


RE: Cannot get cl_simpleitems_postfix to work - deoxys - 09-08-2014

(09-08-2014, 01:57 AM)sev Wrote: deox's pack rewrites the existing "_simple" skin files, which is the only method I know to work.
However, that causes a competition of resources, forcing players to handle file conflicts of multiple icon sets.

yeah, i tested this with Mario when investigating alternatives, and found the cvar was rendered useless. So, I just overwrote the files.


RE: Cannot get cl_simpleitems_postfix to work - sev - 09-08-2014

(09-08-2014, 01:24 PM)Mr. Bougo Wrote: This should read "autocvar". This means that the QC variable doesn't point to the right cvar, hence the constant value.

Yes, that's it, thank you very much. With the "a" inserted, it works correctly. I should have noticed that myself, sorry for the extra work.
Since you asked for repository access, I assume you already made the fix, ready to be pushed. Otherwise, I'll give it a shot.


(09-08-2014, 02:06 PM)deoxys Wrote: yeah, i tested this with Mario when investigating alternatives, and found the cvar was rendered useless. So, I just overwrote the files.

With the fix, every set can use its own models, enabling players to easily switch between them.


RE: Cannot get cl_simpleitems_postfix to work - deoxys - 09-08-2014

i'm aware, good. This seems like a good time to bring up the problems smilecythe and some others have had, that simple items dont show up with r_showsurfaces 3 (i believe that was what it was).


RE: Cannot get cl_simpleitems_postfix to work - sev - 09-08-2014

(09-08-2014, 03:01 PM)deoxys Wrote: i'm aware, good. This seems like a good time to bring up the problems smilecythe and some others have had, that simple items dont show up with r_showsurfaces 3 (i believe that was what it was).

Well, r_showsurfaces replaces all shaders with single colors. If the simple model is just a square with a "deformVertexes autosprite" shader, you end up with a colored square.
Someone will have to make actual 3D models to make them work with r_showsurfaces.


RE: Cannot get cl_simpleitems_postfix to work - deoxys - 09-08-2014

haha no not happening that's ridiculous


RE: Cannot get cl_simpleitems_postfix to work - Mr. Bougo - 09-08-2014

(09-08-2014, 02:14 PM)sev Wrote: Yes, that's it, thank you very much. With the "a" inserted, it works correctly. I should have noticed that myself, sorry for the extra work.
Since you asked for repository access, I assume you already made the fix, ready to be pushed. Otherwise, I'll give it a shot.

No sweat, I noticed it as soon as I looked at the code. I didn't make the fix, so if you have repo access, feel free to push it. I say the first one of us who has time for that should do it and update this thread Smile

I'll try to remember tomorrow.

EDIT: As for r_showsurfaces, damn, that's a good point. I don't think much can be done about that! r_showsurfaces is originally meant to be a debugging feature...


RE: Cannot get cl_simpleitems_postfix to work - sev - 09-08-2014

(09-08-2014, 05:52 PM)Mr. Bougo Wrote: I didn't make the fix, so if you have repo access, feel free to push it. I say the first one of us who has time for that should do it and update this thread Smile

Done. At least I think so, I'm new to gitlab as well Big Grin


RE: Cannot get cl_simpleitems_postfix to work - deoxys - 09-08-2014

(09-08-2014, 05:52 PM)Mr. Bougo Wrote: EDIT: As for r_showsurfaces, damn, that's a good point. I don't think much can be done about that! r_showsurfaces is originally meant to be a debugging feature...

i know, but it's smilecythe's preference to set the graphics that way for competitive play for some insane reason, so i tried to provide for him Tongue


RE: Cannot get cl_simpleitems_postfix to work - tZork - 09-09-2014

Closed issue #86, thanks sev.