R

> 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,...

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...

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

The feature branch for this is: https://github.com/picobyte/sulis/tree/identification

Sorry, I thought taxator and to taxate were English terms (In dutch we have taxateur, it's from Latin), appraiser is probably the better translation: he/she who is able to recognize...

Since the identification will be something of the merchants view, I'll work in a clean branch. I have a running version with most of the framework, but not the merchants...

> I'm not sure what you mean by "unidentified as ability". Sorry, I meant adjective. [`struct ItemSaveState`](https://github.com/Grokmoo/sulis/blob/e69fbd8ff43fad69c75808a2921183fc510f8842/sulis_module/src/inventory_builder.rs#L41) is where you suggest I add this, right? > In item_save_state, we definitely...

The identification changes are in pull request #175 Unidentified is no longer a bonus kind in the yaml. Unidentified is an adjective2. If unidentified is the first item's adjective2 roll,...

In Linux the default nr of open files can be read with `ulimit -n`, for me it is 1024, and can be set using `sudo ulimit -n 8192`. Mac seems...

Could be the user running the script can't write to /tmp (try touching a (new) file in /tmp), or a parent directory was not created in the script; a mkdir...