[DOC] solves several small raised issues #266
Conversation
| @@ -273,6 +273,7 @@ class ITML_Supervised(_BaseITML, TransformerMixin): | |||
| be removed in 0.6.0. | |||
| num_constraints: int, optional | |||
| number of constraints to generate | |||
| (`20 * num_classes**2` constraints by default) | |||
There was a problem hiding this comment.
We may need to use double-backticks here to prevent the docs from interpreting this as a link target. Can you build the HTML docs with these changes to verify?
There was a problem hiding this comment.
I tried it locally and I see it works as expected, also there is a itml_supervised.fit(X, y, bounds=...) that does not uses double backticks in the code a few lines below.
Should I make a pull request for the documentation that I generated as well ?
The commits that update the documentation look a bit heavy for minor modifications and include some system-specific results.
|
Merged. Thanks! |
|
@RobinVogel: for next PRs, you can use keywords which will automatically close the issues when the PR is merged, see |
Solves #258, #261, #256.