search

DestinyItemManager / DIM

Destiny Item Manager
https://destinyitemmanager.com
MIT License
2.07k stars 643 forks source link

Expanding the "randomize loadout" feature #9358

Closed justintricate closed 1 year ago

justintricate commented 1 year ago

Proposed change

Inspired by Frostbolt's recent YouTube video, it'd be nice if the loadout randomizer could have some rules applied to it. Eg "disallow sunset weapons" for things like trials, or "require at least 1 primary weapon"

How does this fit into your workflow?

This change would be aimed at content creators, for challenges on stream, or YouTube videos. Could also be fun just for weekend trials with your own gang, a la the old league of legends "ultimate bravery" challenge

justintricate commented 1 year ago

Screenshot_20230403-125014

I believe this would be a great place to put some knobs that allow tweaking the randomize rules. I'd be happy to work on this if it's deemed A-OK by the DIM team. Could include things like "only void weapons" or the ability to exclude certain weapons/types of weapons, down the line

robojumper commented 1 year ago

Duplicate of #9000? Yes, we're into it.

bhollis commented 1 year ago

Also, check out the user guide for this feature: https://github.com/DestinyItemManager/DIM/wiki/Randomize-Loadout - you can already quite comprehensively choose what gets randomized.

justintricate commented 1 year ago

I wonder if it'd be worth more prominent documentation on this feature, somewhere on the tool itself? I have a sneaking suspicion the github wiki isn't going to be read by many, unfortunately.

justintricate commented 1 year ago

@bhollis beat me to the punch, good call :)

https://github.com/DestinyItemManager/DIM/pull/9362 addresses this, thanks Ben!

bhollis commented 1 year ago

There are lots of links to the user guide all over the app but yeah, nobody reads it.