sulis icon indicating copy to clipboard operation
sulis copied to clipboard

Identification try2

Open picobyte opened this issue 5 years ago • 4 comments

picobyte avatar Mar 09 '20 18:03 picobyte

I think you missed the level1 loot_lists? Or was that intentional? Not sure anymore, but added.

Several issues are addressed in latest pull, but some more changes will follow, these should probably include:

  • adjective3 for unidentified state = fixed
  • Accessories could have unidentified status, although it's adjective 2 not 3 - if we use three => fixed
  • multiple "unidentified" adjectives in save for items => fixed
  • weighted choice alternate behaviour
  • base item value for identification price and identification notification at merchant
  • merchant identify fraction configurable and saving => done
  • right click mouse menu identify option at merchant => actually already there.

picobyte avatar Mar 14 '20 08:03 picobyte

Hope all is well, life somewhat grinds to a halt here, in Europe. I've addressed most issues, not sure what's causes the weighted selection difference, and there are some open questions

picobyte avatar Mar 18 '20 20:03 picobyte

With the kids at home all day and still working full time, I am bit limited on free time at the moment.

I'll try to take a look at your recent changes soon. Hope you are doing well.

Grokmoo avatar Mar 20 '20 00:03 Grokmoo

That's good to hear, I'm ok. Don't worry about this pull request, priorities first stay safe and take care.

picobyte avatar Mar 20 '20 22:03 picobyte