search

opencog / benchmark

Benchmarking the AtomSpace, the pattern matcher and other OpenCog systems
GNU Affero General Public License v3.0
6 stars 9 forks source link

Parallel pattern matcher benchmarking #5

Closed vsbogd closed 6 years ago

vsbogd commented 6 years ago

Tuning to measure parallel pattern matcher performance.

ngeiswei commented 6 years ago

@vsbogd Could you as well update https://github.com/opencog/benchmark/blob/master/atomspace/atomspace/README.md to reflect the new path?

ngeiswei commented 6 years ago

That looks good to me, just update the README.md of the atomspace benchmark since you've touched that part.

linas commented 6 years ago

This pull req is OK by me, I guess.

Please be aware that the benchmark code here is incredibly ugly, filled with hacks, and is in need of some large rewrite/redesign. Some of the things it measures no longer exist (e.g. the distinction between the table and the atomspace is gone; they are essentially the same, now)

Also, I am not convinced that the way it counts time is correct. It's filled it with a lot of beginner errors; the person who created this had no experience in benchmarking.

vsbogd commented 6 years ago

Fixed documentation and added an issue https://github.com/opencog/benchmark/issues/6 to use some standard C++ benchmarking library to simplify adding benchmarks.