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

Group and sort API reference #2281

Closed
wants to merge 1 commit into from
Closed

Conversation

horpto
Copy link
Contributor

@horpto horpto commented Nov 30, 2018

Current API ref is awful. All is in one big heap and not sorted by anyhow.

  • I've found that ftmodel is not specified in the API list.

@menshikh-iv
Copy link
Contributor

@horpto

  1. fasttext wrapper shouldn't be in doc (because we have our own implementation, this wrapper is deprecated and will be removed in 4.0.0)
  2. From PoV of "grouping" all stuff, we need to cleanup & finish restyle API Ref #2102 (here we re-group all stuff, use plugins for auto-generate "rst boilerplates" and so on)

It will be pretty nice if you'll work on #2102 instead.

@horpto
Copy link
Contributor Author

horpto commented Dec 14, 2018

I'm closing this PR until issue #2102 will not be closed, then I'll reopen it.

@horpto horpto closed this Dec 14, 2018
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

Successfully merging this pull request may close these issues.

2 participants