search

SpenceKonde / AzzyAI

AzzyAI for Ragnarok Online
29 stars 29 forks source link

New Homun skills level cap #10

Open leeandreic opened 5 years ago

leeandreic commented 5 years ago

Please take a look if the AI can support new level caps of the homun S mainly. https://forums.warpportal.com/index.php?/topic/226517-kro-job-class-improvement-project-homunculus/

SpenceKonde commented 5 years ago

Are there also API changes? It sounds like there might be - or maybe just changes to the default AI?

IIRC there wasn't a way to determine the level of a skill that a homun had? At least it didn't work when I was working on it. Fiddling with the levels in H_SkillList.lua might be a good first step, though.

As it says on the official website and repository home, I no longer play RO, so I am not in a position to continue AI development. I am happy to merge pull requests if others are able to make the required changes, or to provide advise on this to someone who wants to fork and/or take over the AI. I do not have the time to take this on unpaid.

SpenceKonde commented 5 years ago

getSAtkSkill() might also need to be changed - though for testing you can just change the skill level variables in H_Config it looks like?

I wonder where the AzzyAIConfig code is? I'm pretty sure I still have it. That would need to be updated to support the higher skill levels.

If there aren't API changes, this might not be that bad - but if there are, it could be a big mess, esp considering the kRO devs' history of making everything worse when they try to improve stuff.

leeandreic commented 5 years ago

Praise God you replied. Haha I thought you wouldn't because it's hella emphasized you are out of the scene but to the point:

I am unsure regarding API changes. It could very well be because many commented your program is pretty much dead in the newer client as Gravity imposes interactivity of homun.

But my server is a privately owned yet uses your v1.551 program so I'm reaching out.

On the other hand, I did my best to read the H_SkillList.lua and did change Sera's. Because the update affected all Homun S I'm not sure about getSAtkSKill(), where would it be?

SpenceKonde commented 5 years ago

I thought it was gone for good.... but today I got annoyed by all the icons on my desktop, and started cleaning up, and realized that the source code for the config tool was sitting there the whole time, literally staring me in the face!

I've committed it to github.

-Spence

On Tue, Jan 29, 2019 at 1:51 AM cidimmacula notifications@github.com wrote:

Hi, I'm surprised to see replies here. I'm not sure if the Skill Level updates are official (I'd like to think they do, but I no longer play official) and I have actually gotten them to work by editing both H_SkillList.lua and the H_Config.lua to circumvent the Configurator. I have been trying to undertake the task of updating the AzzyAI Configurator at least as a first step before completing all the Skill Level Updates info as I realized there is a hard limiter on setting the skill levels on the Configurator.

Do you still have the source for AzzyAIConfig code for review? Or perhaps available for a pull request/clone so that it would not need to be rewritten from scratch?

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/SpenceKonde/AzzyAI/issues/10#issuecomment-458427408, or mute the thread https://github.com/notifications/unsubscribe-auth/AGdyWwEcZc2InCZOaAEWbQeQK3hkvYunks5vH--EgaJpZM4WQsCr .

--

Spence Konde Azzy’S Electronics

New products! Check them out at tindie.com/stores/DrAzzy GitHub: github.com/SpenceKonde ATTinyCore: Arduino support for almost every ATTiny microcontroller Contact: spencekonde@gmail.com