[DEV-8143] Prep for neutral

[charlta]

Description

Covers a lot of ground and fixes a heap of issues that were hidden by poor testing practices and duplicated (but not consistent) Hypothesis use.

Checklist

If any of these are not applicable, strikethrough the line ~like this~. Do not delete it!. Let the reviewer decide if you should have done it.

Code

• I have performed a self review of my own code (including checking issues raised when creating the PR).
• I have added/updated unit tests for these changes, and if not I have explained why they are not necessary.
• I have commented my code in any hard-to-understand or hacky areas.
• I have handled all new warnings generated by the compiler or IDE.
• I have rebased onto the target branch (usually main).

Security

When developing applications, use following guidelines for information security considerations:

• Access to applications should be protected with security keys/tokens or usernames and passwords;
• All sessions are encrypted if possible;
• All application input is sanitised before being acted on (ie SQL statements, etc);
• Log messages, and especially client-facing ones, must be handled securely and must not leak credentials information (internal URLs, passwords, tokens).

• I have considered if this change impacts information security and made sure those impacts are handled.

Documentation

• I have updated the changelog.
• I have updated any documentation required for these changes.

Breaking Changes

• I have considered if this is a breaking change and will communicate it with other team members by posting it on the Slack breaking-changes channel.

• Updated SetDirection to correctly use the @singledispatchmethod registration run instead of the old run_terminal. Simply replace your run_terminal
  call with run.
• EquipmentTreeBuilder.roots is now a dict keyed by the start_item rather than a Generator. This allows better lookup of root items when you are looking
  for an explicit items tree.
• Renamed PanDemandResponseFunction constructor argument appliances -> appliance to match the class property.
• Deprecated the following side-hustle shadowy names for PowerSystemResource.asset_info, just use asset_info directly:
  • Conductor.wire_info
  • CurrentTransformer.current_transformer_info
  • PotentialTransformer.potential_transformer_info
  • PowerTransformer.power_transformer_info
  • ProtectionRelayFunction.relay_info
  • ShuntCompensator.shunt_compensator_info
  • Switch.switch_info

[ryanjcoleman]

Task linked: DEV-8143 Python SDK prep work for neutral networks

[chestm007]

nit: is it worth leaving a comment as to why this was done?

[charlta]

It certainly is. I clearly didn't add anything back when I readded this after my rebase lost it.

Nice side effect is I have now put a lot more detail into my message than I originally had.

[chestm007]

Lgtm!