search

wukong-m2m / wukong-darjeeling

Darjeeling for WuKong
Other
20 stars 17 forks source link

If we use the same component in different page, they will be map twice #90

Closed wycc closed 10 years ago

wycc commented 10 years ago

We should not map the component multiple times.

wycc commented 10 years ago

a

In this example, the Threshold is used in two pages. It should be shown once only.

wycc commented 10 years ago

This is not displayed issue only. It will actually create unusable result since the component_index are incorrect.

wycc commented 10 years ago

Please merge the components before we call the mapper.

wycc commented 10 years ago

4fbbbb103403a27a72819c1ed1860229fa87a9e6

This patch merge the components before we iterate the whole list. It works for me now,

senatorjo commented 10 years ago

Further updated the algorithm here after new data structure, should be working after commit 11e29552ec54503939e2495542050982362ca002.