Disallow calling `del_component` with ComponentData arguments

[emma58]

Fixes #3435

Summary/Motivation:

I think silently deleting the containing component when we call del_component on a ComponentData is rude, and the post-processing in Fourier-Motzkin elimination was falling victim to this mistake. This PR adds a ValueError to del_component if it is given a ComponentData and revises the FME transformation post-processing routine.

Changes proposed in this PR:

• Sneaks in a documentation fix for ComponentData
• Adds error to del_component if the arguments is a ComponentData
• Revises FME post-processing routine to not misuse del_component and, in fact, to not add and delete as many components in general.

Legal Acknowledgement

By contributing to this software project, I have read the contribution guide and agree to the following terms and conditions for my contribution:

1. I agree my contributions are submitted under the BSD license.
2. I represent I am authorized to make the contributions and grant the license. If my employer has rights to intellectual property that includes these contributions, I represent that I have received permission to make contributions and grant the required license on behalf of that employer.

[emma58]

I agree with @michaelbynum that this is a bug and shouldn't have a deprecation path. Especially after I saw where it got hit in the codebase already... Assuming tests pass (I think they finally will?), this is ready for review again.

[mrmundt]

We are probably not going to aim to merge this before the patch release next week (because it's fairly intricate / could break some folks). We don't want to sneak this in without sufficient planning/warning.

[emma58]

This is ready to merge when tests pass.