StDoodle's Custom Daily Deeds

TeKRunneR

New member
One more little bug: still uses show up in the list even when I don't have any left (and the parameter is set to 1).

Also, with v1.2 of the script, and mafia r8137, I'm getting "0 of 3" for VIP pool uses, even though I've used them all. I'm on the first day of a run, in case that matters.
 

StDoodle

Minion
One more little bug: still uses show up in the list even when I don't have any left (and the parameter is set to 1).

Have you tested this under 1.2? Still tracking was backwards in v1.1 (I reported uses left instead of used, der!).

Also, with v1.2 of the script, and mafia r8137, I'm getting "0 of 3" for VIP pool uses, even though I've used them all. I'm on the first day of a run, in case that matters.

Got Churl2 to help test this; it appears the counter doesn't increment after 3 uses for the day, without an exception being made for ascension; but it does reset to 0 at ascension (so, if you've used 3 pool buffs pre-gash, it will never increment day 1). I could be wrong, but I've posted it as a bug in case (Don't shoot me V & jason!)
 

zarqon

Well-known member
Since I don't think it was ever quite resolved:

Code:
int tot;
foreach dealy,amt in get_inventory() tot = tot + autosell_price(dealy)*amt;

Hehe.
 

fronobulax

Developer
Staff member
mafia's breakfast checks the mirror and gets the potion right now. mafia checks the tree but I think it just reports the presence of a present and does not actually get it.
 

AlbinoRhino

Active member
Yes. What I am loving about this script is the summary table inserted into the character sheet. I just thought there was some way to use the script settings to add those items, but I'm just not understanding how to do it.

A couple of other things I noticed. I have a character who has a Gravyskin Belt of the Sauceblob and the character sheet and the gCLI reports "8 of 5" reagents as summoned.

Also, every clan I belong to has a pool table and using the "list" mode in the gCLI correctly reports the uses. However, the character sheet table always says "You don't have access to a clan Pool Table."

The gCLI "clan" section seems to report everything correctly, but the character sheet "clan" section reports everything as "X of 0". For example, Ball pit uses, 1 of 0 on character sheet but 1 of 1 in the gCLI. All items in the clan section do this.

Is there some obvious thing that I am doing wrong ?
 

StDoodle

Minion
Yes. What I am loving about this script is the summary table inserted into the character sheet. I just thought there was some way to use the script settings to add those items, but I'm just not understanding how to do it.

There is a way to add custom tracking; try reading through "HELP" (ie call the script with HELP typed as a parameter). (I'd give more helpful advice, but honestly I've forgotten. ;) ). And if you love the charsheet mods, you'll enjoy what I'm planning to add over the next week or so (extensive configuration & such via the relay browser).

A couple of other things I noticed. I have a character who has a Gravyskin Belt of the Sauceblob and the character sheet and the gCLI reports "8 of 5" reagents as summoned.
I haven't worked on the script since the Nemesis stuff came out, so I'll need to go through and add in new summoning-limit checks.

Also, every clan I belong to has a pool table and using the "list" mode in the gCLI correctly reports the uses. However, the character sheet table always says "You don't have access to a clan Pool Table."

The gCLI "clan" section seems to report everything correctly, but the character sheet "clan" section reports everything as "X of 0". For example, Ball pit uses, 1 of 0 on character sheet but 1 of 1 in the gCLI. All items in the clan section do this.

Is there some obvious thing that I am doing wrong ?
The only "obvious" problem I can think of would be if you're clan-hopping to a clan-less state while viewing it that way (it only reports uses based on your current clan, just in case you get de-whitelisted). Sounds more likely that I have a bug in the relay browser portion though, and I'll look into that.

Edit ('cause I forgot to respond to earlier posts):

I'll probably add in something for the mirror (again, it didn't exist last I worked on the script). I dunno about the crimbo tree; mafia reports on it when you login, and it isn't really "daily" so much as "every once in a while," but I can see the upside of including it. For reference, some things will likely never be included (like tower items) since they don't really need a daily check, and mafia already has excellent built-in tools for handling it.
 
Last edited:

AlbinoRhino

Active member
I appreciate your response .. thanks. Here is a screenshot of the behavior I am getting. I deleted all of the data files and started over fresh, but this was still the result.

screenshot
 

StDoodle

Minion
I cannot reproduce this on my end. Can you do a couple of things for me?
1) Make sure you have the latest version dl'd. Since it appears you haven't started using these scripts until just recently, I doubt this is an issue; but in case you dl'd them earlier than a few weeks ago, please try this first.
2) Try to see if you can reproduce this behavior, making sure not to clan-hop at any point during testing.
3) If you can reproduce, please let me know what the last line listed on the gCLI says; it's possible that could help.

As far as I can tell, my script's relay function works exactly like calling main() with "LIST ALL" as parameters, so I have no clue what this could be. Please let me know if the problem persists.

Thanks!
 

AlbinoRhino

Active member
OK. I deleted both script files, as well as all of the "custd_" data files. I downloaded new copies from the first post of this thread. I downloaded charsheet.ash into the relay folder and customdaily.ash into the scripts folder. I did not change clans at any point. I called customdaily.ash from the Scripts menu and typed "setup" into the popup. I answered "Yes" to every prompt (as I did both previous times). The gCLI gave me - "Successfully ran "SETUP" mode." - after all of the prompts. In fact, that is the only thing printed to the gCLI. I still get the same behavior. (This is all with my softcore character, MeatBall.) I also get this behavior with the AlbinoRhino character. However, I tried this script with another character, AlbinoWino, and everything worked as expected. The only difference between the first two characters and the third one, which immediately pops to mind, is that the first two have telescopes while the third does not. Also the first two have bookshelves in their campgrounds and the third doesn't. In any event, the gCLI reporting is always correct. Perhaps I will just do without the character sheet enhancement, as it is optional anyway. If I can be of any further assistance, please let me know.
 

