These are chat archives for BSData/warhammer-age-of-sigmar

26th
Apr 2018
mdtm1g14
@mdtm1g14
Apr 26 2018 08:04
Is it possible to automatically deselect something when it is hidden, I'm not too happy with how my cat is working with certain Enclaves specifying spells that must be used, is this the case in any other tome?
Iain Launchbury
@Mad-Spy
Apr 26 2018 08:07
nope, you can't do it
You're better off making them max 0 rather than trying to hide them if possible.
or min 1 if the Enclave is selected.
mdtm1g14
@mdtm1g14
Apr 26 2018 08:09
I tried that and it didn't seem to work too well, neither method is producing errors until you manually start to correct it, everything seems to throw errors when hidden things are selected
Iain Launchbury
@Mad-Spy
Apr 26 2018 08:11
it will depend what you're triggering them off. BS tends to be quirky about when it throws errors depending on what you base your modifiers on. Give an example and I'll take a look at how you've done it.
mdtm1g14
@mdtm1g14
Apr 26 2018 08:25
It should be implemented in my latest branch on the repo, I am updating the enclave in the allegiance but it wont cause errors on the Tidecaster's spell unless I start to change them
Iain Launchbury
@Mad-Spy
Apr 26 2018 08:30
is that a branch on your fork?
I don't see a branch for you on the main repo
I've invited you as a collaborator, so you should be able to create a branch on the main repo now.
mdtm1g14
@mdtm1g14
Apr 26 2018 09:05
You should be able to see it now
@Mad-Spy
Iain Launchbury
@Mad-Spy
Apr 26 2018 09:56
I've pushed my preferred version. You were over complicating it a bit. I've also fixed the fact the both the Khorne and Deepkin cats had the same main ID (I guess you copied the Khorne cat to create yours?)
Why do you have per-faction general entries?
mdtm1g14
@mdtm1g14
Apr 26 2018 10:00
It seemed the easiest way to have the units change to battleline for multiple characters based on keyword, I did copy it as a basis yeah, I'll have a look through what you changed
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:06
Why do you have duplicated Leader RSE entries? Leaders are always Leaders, so you don't need to dupe them.
They don't need (and shouldn't have) an Allies tag.
mdtm1g14
@mdtm1g14
Apr 26 2018 10:09
I was following the implementation guide (https://github.com/BSData/warhammer-age-of-sigmar/wiki/Implementation-Guidelines) that states there should be two RSEs for each unit, a standard and an ally one, it specifically mentions leaders
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:09
Yeah, sorry, that's a bit out of date.
mdtm1g14
@mdtm1g14
Apr 26 2018 10:10
Ahh no worries, apologies for being a bit behind/less competent, still getting used to this, was there anything else that stood out?
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:11
They don't need it any more since we changed how we're doing allies
No worries. I wrote those in a hurry a while back and then when RL got too busy and I had to step back they didn't get updated.
There are a bunch of formatting issues on abilities (where you've copied and pasted them from the PDF/epub) but they're not major.
mdtm1g14
@mdtm1g14
Apr 26 2018 10:15
I thought I had sorted them as I copied them across, I'll have another look through and correct any that I find
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:24
I'd move the Tides of Death main text into the table (enter the first field as "-").
Also, the text is missing for all of those except the first.
mdtm1g14
@mdtm1g14
Apr 26 2018 10:25
I had planned to do that, and just seen the missing text, not sure what happened there but its an easy fix at least
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:27
Probably switched the profile type? It will delete the data when you do.
mdtm1g14
@mdtm1g14
Apr 26 2018 10:33
Yeah probably, I've removed all the duplicate leaders (does that apply to all units?) and tidied up a few things, hopefully that should be the Idoneth almost done
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:34
you don't need "Allies" category entries. You will need a "Battleline" and "Other" entry for any thing that is Battleline (or optionally Battleline)
mdtm1g14
@mdtm1g14
Apr 26 2018 10:35
Ok I will get that sorted
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:35
so if you have an "Other" and "Other Allies" entry, you can delete the second, as long as the first will show correctly in Allies detachments.
OftKilted
@OftKilted
Apr 26 2018 10:35
@mdtm1g14 don’t forget to submit a request to join as a contributor to the AoS files
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:37
I already added him as a Collaborator
mdtm1g14
@mdtm1g14
Apr 26 2018 10:38
Does anything need an allies tag, I'm assuming not now?
Iain Launchbury
@Mad-Spy
Apr 26 2018 10:38
nope.
mdtm1g14
@mdtm1g14
Apr 26 2018 10:40
All sorted and extra entries removed
tekton
@tekton
Apr 26 2018 12:49
I got half way throughly changes for artefscts when I had to deal with something blowing up at one of my offices (not a euphemism either...). Once today calms down I’ll finish that off and get the PR up.
Still haven’t slept since then either...
OftKilted
@OftKilted
Apr 26 2018 13:38
@tekton Uuuf. Sounds like more fun than I want to have. (And don’t forget to get a little sleep...)
While many people are of the opinion that “I’ll sleep when I’m dead” you’re not completing the Death catalogs... so real sleep might be appropriate.
;)
tekton
@tekton
Apr 26 2018 13:55
My second shelf is full of Death models though...
Because a necromancer in a horde of 40 skeletons is hilarious to watch people try to deal with
OftKilted
@OftKilted
Apr 26 2018 14:56
Artillery sniping ...
tekton
@tekton
Apr 26 2018 15:13
Skeletons are ablative wounds that come back and back...
OftKilted
@OftKilted
Apr 26 2018 15:42
@tekton And necromancers are why we can’t have nice things... stupid dead/undead things ... ;)
(Necromancers are really just really delayed healers .... “I didn’t fail medical school for nothing ...”)
OftKilted
@OftKilted
Apr 26 2018 17:27
@alphalas & @Cupropituvanso I'm cleaning out old branches that have already had their commits confirmed. If you still need yours please let me know. I'll be deleting the already branches tomorrow as they appear to be no longer needed.