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

java.lang.NegativeArraySizeException error for secondary indicators #56

Open
noeliaoses opened this issue Jul 12, 2018 · 3 comments
Open

Comments

@noeliaoses
Copy link

Hi,

The UR throws a java.lang.NegativeArraySizeException when there isn't at least one occurrence of the primary indicator. But it also throws the same error when no users in the secondary that are also in the primary (no cross-occurrences), as explained by Pat Ferrel in actionml's googlegroup. It would be great if the error messages were different for both cases and/or more informative.

Thank you very much for all the great work!!!
noelia

@pferrel
Copy link
Collaborator

pferrel commented Jul 12, 2018

This will only happen when there is no data or no cross-occurrences. Therefore you should not be using the events in question, it just wastes cycles. I suggest some kind of serious warning here.

To collect data for maybe 10000 users and have no cross-occurrences is a serious data problem. It may mean you are collecting in the wrong way or have encoded user-ids differently in the primary vs the secondary events or the event has no meaning WRT cross-occurrences. Every time we have seen this is was a too small sample or a data encoding error. This should not happen in real-world situations or the recommender will have NO data to use. It means that not one of the users has meaningful data for the event.

As I understand it this happens for you with some test sample, so the error is annoying when doing a proof of concept. My worry is that a non-fatal error will allow data problems or malformed data to not be noticed.

@pferrel
Copy link
Collaborator

pferrel commented Jul 12, 2018

At very least the error should be more informative, giving the event name and better message.

@wanxsb
Copy link

wanxsb commented Oct 20, 2018

Please decrease minEventsPerUser if you have make sure nothing wrong.

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

No branches or pull requests

3 participants