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 main: LF announcement and GRU update #2006

Merged
merged 38 commits into from
Oct 10, 2023
Merged

Staging to main: LF announcement and GRU update #2006

merged 38 commits into from
Oct 10, 2023

Conversation

miguelgfierro
Copy link
Collaborator

Description

Related Issues

References

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.

miguelgfierro and others added 25 commits September 27, 2023 09:09
@miguelgfierro
Replacing references GRU4Rec to GRU
42e6982 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Replacing references GRU4Rec to GRU
b5f0f3b 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Replacing references GRU4Rec in config files
4ba66e8 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Update references
efd001a 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Merge pull request #2000 from recommenders-team/gru_fix
48ebb26 
Replace GRU4Rec references
@miguelgfierro
datasets with Sgit st!
3bb666c 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Added init to fix import error
dbbbb8b 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
update repo url
92eb223 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
ignore xlearn
3738f03 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Add init files
0ce2c79 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
update info from MS
011153d 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
update info from MS
d1fb6dc 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
📝
dc7f6a7 
Signed-off-by: miguelgfierro <[email protected]>
@anargyri @miguelgfierro
Minor syntax in README
457312d 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
📝
ff4634d 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Announcement LF
ccb7b75 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Update email
bfe470c 
Signed-off-by: miguelgfierro <[email protected]>
@anargyri @miguelgfierro
Update README.md
b7ee345 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Merge branch 'staging' into lf
33104a1
@miguelgfierro
security
4a76743 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
license and contribution notice
dc459bb 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
update author link
6de8ce9 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Add new code of conduct from LF
63b352b 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Merge branch 'lf' of github.com:recommenders-team/recommenders into lf
89cbebb
@miguelgfierro
Merge pull request #2003 from recommenders-team/lf
3643818 
Announcement LF
wutaomsft
wutaomsft previously requested changes Oct 3, 2023
View reviewed changes
NEWS.md Outdated
# What's New

## Update October 3, 2023

We are pleased to announced that we have joined the [Linux Foundation of AI and Data](https://lfaidata.foundation/) (LF AI & Data). With this move, our objective is to keep building an ecosystem to sustain open source innovation and collaboration in recommendation systems.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think it is important to note prominently in the home page that that this repo used to be "Microsoft Recommenders", so people are sure that they don't get redirected by malware, etc.

Suggest to change the Update to be: "We are pleased to announce that this repository (formerly known as Microsoft Recommenders, https://github.com/microsoft/recommenders)", has joined the Linux Foundation of AI and Data (LF AI & Data)! The new organization, recommenders-team, reflects this change.

We hope this move makes it easy for anyone to contribute! Our objective continues to be building an ecosystem and a community to sustain open source innovations and collaborations in recommendation systems. "

Copy link
Collaborator

Choose a reason for hiding this comment

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

It may be good to add a few words about the active work going on at the repo, e.g., support of newer version of python, etc.

Copy link
Collaborator

Choose a reason for hiding this comment

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

@miguelgfierro did you see these comments?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@wutaomsft new PR: #2008 please review

README.md Outdated

We moved to a new organization! Now to access the repo, instead of going to https://github.com/microsoft/recommenders, you need to go to https://github.com/recommenders-team/recommenders. The old URL will still resolve to the new one, but we recommend that you update your bookmarks.
We are pleased to announced that we have joined the [Linux Foundation of AI and Data](https://lfaidata.foundation/) (LF AI & Data). With this move, our objective is to keep building an ecosystem to sustain open source innovation and collaboration in recommendation systems.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same comment as above.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

PR here: #2008

@miguelgfierro
Copy link
Collaborator Author

@anargyri @wutaomsft @loomlike @SimonYansenZhao @yueguoguo the official announcement will be Tuesday, 10/10 at 9:00 AM PST.

There will be a blog post from MS and another from the LF

@yueguoguo
Copy link
Collaborator

Thanks for driving this @miguelgfierro and team!

miguelgfierro and others added 10 commits October 5, 2023 21:52
@miguelgfierro @anargyri
Update NEWS.md
1b6ca64 
Co-authored-by: Andreas Argyriou <[email protected]>
@miguelgfierro @anargyri
Update README.md
0a40ee0 
Co-authored-by: Andreas Argyriou <[email protected]>
@miguelgfierro
Merge pull request #2008 from recommenders-team/miguel/readme_ms
793ef41 
Update readme LF feedback @wutaomsft
@miguelgfierro
📝
2beb9dd 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
remove unused input
2036105 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
📝
512581c 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
no need to output the logs twice
6a8d770 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
packages
6af75fd 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
skipping flaky test
a1a1785 
Signed-off-by: miguelgfierro <[email protected]>
@miguelgfierro
Merge pull request #2009 from recommenders-team/miguel/review_azureml…
d3da0fe 
…_tests

Review azureml tests
@miguelgfierro
Copy link
Collaborator Author

Added ideas for collaborators #2011

@miguelgfierro miguelgfierro merged commit 305f6b6 into main Oct 10, 2023
45 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anargyri anargyri anargyri approved these changes

@loomlike loomlike loomlike approved these changes

@gramhagen gramhagen Awaiting requested review from gramhagen gramhagen is a code owner

@SimonYansenZhao SimonYansenZhao Awaiting requested review from SimonYansenZhao SimonYansenZhao is a code owner

@wutaomsft wutaomsft Awaiting requested review from wutaomsft wutaomsft is a code owner

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

Successfully merging this pull request may close these issues.
5 participants
@miguelgfierro @yueguoguo @anargyri @wutaomsft @loomlike