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

AzureML Designer integration with SAR #1060

Merged
merged 66 commits into from
Mar 9, 2020
Merged

AzureML Designer integration with SAR #1060

merged 66 commits into from
Mar 9, 2020

Conversation

miguelgfierro
Copy link
Collaborator

Description

One end 2 end example of Designer integration with SAR

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 and not master.

Heyi Tang and others added 30 commits December 17, 2019 10:54
Add module Stratified splitter as AzureML Designer Module
5f1c49d
Update version and arg format
a7aa230
@zzn2
Merge pull request #3 from dwt0317/weda/demo
9852581 
Add entry file and yaml spec for stratified splitter
@dwt0317
Merge pull request #2 from dwt0317/weda/demo
cd805bb 
Add entry and yaml files
Add SAR score and train
eef60b1
Add entry file
a859fbb
Change column back to string
1730193
Fix bug of sar score
9273ea4
Add predict_training_item and return dataframe
af16d38
fix get-item-based-k
91c65d5
Update options
2aae859
Change output format to ModelDirectory
59fa6c8
update version
d15cca1
Update strafified version
6e13efb
@weedqian
Merge pull request #4 from dwt0317/weda/score
1709375 
Add train SAR and score SAR module
Update dependency
3ece9ce
@dwt0317
Merge pull request #3 from dwt0317/weda/score
bc3e392 
Weda/score
Update version
29d9ec3
@dwt0317
Merge pull request #4 from dwt0317/weda/score
1845f8e 
Update version
update conda
fad1737
Reset version
a4ab496
@weedqian
Merge pull request #5 from dwt0317/weda/score
bcfef15 
Update conda config, reset version number
Remove train spec and entry
78d232b
@weedqian
Merge pull request #7 from dwt0317/weda/no_train
3bc4961 
Remove train spec and entry
@miguelgfierro
Merge pull request #1036 from hirofumi-s-friends/master
8591211 
Initial code of AzureML designer to a new branch in recommenders repo
@miguelgfierro
changed name from Experimentation to Recommenders
5bb9101
@miguelgfierro
changed names
0581ddf
@miguelgfierro
added sar training
a717b61
@miguelgfierro
added sar training
f167296
@miguelgfierro
🐛
a5ce8f6
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@miguelgfierro miguelgfierro changed the base branch from master to staging February 28, 2020 16:22
@miguelgfierro miguelgfierro merged commit 8797561 into staging Mar 9, 2020
@miguelgfierro miguelgfierro deleted the designer branch March 9, 2020 15:47
@miguelgfierro miguelgfierro mentioned this pull request Mar 9, 2020
Closed
@anargyri
Copy link
Collaborator

anargyri commented Mar 9, 2020

Nitpicking: when you say pip install azureml-designer-tools==0.1.18 is 0.1.18 the latest version? We probably don't want to update this file whenever there is a new release of azureml-designer-tools.

@miguelgfierro
Copy link
Collaborator Author

this notebook is focused on that version, which is the one it was tested (in fact also 0.1.10), since designer is still under strong development, we pinned the version. When it is more stable, we can revisit this rule

@miguelgfierro miguelgfierro mentioned this pull request Mar 10, 2020
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dwt0317 dwt0317 dwt0317 approved these changes

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

@yueguoguo yueguoguo Awaiting requested review from yueguoguo

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

@weedqian weedqian Awaiting requested review from weedqian

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

@zzn2 zzn2 Awaiting requested review from zzn2

@youngpark youngpark Awaiting requested review from youngpark

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 @anargyri @dwt0317 @zzn2 @weedqian