Unused properties in input and output structs

[alexhroom]

• bayesResults.dreamOutput.modelOutput is unused - it is set to zero on initialisation and is never changed. - removed in Refactors build and results struct #369
• problemStruct.otherParams and problemStruct.otherLimits are not necessary - removed in Removes unused properties from the API #356
• For the Priors struct , only priors.priorNames and priors.priorValues are used the remaining properties e.g. priors.bulkIn etc. should be removed. Removes unused fields from priors struct #348

We should consider removing placeholders from users facing code (core calculation should not be touched) to avoid confusing the users since all these things will show up in the documentation API reference. All can be re-added as needed after the release

• QzShift - this is still present in the problemStruct which affects the Python code - removed in Removes unused properties from the API #356
• problemStruct.oilChiDataPresent, contrastClass.oilChiData and the oilWater projects - this show up in the docs which can be confusing - removed in Removes unused properties from the API #356
• Magnetic and MagneticDomains - these are very easy to add back but should be removed before release to avoid confusion - removed in Removes unused properties from the API #356
• problemStruct.repeatLayers not implemented - implemented in Implements repeat layers #383

[alexhroom]

@DrPaulSharp should this be closed now #383 is merged?

[DrPaulSharp]

Yes, it should.