Simon Zhao

Resolve issue #2018

10

### Description This PR resolves the issue #2018 by commenting out the tests in [tests/ci/azureml_tests/test_groups.py](https://github.com/recommenders-team/recommenders/blob/simonz/tf-security-alert-issue-2018/tests/ci/azureml_tests/test_groups.py) to disable testing for [deeprec](https://github.com/recommenders-team/recommenders/blob/simonz/tf-security-alert-issue-2018/tests/unit/recommenders/models/test_deeprec_model.py). ### Related Issues * #2018 ### References * TensorFlow +...

[BUG] inconsistent dependency on Pymanopt

2

### Description In [setup.py](https://github.com/recommenders-team/recommenders/blob/b000b78ceb3cbe52a0200922f2b2412d830274af/setup.py#L90), pymanopt is required as a dependent package: ```python extras_require["experimental"] = [ # xlearn requires cmake to be pre-installed "xlearn==0.40a1", # VW C++ binary needs to be...

### Description fastai has breaking changes in v2, which makes [the code in recommenders](https://github.com/recommenders-team/recommenders/actions/runs/6168006071/job/16739851647#step:3:2847) incompatible with fastai v2. For example, [`CollabDataBunch`](https://github.com/fastai/fastai1/blob/15bc02c61673400178fbdaf3417485755c299561/fastai/collab.py#L50) is removed in [fastai v2](https://github.com/fastai/fastai/blob/master/fastai/collab.py). ### Expected behavior with...

[FEATURE] Switch scipy sparse matrix to scipy sparse array

2

### Description As mentioned in [the doc of `scipy.sparse`](https://docs.scipy.org/doc/scipy/reference/sparse.html), the **scipy sparse matrix** interface will be deprecated and replaced by the **scipy sparse array** interface. All code in Recommenders now...

[BUG] tensorflow-estimator is removed from tensorflow 2.16.1

5

### Description As mentioned in [TensorFlow's release note for version 2.16.1](https://github.com/tensorflow/tensorflow/releases/tag/v2.16.1), the breaking change that removes [tensorflow-estimator](https://pypi.org/project/tensorflow-estimator/) is complete in TensorFlow 2.16.1. All code using TF estimator in Recommenders needs...

[BUG] Migrate AzureML SDK from v1 to v2

1

### Description The AzureML SDK used in the testing for Recommenders is of [version v1](https://learn.microsoft.com/en-us/python/api/overview/azure/ml/install?view=azure-ml-py) (named `azureml-core`): https://github.com/recommenders-team/recommenders/blob/310370b2a0fd3366bce264b9b007ad6d6af330e0/.github/actions/azureml-test/action.yml#L75-L77 https://github.com/recommenders-team/recommenders/blob/310370b2a0fd3366bce264b9b007ad6d6af330e0/tests/ci/azureml_tests/submit_groupwise_azureml_pytest.py#L42-L50 Although v1 is still supported, It's better to migrate to [version...

### Description I saw lots of warnings from the result of pytest. Those warnings were not taken into considration because they are suppressed in the CICD testing workflow. Many warnings...

### Description The PR #2127 set `scipy2.2.1) that resolves the problem. ### In which platform does it happen? ### How do we replicate the issue? ### Expected behavior (i.e. solution)...

### Description This PR resolves the issue https://github.com/recommenders-team/recommenders/issues/2171 by updating the testing workflows to use the Azure VM managed identity for login. It's tested in the PR https://github.com/recommenders-team/recommenders/pull/2174 . -...

### Description This is just a test for using Azure VM managed identity for login instead of service principal. ### Related Issues ### References ### Checklist: - [ ] I...