These are chat archives for Spring-Chobby/Chobby

8th
Aug 2016
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:28
@GoogleFrog as I suspected, CUS inside zk is customized enough so that you can't simply drop it into something else. You guys should make upstream pull requests... seriously.
GoogleFrog
@GoogleFrog
Aug 08 2016 04:35
upstream has a bad format
and I copied the most recent version from BAR
which is svn so I'm not touching it
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:35
how is cus differently formatted?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:35
you probably can't drop it in because I changed the format of the incoming data
it should be indexed by unitDefID instead of unitDefName
at least, perhaps other things too
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:36
Have you talked to gajop about it?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:36
I think so
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:36
So here is what I don't understand
if you have the utilities, the gadget, the shaders....
is a special gadgethandler required? and if so... wtf?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:37
probably not
but some of the parameters in the gadget callins might be reversed in ZK
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:37
yet it errors all over the place
for example in the gadget, on line 555 maps returns nothing
mats*
Thats fairly fundamental
GoogleFrog
@GoogleFrog
Aug 08 2016 04:38
I know less than you at the moment
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:38
The reason I'm testy about it is because there are a lot of things that don't work properly without a functioning UCS.
CUS*
And apparently the people who know what's up won't put all their groceries into one bag
GoogleFrog
@GoogleFrog
Aug 08 2016 04:39
my extended knowledge is missing because I don't have spring or any local clones here
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:40
here is another issue that maybe you know more about...
LUPS in 103 for me is ... screwy
GoogleFrog
@GoogleFrog
Aug 08 2016 04:40
mats is probably just in the wrong place
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:40
I updated LUPS from sauce
Yet the included nanospray doesn't work for example
I can copy the one from zk which works, but I can see enemy spray
So did you guys edit lups too?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:41
why should source work? Afaik nobody uses it
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:41
... wtf?
why does it exist then?
why would source work? because source should be where everyone gets it to implement it in their game
GoogleFrog
@GoogleFrog
Aug 08 2016 04:42
my theory is that source is not updated
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:42
next I'mma find out that chili source is horribly out of date
GoogleFrog
@GoogleFrog
Aug 08 2016 04:42
the LOS changes for 103 also involved some broken compatibility for LOS related lua callouts iirc
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:43
If I were to pull lups from ZK
GoogleFrog
@GoogleFrog
Aug 08 2016 04:43
I recall poking around in this type of unsynced gadget to fix the LOS stuff
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:43
what would I need?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:43
idk
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:43
just lups folder and gadgets?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:43
there is a lups widget too I think
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:43
Yeah, no one seems to know, and that is a SERIOUS problem :-(
GoogleFrog
@GoogleFrog
Aug 08 2016 04:43
and config files for unit lups
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:43
yeah I have all that
but ok so dropping in should theoretically work then
GoogleFrog
@GoogleFrog
Aug 08 2016 04:44
lups works in ZK and if it doesn't I methodically plod through the errors
but don't keep an overall design in my head between fixing sessions
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:44
Why not start another lups repo that works?
if JK cbf to do it properly, then you could?
I mean, that's what I would do
but it would eb useless for me to try because I wouldn't be able top fix issues that came up, however you're actively doign that azlready so...
GoogleFrog
@GoogleFrog
Aug 08 2016 04:45
complicated git structures are complicated. I don't want to use them. So the repos would not be linked in any systematic way
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:45
I'm not being an asshole, I'm asking it as a genuine question because I bet you I'm not the only one having issues
They don't need to be linked
just fork JK's repo, put your stuff in, and announce on forums that because jk doesn't update it, you put your own repo up with fixed stuff and ppl can use that
simple and solved
GoogleFrog
@GoogleFrog
Aug 08 2016 04:46
then it gets outdated
and I think that is effectively what gajop has done
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:47
Nonsense, simply copy changes that you make when fixing in zk to your repo
GoogleFrog
@GoogleFrog
Aug 08 2016 04:47
also your chili is probably very out of date
but so is the ZK one
but latest chili doesn't work anyway
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:48
my chili is direct from source
what the hell? it's out of date?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:48
recently?
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:48
Umm march I think?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:48
then it will be in-date. It just won't work.
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:48
My chili in evo works
Although the only thing I use it for is selections
because there are no normal build menu widgets unfortunately
I tried to implement KBMenu
but it seems to be buggy
GoogleFrog
@GoogleFrog
Aug 08 2016 04:49
well then you might not notice. Chobby chili is latest and proved to have issues with the features jK added since ZK chili
KBMenu is crazy
Integral is pretty standard I think
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:49
It's a great idea
integral sucks for one reason
It requires epicmenu
two reasons
it requires a config for build options
GoogleFrog
@GoogleFrog
Aug 08 2016 04:50
not if you poke around in it
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:50
instead of just generating positions on the fly
GoogleFrog
@GoogleFrog
Aug 08 2016 04:50
everything in ZK will require epic menu and a config because its the UI options and positioning framework
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:51
Epicmenu has no source repo
making it dangerous to use
GoogleFrog
@GoogleFrog
Aug 08 2016 04:51
I don't think you should use it
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:51
I don't want to
evo doesn't need it
I wish I didn't have to sue default build menu, but sadly
ok copying lups from zk
lets see what happens
Thanks for the info btw
GoogleFrog
@GoogleFrog
Aug 08 2016 04:53
it'll probably crash on a missing WG or GG. Perhaps a customParam which must exist
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:54
why?
why no fault tolerance?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:54
or the basecontent gadget handler may be out of date and not include a required callin
faults are fixed when they crash
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:55
No... fault tolerance
GoogleFrog
@GoogleFrog
Aug 08 2016 04:55
I didn't write lups so most of my changes are to add fault tolerance whenever something comes up
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:55
someThing = value or "Default"
things like that
GoogleFrog
@GoogleFrog
Aug 08 2016 04:55
nil checks
I have never added the nil checks for leaving ZK because its never left ZK
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:56
I don't mean to dump on you, but it's as if no one in spring ever accounts for anything outside of the ideal case scenario
GoogleFrog
@GoogleFrog
Aug 08 2016 04:56
I like defensive coding
check for nil everywhere
we talked about this with chobby and gajop does not like it
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:57
Well, he's wrong
by every single coding standard out there
check for nill and issue warning
yes it is more work, but it makes your code FAR more robust
he doesn';t liek it because "code bloat"?
GoogleFrog
@GoogleFrog
Aug 08 2016 04:58
I think he wants to know about errors
ForbodingAngel
@ForbodingAngel
Aug 08 2016 04:59
that's why you issue warnings
btw lups works
so does nanospry
thank you
GoogleFrog
@GoogleFrog
Aug 08 2016 04:59
a good 95% of my commits to this jK stuff will have been nil checks
lups, cus, chili
which files did you use?
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:00
Any chance I could get you to help me fix CUS in evo? I made it work by frankensteining some stuff from source, zk and bar together
but it spams warnings all over the place
the files I used were
GoogleFrog
@GoogleFrog
Aug 08 2016 05:01
what warnings?
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:01
incorrect material stuff, sec I'll show you a screenshot
GoogleFrog
@GoogleFrog
Aug 08 2016 05:01
no idea then
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:01
lemmie merge my lups branch to master real quick so I don't lose this
GoogleFrog
@GoogleFrog
Aug 08 2016 05:01
that is engine or gsl stuff
to use the ZK cus you make your cus config return a table of unitDefIDs
and maybe you should use the ZK materials?
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:03
That's what I was doing
btw did oyu see my PR?
my PR to ZK I mean
it catches basically everything that currently exists
Tested and working on my end, fwiw
GoogleFrog
@GoogleFrog
Aug 08 2016 05:05
idk if it does
because the cus has some automatic tree detection
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:05
It does :-) Trust me on this
GoogleFrog
@GoogleFrog
Aug 08 2016 05:05
I'll have to look at it later
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:05
It also catches engine trees
GoogleFrog
@GoogleFrog
Aug 08 2016 05:06
it might mess up other things
bbl
ForbodingAngel
@ForbodingAngel
Aug 08 2016 05:06
Okies :-)
thank you!
Анархид
@Anarchid
Aug 08 2016 11:46
@ForbodingAngel isn't springfeatures on github?
i think there's a typo in baobab that makes it become written as baobob but i suspect that's just how it's in springfeatures
ForbodingAngel
@ForbodingAngel
Aug 08 2016 20:25
That's a typo directly from beherith then @Anarchid
thanks for the heads up
nope
in spring features it's baobab
ForbodingAngel
@ForbodingAngel
Aug 08 2016 23:38
The typo is my fault in that case in the PR