Skip to content

Conversation

@gwnoseworthy
Copy link

feature: Add filtering of low value items from shops and chests. flag…: -nti --no-trash-items

Replacing low value items in chests with their sell GP value
added sell_value and is_trash props to Item

created list of trash items using: https://docs.google.com/spreadsheets/d/1Cit5Xl_TCBPFI4q1NEVQhSYGH1wKY9st_1yee3lzCP8/edit#gid=174771622
Filtered out armors that were bad and had no bonus
Removed ["Charm Bangle", "Coin Toss", "FakeMustache"] just to have a couple relics out of the pool

Trash items are added to excluded items for shops
and low value chests are converted during shuffle_random

@gwnoseworthy
Copy link
Author

image

Saw this during a run through, WoB Narshe Wolf room.

@gwnoseworthy
Copy link
Author

image
Saw this after pundababa check.

@gwnoseworthy
Copy link
Author

The weird garlic symbol is caused by having a chest containing 0gp,

"Dirk", "MithrilKnife", "Guardian", "MithrilBlade", "RegalCutlass", "Crystal", "Ogre Nix", "Mithril Pike",
"Stout Spear", "Gold Lance", "Partisan", "Imperial", "Kodachi", "Hardened", "Ashura", "Kotetsu", "Forged",
"Aura", "Strato", "Poison Rod", "Punisher", "Gravity Rod", "MetalKnuckle", "Mithril Claw", "Kaiser",
"Flail", "Morning Star", "Full Moon", "Boomerang", "Rising Sun", "Cards", "Darts", "Chocobo Brsh", "DaVinci Brsh"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please keep the DaVinci Brsh, Poison Rod, Gravity Rod, Gold Lance, Partisan, and Ogre Nix -- they've all niche use

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Despite my feelings about Ogre Nix, I will make this change

@asilverthorn asilverthorn added the On Dev Merged into dev branch label Apr 8, 2023
@asilverthorn asilverthorn mentioned this pull request Apr 8, 2023
for chest in possible_chests:
if not chest.type == Chest.ITEM:
continue
item = Item(chest.contents, self.rom)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change to item = self.items.items[chest.contents] (otherwise there's a merge conflict with other Item changes)

elif chest.type == Chest.ITEM:
chest.contents = self.items.get_random()

if self.args.no_trash_chests:
Copy link
Collaborator

@asilverthorn asilverthorn Apr 11, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change only affects -ccsr # (where number is non-zero). I recommend making it apply for all other chest options as well by moving it into its own method called by mod.

@asilverthorn asilverthorn added Needs Changes The feature needs changes before being merged and removed On Dev Merged into dev branch labels Apr 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Changes The feature needs changes before being merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants