Skip to content

Fix Freebase entity prefix validation - #236

Open
appleweiping wants to merge 1 commit into
THUDM:mainfrom
appleweiping:fix/181-freebase-entity-prefix
Open

Fix Freebase entity prefix validation#236
appleweiping wants to merge 1 commit into
THUDM:mainfrom
appleweiping:fix/181-freebase-entity-prefix

Conversation

@appleweiping

Copy link
Copy Markdown

Fixes #181

Freebase IDs in the knowledge graph data use m. and g. prefixes. The current validation accepts f. and rejects g.

This updates get_relations and get_neighbors and adds unit tests for both entry points.

Validation

python -m unittest tests.test_knowledgegraph_entity_ids -v

The current std.json contains 274 m IDs and 3 g IDs, with no f IDs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug/Assistance] The wrong entity pattern in task knowledgegraph

1 participant