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

New message pops up when using Inference #390

Open
kordjamshidi opened this issue Sep 14, 2016 · 3 comments
Open

New message pops up when using Inference #390

kordjamshidi opened this issue Sep 14, 2016 · 3 comments
Assignees
Labels

Comments

@kordjamshidi
Copy link
Member

kordjamshidi commented Sep 14, 2016

I am running the EntityRelationApp$ and I see for all instances this warning is shown,

23:21:10 ERROR EntityRelationConstrainedClassifiers$LocConstrainedClassifier$:55 - Warning: Failed to find head

this was not the case before, does anyone made a change for logging? Does it mean the inference is not performed correctly?

@danyaljj
Copy link
Member

danyaljj commented Sep 14, 2016

As far as I know, this class hasn't been changed functionality-wise since May. Not sure abt the logger; you can check the history. What other strange behavior do you see? If the only issue is logger msgs, we can change its logging level.

@bhargav
Copy link
Contributor

bhargav commented Sep 14, 2016

There was one change in behavior made to ConstrainedClassifier. (I can find the specific commit).

When calling the .test() method on a ConstrainedClassifier, the test would run only on node instances that were accessed by the pathToHead constraint and not all testingIntances for the node. This was fixed so that the .test() method actually runs on all node.getAllTestingInstances. For node items that satisfy the constraint (via pathToHead, we build the InferenceCondition to get the result else we directly use the onClassifier.discreteValue.

You get the warnings here for those item that do not participate in the constraint represented by pathToHead.

@kordjamshidi
Copy link
Member Author

Ok, thanks, this was useful. I am going to re-check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants