Skip to content
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.

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

KBB file: numeric attribute value stored only as string #439

Open
amnonma opened this issue May 27, 2023 · 2 comments
Open

KBB file: numeric attribute value stored only as string #439

amnonma opened this issue May 27, 2023 · 2 comments
Labels
bug Something isn't working priority low Low priority

Comments

@amnonma
Copy link
Contributor

amnonma commented May 27, 2023

@dehilsterlexis
While the NLP++ KB can store numeric attribute values, the KBB file stores numbers only as strings.
Properly, string values should have double quotes on them, to distinguish from non-string attribute values like num and float.
At the very least, VT/NLP++ user needs to be warned that every value in a KBB file is a string.
While here, a way to store concept pointers as values should also be possible in a KBB file.

@amnonma amnonma added bug Something isn't working priority low Low priority labels May 27, 2023
@amnonma
Copy link
Contributor Author

amnonma commented May 27, 2023

There may be a long-standing bug in NLP++ KBs regarding string versus integer values.
Seems numbers can be stored as strings, but then must be fetched as integers.
Some kind of mess here.

@dehilsterlexis
Copy link
Contributor

@amnonma actually, that should be working. I thought I fixed that a while back. I will check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority low Low priority
Projects
None yet
Development

No branches or pull requests

2 participants