Skip to content

Commit 3209de4

Browse files
authored
Git patch 1 (#6)
* doc: Update the mathematical formula of MF in API - correct the dimension and formula of MF constraint - modified a display error in plq_Ridge_Regressor.predict() * doc: fixed a notation error in MF example page
1 parent 59ab8a1 commit 3209de4

File tree

4 files changed

+11
-16
lines changed

4 files changed

+11
-16
lines changed

doc/source/examples/MF.ipynb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@
2525
" \\pmb{\\beta} \\in \\mathbb{R}^m\n",
2626
" }} \n",
2727
" \\left[\n",
28-
" \\sum_{(u,i)\\in \\Omega} C \\cdot \\text{PLQ}(r_{ui}, \\ \\mathbf{p}_u^T \\mathbf{q}_i + \\alpha_u + \\beta_i) \n",
28+
" \\sum_{(u,i)\\in \\Omega} C \\cdot \\phi(r_{ui}, \\ \\mathbf{p}_u^T \\mathbf{q}_i + \\alpha_u + \\beta_i) \n",
2929
" \\right] \n",
3030
" + \n",
3131
" \\left[ \n",
@@ -36,9 +36,9 @@
3636
"\n",
3737
"$$\n",
3838
" \\ \\text{ s.t. } \\ \n",
39-
" \\mathbf{A} \\begin{pmatrix} \\alpha_u \\\\ \\mathbf{p}_u \\end{pmatrix} + \\mathbf{b} \\geq \\mathbf{0},\\ \\forall u \\in [n]\n",
39+
" \\mathbf{A} \\begin{pmatrix} \\alpha_u \\\\ \\mathbf{p}_u \\end{pmatrix} + \\mathbf{b} \\geq \\mathbf{0},\\ u = 1,\\dots,n\n",
4040
" \\quad \\text{and} \\quad\n",
41-
" \\mathbf{A} \\begin{pmatrix} \\beta_i \\\\ \\mathbf{q}_i \\end{pmatrix} + \\mathbf{b} \\geq \\mathbf{0},\\ \\forall i \\in [m]\n",
41+
" \\mathbf{A} \\begin{pmatrix} \\beta_i \\\\ \\mathbf{q}_i \\end{pmatrix} + \\mathbf{b} \\geq \\mathbf{0},\\ i = 1,\\dots,m\n",
4242
"$$\n",
4343
"\n",
4444
"where\n",

doc/source/tutorials/ReHLine_MF.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,9 +26,9 @@ Considering a User-Item-Rating triplet dataset :math:`(u, i, r_{ui})` derived fr
2626
2727
.. math::
2828
\ \text{ s.t. } \
29-
\mathbf{A} \begin{pmatrix} \alpha_u \\ \mathbf{p}_u \end{pmatrix} + \mathbf{b} \geq \mathbf{0},\ \forall u \in [n]
29+
\mathbf{A} \begin{pmatrix} \alpha_u \\ \mathbf{p}_u \end{pmatrix} + \mathbf{b} \geq \mathbf{0},\ u = 1,\dots,n
3030
\quad \text{and} \quad
31-
\mathbf{A} \begin{pmatrix} \beta_i \\ \mathbf{q}_i \end{pmatrix} + \mathbf{b} \geq \mathbf{0},\ \forall i \in [m]
31+
\mathbf{A} \begin{pmatrix} \beta_i \\ \mathbf{q}_i \end{pmatrix} + \mathbf{b} \geq \mathbf{0},\ i = 1,\dots,m
3232
3333
where
3434

rehline/_mf_class.py

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -16,9 +16,9 @@ class plqMF_Ridge(_BaseReHLine, BaseEstimator):
1616
1717
.. math::
1818
\min_{\substack{
19-
\mathbf{P} \in \mathbb{R}^{n \times r}\
19+
\mathbf{P} \in \mathbb{R}^{n \times k}\
2020
\pmb{\alpha} \in \mathbb{R}^n \\
21-
\mathbf{Q} \in \mathbb{R}^{m \times r}\
21+
\mathbf{Q} \in \mathbb{R}^{m \times k}\
2222
\pmb{\beta} \in \mathbb{R}^m
2323
}}
2424
\left[
@@ -32,15 +32,9 @@ class plqMF_Ridge(_BaseReHLine, BaseEstimator):
3232
3333
.. math::
3434
\ \text{ s.t. } \
35-
\mathbf{A} \begin{bmatrix}
36-
\pmb{\alpha} & \mathbf{P}
37-
\end{bmatrix}^T +
38-
\mathbf{b}\mathbf{1}_{n}^T \geq \mathbf{0}
39-
\ \text{ and } \
40-
\mathbf{A} \begin{bmatrix}
41-
\pmb{\beta} & \mathbf{Q}
42-
\end{bmatrix}^T +
43-
\mathbf{b}\mathbf{1}_{m}^T \geq \mathbf{0}
35+
\mathbf{A} \begin{pmatrix} \alpha_u \\ \mathbf{p}_u \end{pmatrix} + \mathbf{b} \geq \mathbf{0},\ u = 1,\dots,n
36+
\quad \text{and} \quad
37+
\mathbf{A} \begin{pmatrix} \beta_i \\ \mathbf{q}_i \end{pmatrix} + \mathbf{b} \geq \mathbf{0},\ i = 1,\dots,m
4438
4539
The function supports various loss functions, including:
4640
- 'hinge', 'svm' or 'SVM'

rehline/_sklearn_mixin.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -442,6 +442,7 @@ def decision_function(self, X):
442442
def predict(self, X):
443443
"""
444444
Predict targets as the linear decision function.
445+
445446
Parameters
446447
----------
447448
X : ndarray of shape (n_samples, n_features)

0 commit comments

Comments
 (0)