8000 Expecting property name enclosed in double quotes · Issue #40 · allenai/kb · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Expecting property name enclosed in double quotes #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
lshowway opened this issue Dec 5, 2021 · 4 comments
Open

Expecting property name enclosed in double quotes #40

lshowway opened this issue Dec 5, 2021 · 4 comments

Comments

@lshowway
Copy link
lshowway commented Dec 5, 2021

when I run the example of How to embed sentences or sentence pairs programmatically, I got an error: json.decoder.JSONDecodeError: Expecting property name enclosed in double quotes: line 242 column 5 (char 8926) when loading config.json. I also load config.json solely, I get the same error.
@lshowway
Copy link
Author
lshowway commented Dec 5, 2021 •
edited
Loading

I have solved the problem by downloading archive_file = 'https://allennlp.s3-us-west-2.amazonaws.com/knowbert/models/knowbert_wiki_model.tar.gz' manually and removing the last comma. In addition, function _extract_config_from_archive in knowbert_utils.py is also needed to be modified.
So I recommend that the config.json should be modified.

@gpiat
Copy link
gpiat commented Dec 9, 2021

It's been a year and a half since the last commit on this project, and I don't believe the AllenAI team is actively maintaining it. I think the best way of helping others work around this issue would be for you to submit a pull request if you can.

@GeorgeVern
Copy link
GeorgeVern commented Dec 13, 2021 •
edited
Loading

I have removed the last comma from the config.json but now I get the error knowbert is not a registered name for Model. Any thoughts?

@gpiat
Copy link
gpiat commented Dec 14, 2021 •
edited
Loading

I believe this error means that the following line is not processed before an attempt at instanciating the model is made:
kb/knowbert.py line 776 : @Model.register("knowbert")
I assume you're using allennlp train, did you remember to launch your command with the --include-package kb.include_all option?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@lshowway @GeorgeVern @gpiat
0