StDoodle

Minion
I would definitely like to get to the bottom of this bug. Unfortunately, it looks like it's going to be messy; we'll probably need to go back-and-forth a bit to track it down. Please PM me with your availability if you're up for helping me on this (or join the #kolmafia IRC channel & give me a shout, which would be easiest, probably).
 

AlbinoRhino

Active member
(or join the #kolmafia IRC channel & give me a shout, which would be easiest, probably).

How do I do that ? I couldn't find anything on the wiki about IRC. I tried "#kolmafia" from a web-based client but it only took me to an empty room. Any pointers ?

Edit: Disregard this. I went to the sourceforge page and figured it out.
 
Last edited:

AlbinoRhino

Active member
From lines 212 to 218 of the script :

Code:
int def_REAGENT() {
	return to_int(have_skill($skill[Advanced Cocktailcrafting]));
}

int def_GARNISH() {
	return to_int(have_skill($skill[Advanced Saucecrafting]));
}

I think those 2 are reversed ? I hadn't noticed it until I logged on to a character who has saucerafting and way of sauce, but no cocktail skills at all. So I am getting a counter for garnishes but none for reagents. I swapped the skill names around and everything is checking correctly now.

edit:
Another issue has turned up. The other night, you ended with this code for the clan hall matching :

Code:
string current_clan() {
	matcher mat = create_matcher("<b>Clan Hall</b></td></tr><tr><td style=\"padding: 5px; border: 1px solid " + get_property("defaultBorderColor") +";\"><center><table><tr><td><center><b>(.+?)</b>", visit_url("clan_hall.php"));
	if (find(mat)) {return group(mat,1); }
	return "";
}

I happened to run the script with "setup all" again and was surprised to see that the prompts relating to the clan were skipped and that clan information was not displayed on the character sheet or in the gCLI. After some tinkering I determined that when mafia is used to change the value for defaultBorderColor, not only does it use a hex color value, it includes an "\" before the color value in GLOBAL_prefs.txt. I assume this is for escaping the "#" whenever mafia uses the property. In any event, the matching fails and the script believes there is no clan ? At least, all the clan-related settings get set to "0" in the custd_ ... _settings.txt file. I used "set defaultBorderColor=blue" to change the property back to a text value and everything worked perfectly again. Apparently the matcher would have to account for the "\" in the GLOBAL_prefs.txt file for anyone who has changed the border color, as well as be able to match a text color value for those who haven't ?

However, if I run a successful setup, where the clan info is prompted for and collected, and then subsequently change the color, everything still works. Until "setup" is run again, and then all clan settings are set to "0".
 
Last edited:

StDoodle

Minion
Re: Reagants & cocktails: I thought I'd fixed that bug ages ago... but apparently, not everywhere.

Re: the other issue (clans): Yeah, I realized just after you left that changing that line would fix things from the relay browser, but break (some of) them from the CLI. If you change it back to "blue" it should work from the CLI again, but I have to do a bit more fiddling to get it to work on both. Don't worry about the escape character "\" as it's generally ignored in string matching.

For reference, I believe you're pretty close on what that change breaks; it will work fine if you have saved clan information that's matched, but any time it looks for new clan info (such as when re-running SETUP), it won't work from the CLI. Grrr for stupid bugs. ;)

I'll look at a quick-fix to these issues tonight and put them up.

Edit: updates enclosed. No time to test, so they'll be changed in the first post later.
 

Attachments

  • customdaily.ash
    83.4 KB · Views: 35
  • charsheet.ash
    359 bytes · Views: 29
Last edited:

AlbinoRhino

Active member
Re: Reagants & cocktails: I thought I'd fixed that bug ages ago... but apparently, not everywhere.

Re: the other issue (clans): Yeah, I realized just after you left that changing that line would fix things from the relay browser, but break (some of) them from the CLI. If you change it back to "blue" it should work from the CLI again, but I have to do a bit more fiddling to get it to work on both. Don't worry about the escape character "\" as it's generally ignored in string matching.

For reference, I believe you're pretty close on what that change breaks; it will work fine if you have saved clan information that's matched, but any time it looks for new clan info (such as when re-running SETUP), it won't work from the CLI. Grrr for stupid bugs. ;)

I'll look at a quick-fix to these issues tonight and put them up.

Edit: updates enclosed. No time to test, so they'll be changed in the first post later.

Yes, I eventually determined (after I had written the above) that only "blue" would work.

I have tried this new version many times - changing colors, changing clans, running setup over and over - with multiple characters ... and have been unable to confuse it. It all seems to be working beautifully with the new files. Well done !
 

StDoodle

Minion
Hey your scripts doesn't seem to know that i already cast Summon Crimbo Candy for that day

I don't have the skill, so I've never been able to test that one. Does anyone know if the pref "_candySummons" reflects the Summon Crimbo Candy skill?
 

StDoodle

Minion
PLEASE NOTE: This post is a formatting test only for script debugging (kthnx gby)

[p style="display: none !important;"]Hello[/p]
 
Top