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

Zhangya/metrics formula #1488

Merged
merged 6 commits into from
Jul 27, 2021
Merged

Zhangya/metrics formula #1488

merged 6 commits into from
Jul 27, 2021

Conversation

YanZhangADS
Copy link
Collaborator

Description

The formula cannot be properly displayed

Related Issues

Checklist:

  • I have followed the contribution guidelines and code style for this project.
  • I have added tests covering my contributions.
  • I have updated the documentation accordingly.
  • This PR is being made to staging branch and not to main branch.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Comment on lines 168 to 173
"from recommenders.utils.timer import Timer\n",
"from recommenders.datasets import movielens\n",
"from recommenders.utils.notebook_utils import is_jupyter\n",
"from recommenders.datasets.spark_splitters import spark_random_split\n",
"from recommenders.evaluation.spark_evaluation import SparkRatingEvaluation, SparkRankingEvaluation, SparkDiversityEvaluation\n",
"from recommenders.utils.spark_utils import start_or_get_spark\n",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey Yan, can you please revert this change. We renamed reco_utils to recommenders, see this PR: #1485 this is why the test it is failing.

Also, you need to reinstall recommenders in your local machine: pip install -e .[all]

@miguelgfierro miguelgfierro merged commit b622952 into staging Jul 27, 2021
@miguelgfierro miguelgfierro deleted the zhangya/metrics_formula branch July 27, 2021 12:31
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