The assignment dropped all changes that were made by molck

This fixes test_biounit.py on mac

This fixes test_complib.py on mac

This unit test has been observed to fail (EXTREMELY RARELY)
It tests the edge case of 3 atoms that are all on a line.
From an external coordinate system perspective, the dihedral
is not defined in this case.
From an internal coordinate perspective this should not matter.
The last line is the failing line. The expected behaviour is
that a rotation of x around t1 (The "linear" dihedral) should
propagate and affect t2 accordingly. Observed failure:
difference{4} between s.t2.GetAngle(){-2.3561945} and Real(3.14159265358979323846/4){0.785398185} exceeds 0.0001%

In the meantime we propose to use internal coordinates with caution and
open a task in JIRA for someone brave enough to deep dive into the code.

Xavier Robin authored 1 year ago and Xavier Robin committed 1 year ago

An overflow would happen when the residue contains unknown atoms that
are not in the compound, and can prevent bonds from being properly
created in ConnectAtomsOfResidue.

This reverts commit 9153648f.

The intention of this commit was to fix potential issues when operating on a
dihedral connecting 4 atoms where at least 3 atoms are exactly on a line.
Dihedrals are ill defined in this case but "expected behaviour"
(whatever that means) is tested as a unit test.
A fail was observed in two cases. First case that triggered commit
9153648f, second case that showed that the
problem is not solved. Therefore reverting to original state in order to find
different solution.

There are two use cases for this parameter: 1) decide whether a chain
mapping problem is actually solvable by naive enumeration and crash out if not
2) As a decision whether one should use naive enumeration or a heuristic
approach for chain mapping.

For the latter, n_max_naive meant the number of chains in the underlying
structure. Now its number of possible mappings for both cases.

- Make original structures available as attribute in mol.alg.Scorer object.
  Thats the actual input structures without Molck etc.
- Ensure unique interface definitions by lexicographically sorting the respective
  chain name lists
- Optionally dump additional alignment information in the compare-structures action.
  Explicitely dump residue numbers etc. of the aligned residues.

requested by Andriy. Computes all IPS metrics on a per-interface basis

requested by Andriy. Computes all ICS metrics on a per-interface basis

Coordinates are delta encoded. That means the first element corresponds
to the first coordinate and all the following elements are the
respective diffs. The first element is potentially much larger than
the rest and can cause excessive integer packing due to overflows.
This change stores that first element separately.

Not sure why this doesn't cause an error here, but the code is invalid.

The chain naming strategy can be read in the documentation