From 2871cbe75615afb32cec1d66b2d65867a5dbbac5 Mon Sep 17 00:00:00 2001
From: marco <marco@5a81b35b-ba03-0410-adc8-b2c5c5119f08>
Date: Sun, 21 Mar 2010 17:38:18 +0000
Subject: [PATCH] added blurp to explain the idea behind the POV export unit
test
git-svn-id: https://dng.biozentrum.unibas.ch/svn/openstructure/trunk@1861 5a81b35b-ba03-0410-adc8-b2c5c5119f08
---
modules/gfx/tests/test_ent_pov_export.cc | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/modules/gfx/tests/test_ent_pov_export.cc b/modules/gfx/tests/test_ent_pov_export.cc
index 0c44aa273..450051dc7 100644
--- a/modules/gfx/tests/test_ent_pov_export.cc
+++ b/modules/gfx/tests/test_ent_pov_export.cc
@@ -72,6 +72,12 @@ bool compare_files(const String& test, const String& gold_standard)
return true;
}
+// The GfxView uses a std::map for efficient access to atoms. This however has
+// implications for the POV export. In general we can't assume that the atoms
+// are written in any particular order. That's why we first filter out all
+// cylinder and sphere objects and sort them before we compare them against
+// the gold standard. For the trace-based renderers this is no problem as we
+// always get cylinders and spheres sorted form N- to C-term.
bool compare_sphere_cyl_entries(const String& test,
const String& gold_standard)
{
--
GitLab