eaburns / eaburns/search

Merge the closed list and hash table classes.

Open
#7 0 comments 0 reactions 0 assignees View on GitHub
housekeeping
Dominant language
C++
Stars
34
Forks
22
PR merge metrics
No merged PRs in 30d

Description

ClosedList needs reference to domain to call dom.hash, but HashTable uses Ops::hash. We should be able to handle both cases in one implementation.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by locating the ClosedList and HashTable classes, then inspect how ClosedList calls dom.hash and how HashTable calls Ops::hash. The work is complete when both cases are handled by one implementation without requiring ClosedList to reference the domain separately.

Written by the indexing model from the issue text.

Assessment

Tech stack
cpp
Domain
data
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.