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

Multi-label organisation #159

Open
henrygouk opened this issue Dec 16, 2018 · 1 comment
Open

Multi-label organisation #159

henrygouk opened this issue Dec 16, 2018 · 1 comment

Comments

@henrygouk
Copy link
Contributor

I've noticed a few oddities in the multi-label prediction API:

  • There are multiple MultiLabelInstance classes: moa.core.MultiLabelInstance and com.yahoo.labs.samoa.instances.MultiLabelInstance. Should one of these be removed?
  • BasicMultiLabelLearner and BasicMultiLabelClassifier are in the moa.classifier.multitarget package, but the moa.classifiers.multilabel package also exists. Should they be moved? Is BasicMultiLabelClassifier even needed?
  • The classifiers in the moa.classifiers.multilabel package do not implemented the MultiLabelClassifier interface, so they do not show up in the GUI. Are these defunct? Is there some reason they don't implement this interface?
  • A number of multi-label methods are found in the moa.classifiers.rules.multilabel package, even though they are not rule-based methods. Should all the multi-label methods be moved to moa.classifiers.multilabel?
@abifet
Copy link
Collaborator

abifet commented Jan 26, 2019

@aosojnik What do you think?

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

2 participants