Changeset 1c0b0b for src/Graph/unittests/BoostGraphCreatorUnitTest.cpp

Timestamp:

Oct 23, 2022, 9:08:55 AM (3 years ago)

Author:

Frederik Heber <frederik.heber@…>

Branches:

Candidate_v1.7.0, stable

Children:

894040

Parents:

0daef6

git-author:

Frederik Heber <frederik.heber@…> (10/19/22 19:38:02)

git-committer:

Frederik Heber <frederik.heber@…> (10/23/22 09:08:55)

Message:

Graph6Writer::write_elementlist uses BFS from boundary atom.

• the elementlist is not stable as the set of atoms is arbitrary to any kind of permutation. However, the underlying bond graph is not, even though it may also have some symmetries.
• Therefore, we use a BFS from a non-hydrogen atom on the boundary.
• FIX: BoosGraphCreator gets vector of const atom*.

File:

• src/Graph/unittests/BoostGraphCreatorUnitTest.cpp (modified) (1 diff)

src/Graph/unittests/BoostGraphCreatorUnitTest.cpp

@@ -164 +164 @@
 void BoostGraphCreatorTest::createFromAtomsTest()
 {
-  std::vector<atom *> atoms;
+  std::vector<const atom *> atoms;
   std::copy(TestMolecule->begin(), TestMolecule->end(), std::back_inserter(atoms));
   BGCreator->createFromAtoms(atoms, BreadthFirstSearchGatherer::AlwaysTruePredicate);