ISISNeutronMuon / MDANSE

MDANSE: Molecular Dynamics Analysis for Neutron Scattering Experiments
https://www.isis.stfc.ac.uk/Pages/MDANSEproject.aspx
GNU General Public License v3.0
20 stars 4 forks source link

Check if the "weights" setting is being used correctly #374

Open ChiCheng45 opened 5 months ago

ChiCheng45 commented 5 months ago

Description of the error Some jobs have the weight setting available to be used such as the radius of gyrations. However, by looking at the code it is not being used at all.

In other cases it does not seem to be applied correctly for example in the eccentricity job. The weight factor is only applied to the moment of inertia result but it replaces the mass in the moment of inertia with the weight factor so that the weight factor means that it is not a moment of inertia anymore. It also does not follow the documentation in anyway.

Suggested fix Go through all jobs and try to make sure that it follows what is described in the documentation https://mdanse.readthedocs.io/en/protos/pages/parameters.html#weights and that applying the weighting makes sense or that if doesn't make sense to have weighting it should not be included.

Perhaps the weight setting should be applied for all cases where a mean of some sort is done e.g. RMSD should include the weight setting.

Jobs to check

Dynamics

Structure

Scattering

Thermodynamics

Infrared

Virutal Instruments