Skip to content

Commit

Permalink
Merge pull request #2075 from recommenders-team/daviddavo/less-concats
Browse files Browse the repository at this point in the history
Merged two concats into one
  • Loading branch information
miguelgfierro authored Mar 21, 2024
2 parents 3821655 + 8393344 commit 96fe3cf
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions recommenders/models/rlrmc/RLRMCdataset.py
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,7 @@ def _data_processing(self, train, validation=None, test=None, mean_center=True):
"""
# Data processing and reindexing code is adopted from https://github.com/Microsoft/Recommenders/blob/main/recommenders/models/ncf/dataset.py
# If validation dataset is None
df = train if validation is None else pd.concat([train, validation])
df = df if test is None else pd.concat([df, test])
df = pd.concat([train, validation, test])

# Reindex user and item index
if self.user_idx is None:
Expand Down

0 comments on commit 96fe3cf

Please sign in to comment.