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.

Sign up for GitHub

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

Jump to bottom

Staging to master to add xlearn #973

Merged
merged 184 commits into from
Dec 20, 2019
Merged

Staging to master to add xlearn #973

merged 184 commits into from
Dec 20, 2019

Conversation

miguelgfierro
Copy link
Collaborator

Description

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.

yueguoguo and others added 30 commits August 30, 2019 08:07
NOTEBOOK: add skeleton of fm + xlearn notebook
c01a7fc
@yueguoguo
NOTEBOOK: add details about FM algorithm
26ba382
@yueguoguo
SCRIPT: add xlearn
bcb86c4
@yueguoguo
NOTEBOOK: add xlearn example
8a09ebf
@yueguoguo
NOTEBOOK: add fm implementation in xdeepfm
62a8120
@yueguoguo
NOTEBOOK: add FM explanation
5286ef7
@yueguoguo
NOTEBOOK: finish xlearn ffm notebook
86b8667
@yueguoguo
NOTEBOOK: add libffm converter
372d552
@yueguoguo
NOTEBOOK: minor update
82c7c33
@yueguoguo
NOTEBOOK: add text to precise explain FM
b069b82
@yueguoguo
NOTEBOOK: add reference links to algorithms
3d43516
@yueguoguo
TEST: install cmake in aml test
2184788
@yueguoguo
Merge branch 'staging' into le_xlearn
42002f1
@yueguoguo
SCRIPT: add cmake install
83a1c9f
@yueguoguo
NOTEBOOK: user Timer
288e1d4
@yueguoguo
TEST: remove subprocess
0712f8f
@yueguoguo
SCRIPT: fix issue
3021bc7
@yueguoguo
SCRIPT: conda install cmake
9375621
@yueguoguo
Merge branch 'staging' into le_xlearn
f47bed5
@yueguoguo
DOCKER: add cmake install
5d80dd1
@yueguoguo
DOCKER: add cmake
4e85c3a
@yueguoguo
TEST: use custom docker
1dcd6c7
@yueguoguo
TEST: add customer docker
bc7fcd2
@yueguoguo
TEST: remove irrelevant codes
4f3d2b3
@yueguoguo
TEST: add test argument
e0bc2e4
@yueguoguo
Merge branch 'staging' into le_xlearn
ca42414
@yueguoguo
DOCKER: update cmake
518b05b
@yueguoguo
DOCKER: update
fd511d4
@yueguoguo
NOTEBOOK: update notebook with parameter
9323fa9
@yueguoguo
TEST: add xlearn import just for testing
0e4e063
Leavingseason and others added 28 commits December 13, 2019 11:16
@Leavingseason
Update from MSRA: add sequential recommender models to deeprec
fb8b777
@Leavingseason
update notebook
ba8aa10
@elogicaadith
Fix for issue #970 - model directory obtained by using os.path.join
c4ba08f
@Leavingseason
correct bugs; update notebook; add integration test
28e4a7e
@Leavingseason
update notebook
bf3e2bb
@miguelgfierro
Merge pull request #1011 from elogicaadith/staging
a53e576 
Fix for issue #970
@Leavingseason
update notebook: add CUP time
d8616e3
@miguelgfierro
Merge pull request #1010 from microsoft/sequentialrecsys
d64ebb2 
Sequentialrecsys
@miguelgfierro
docs
0434d1b
@miguelgfierro
docs
c539d1c
@miguelgfierro
docs
1bab011
@miguelgfierro
docs
0eaa4b6
@miguelgfierro
docs
996d290
@miguelgfierro
Update sequential_recsys_amazondataset.ipynb
9cd26ab
@Leavingseason
correct misspells
df7d3b0
@Leavingseason
correct misspells
28b9c4b
@miguelgfierro
revert to previous yamls
9812309
@miguelgfierro
🐛
8ddde5b
@miguelgfierro
🐛
2e50fc2
@miguelgfierro
Merge pull request #1015 from microsoft/miguel/docs_sequential
f674ebf 
Miguel/docs sequential
@miguelgfierro
Merge branch 'staging' into provisional_fix
07ba7b4
@miguelgfierro
Merge branch 'master' into staging
a1d2c5b
@miguelgfierro
Merge branch 'staging' into provisional_fix
fe2e3ab
@Leavingseason
update model save path string
232dc82
@miguelgfierro
Merge branch 'staging' into msra/sequential
c09a882
@miguelgfierro
Merge pull request #1019 from microsoft/provisional_fix
68aba5e 
revert to previous yamls
@miguelgfierro
Merge branch 'staging' into msra/sequential
99f7276
@miguelgfierro
Merge pull request #1017 from microsoft/msra/sequential
919b99b 
correct misspells
gramhagen
gramhagen approved these changes Dec 20, 2019
View reviewed changes
Copy link
Collaborator

@gramhagen gramhagen left a comment

Choose a reason for hiding this comment

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

looks good

@miguelgfierro miguelgfierro merged commit 9b6c065 into master Dec 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dciborow dciborow dciborow left review comments

@loomlike loomlike loomlike approved these changes

@anargyri anargyri anargyri approved these changes

@gramhagen gramhagen gramhagen approved these changes

@roalexan roalexan Awaiting requested review from roalexan

@yueguoguo yueguoguo Awaiting requested review from yueguoguo

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@miguelgfierro @yueguoguo @anargyri @gramhagen @dciborow @loomlike @Chris113113 @elogicaadith @Leavingseason