[Performance] Avoid doing more work than needed in ForwardTranslator

Issue overview

While working on https://github.com/NREL/OpenStudio/pull/4700 I realized that my ZoneMixing object goes FOUR times though the translateZoneMixing in the case the translateZoneMixing returns boost::none. This is mostly because it's added to the list of iddObjectsToTranslate and because of that

https://github.com/NREL/OpenStudio/blob/develop/src/energyplus/ForwardTranslator.cpp#L3247-L3262

Current Behavior

We try to translate my object four times

Expected Behavior

Once is enough!

Possible Solution

Clean up the iddObjectsToTranslate, and probably eliminate that old block of code. Do that after the release though, we can't risk breaking stuff now.

Details

Environment

Some additional details about your environment for this issue (if relevant):

• Platform (Operating system, version): all
• Version of OpenStudio (if using an intermediate build, include SHA): 3.5.0-alpha

Context

Start by reading src/energyplus/ForwardTranslator.cpp around lines 3247-3262 and trace how iddObjectsToTranslate handles a ZoneMixing translation that returns boost::none. Confirm why the object is revisited four times, then make the translation path process it once while preserving existing behavior. The issue is done when the repeated translation no longer occurs.