I think I see what was wrong. I had to add specific tracking (god, it's so complicated lol) in the post allocation trait point calculation to also keep track of the max allowed Vis/Car traits and not exceed it in counting the total amount of possible TP to spend... Basically, since it saw it had VIS Traits in 4 slots it was thinking that equaled 8 Trait Points. However, it can only have a max of 3 VIS Traits. So I had to add in extra logic for that.
Seems to have fixed this issue.
Let me know if you find more!
Meeku
Level 75
Guardian of the Realm
Joined: 7/20/2018
Threads: 33
Posts: 557
Posted: 3/2/2021 at 1:41 AM
Post #173
Oh yay a lot of fixes happened :D
Its kinda odd now, i'm not sure if its a problem but I noticed that some traits are being listed twice, sometimes three times in the regene list.
Maybe the carried traits are still being worked on? i checked a pet with only carried traits and it seems they are all listed twice.
Just thought i'd point it out, in case it matters somehow.
Its kinda odd now, i'm not sure if its a problem but I noticed that some traits are being listed twice, sometimes three times in the regene list.
Maybe the carried traits are still being worked on? i checked a pet with only carried traits and it seems they are all listed twice.
Just thought i'd point it out, in case it matters somehow.
Looks like this is just a display error. It wouldn't have any effect once live as the database table wouldn't accept duplicate entries and even if it did, it wouldn't break anything with the Re-Gen process as essentially it all boils down to 1's or 0's. So multiple 1's is still just 1.
I'll see if I can clean up the preview page to purge out duplicate entries though.
Edited By Krinadon on 3/2/2021 at 11:11 AM.
Breakdown
Level 75
Warden of Umbra
Joined: 3/2/2018
Threads: 10
Posts: 111
Posted: 3/2/2021 at 5:28 PM
Post #175
I'm very excited for this revamp, do you know when the full release is going to be out, by any chance? I have two themed Nyt eggs in my inventory and I am desperate to put them in my hatchery.
I'm very excited for this revamp, do you know when the full release is going to be out, by any chance? I have two themed Nyt eggs in my inventory and I am desperate to put them in my hatchery.
My goal atm is to finalize the revamp after the Week of Love closes.
Breakdown
Level 75
Warden of Umbra
Joined: 3/2/2018
Threads: 10
Posts: 111
Posted: 3/2/2021 at 6:56 PM
Post #177
Thank you, sir! I appreciate it! Looking forward to it. :)
Amethystx
Level 75
Nature Walker
Joined: 11/16/2018
Threads: 11
Posts: 179
Posted: 3/3/2021 at 4:42 AM
Post #178
looks like the Total Trait Points aren't including Carried Traits, only vis and hidden are getting added in.
such as this one https://www.sylestia.com/view/pets/regen_preview.php?pet_id=2877597 which should have 12 points but is missing two,
and this one https://www.sylestia.com/view/pets/regen_preview.php?pet_id=5981539 which should have 16 and is missing four.
LuciDatum
Level 70
Ghost Writer
Joined: 4/2/2019
Threads: 44
Posts: 865
Posted: 3/3/2021 at 7:18 AM
Post #179
Carried traits don't add trait points on or take them off, so that first one should only have 10 points and the second one should only have 12, the math is correct.
Aerii
Level 75
Supernatural Shopaholic
Joined: 10/12/2018
Threads: 100
Posts: 3,107
Posted: 3/3/2021 at 10:10 AM
Post #180
Actually, it's hidden traits that don't have any point value. Visible traits are worth 2 points while carried are worth 1 point. Both pets have their points set up correctly. It may look like more but it's taking into account that Krin has changed it so carried can become hidden and vice versa if necessary.
Go to Page:
1, 2, 3... 17, 18, 19
Confirm Action
Are you sure you wish to delete this post?
Confirm Action
Are you sure you wish to restore this post?
Confirm Action
Are you sure you wish to report this post?
Go to Top
This Page loaded in 0.013 seconds.
Terms of Service | Privacy Policy | Contact Us | Credits | Job Opportunities
