Itm 228 - validator updates for new api

[kaitlyn-sharo]

• No repeated index values in scenes[].
• All characters need vitals
• Verify that MOVE_TO_EVAC evac_id is in the state
• Ensure that none of the action_types in the action_mappings are also in restricted_actions
• Remove the restriction on setting character.visited
• Remove the restriction on requiring characters as long as persist_characters is True
• Update YAML (for, e.g., persist_characters, etc.)
• Allow all injury statuses.
• Allow unconscious patients to have avpu of “UNRESPONSIVE” or “PAIN”.
• Update allowed injury locations in sync with what we support in the simulator
• Add a flag to say if it's a training (sim) scenario
  • Forbid training-only supplies in eval scenarios (Blanket, Epi Pen, Vented Chest Seal)
• Added INFO message if there’s no configured final_scene

[nextcen-dgemoets]

Looks good-- and thank you for your patience as these rules change and change again.

I reinstated the "forbidding state in scene 0" rule. Sorry...there was some reason it was there but I can't for the life of me figure out what it was. Plus it requires them creating a dummy state in Scene 0, so....

I had a few comments and a couple of things that didn't work for me. Happy to discuss.

[nextcen-dgemoets]

Changes look great.
I tweaked the CLI documentation a bit, and allowed training supplies in eval mode as long as the quantity is zero.

[dereknop]

Enforces all of the rules you described 👍