New Content - Implemented Jarlsberg Companions

Veracity

Veracity

Developer
Staff member
I am on my second Jarlsberg run and I like everything that has been added to support it. Thank you! But there is one thing that might be done to make it even better. The green sidebar has a little "no familiar" picture with the label "0 lbs." I think it would be nice if there was a picture of your current companion - not big, like on the charpane, but scaled down to the size of a familiar icon.

And perhaps KoLmafia could allow you to switch companions, if you know more than one skill and have the appropriate ingredient on hand by clicking on the image and selecting from a menu, just as we do for runs where you have a familiar. Can you switch from Egg Man to Radish Horse and back to Egg Man? I don't know; I've never had more than one companion skill.

api.php has this for me:

"jarlcompanion":"1"

Using that, we can recognize companion on log in. We could look at what happens when you cast a companion skill and deduct MP and ingredient and switch companion, if it succeeded.
 
Every time you switch companions it costs an ingredient. For that reason, I don't want to make it too convenient to switch, outside of using the normal skill-casting interface. (In my lazier runs, I pick one companion for the day and never change it. In speed runs, it will be part of my planning spreadsheet.)

Mafia knows what companion you currently have, both from parsing api.php and from parsing the charpane. I thought about making an ASH function to report your current companion (that's why the enum has String names), but no one asked for it.

Ingredients are not currently deducted for using the skills, which is something that needs fixing regardless of any other decisions.
 
Fair enough, regarding not switching outside of the skill casting interface.

I still think it would be nice to see the picture on the green side pane, though.
 
Veracity said:
I still think it would be nice to see the picture on the green side pane, though.
Click to expand...

This would be very nice indeed. I recommend using the icon for the associated skill since it is a reasonable size.


lostcalpolydude said:
I thought about making an ASH function to report your current companion (that's why the enum has String names), but no one asked for it.
Click to expand...

I would make use of that. I didn't ask for it because I assumed that was your next step so I was waiting. Please remember that the "Egg Man" skill summons an "Eggman". That's confusing, but significant if you add the feature.
 
All of this should be done in 11965.

Bale said:
I recommend using the icon for the associated skill since it is a reasonable size.
Click to expand...

Thanks for the suggestion.

I'm not quite happy with using FileUtilities directly from CompactSidePanel. The alternative seems to be making FollowerSomething.java somewhere for Jarlsberg, Boris, and future path followers, and I couldn't decide where that should go.
 
You must log in or register to reply here.
Back
Top