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

all pipelines working #18

Merged
merged 56 commits into from
Jun 1, 2022
Merged

all pipelines working #18

merged 56 commits into from
Jun 1, 2022

Conversation

cindyweng
Copy link
Collaborator

No description provided.

maggiemhanna and others added 30 commits May 24, 2022 19:22
@maggiemhanna
decomponentize pipeline steps
957ae28
@murggu
remove arm
772fce8
@cindyweng
moved envs to data-science
b57beb3
@cindyweng
removed casing folders
cac8fa6
@murggu
Align bicep + tf
013be62
@cindyweng
fixed env paths
e7d3426
@maggiemhanna
adding mlflow logging
e356be6
@maggiemhanna
change mlops dir structure
bcabc8c
@cindyweng
naming changes
0981c71
@cindyweng
Merge pull request #14 from Azure/am-infra
1972c98 
Align bicep and terraform
@cindyweng
changed git connection name
a5edf8e
@cindyweng
changed git connection
691622d
@cindyweng
fixed var names
a0346a8
@maggiemhanna
register model step using mlflow
931fe64
@cindyweng
moving files for deployment
82880e1
@cindyweng
moving files for deployment
1e94455
@cindyweng
added envs
0f2cd19
@cindyweng
fixed paths
6ce2d66
@cindyweng
modified experiment name
6e81c53
@cindyweng
added print statement
368bec8
@cindyweng
added env registration
80d2768
@cindyweng
syntax changes in aml cli v2
18087e3
@cindyweng
cleanup
b172429
@cindyweng
added online endpoints
5c14762
@cindyweng
syntax changes
cdfeb01
@maggiemhanna
add fairlearn and interpret-ml to evaluate step
93e50ca
@maggiemhanna
training pipeline yml file name changed, use custom env
2a50c14
@maggiemhanna
batch deployment using azureml model
68e45f0
@maggiemhanna
batch sample for batch endpoint
1605839
@maggiemhanna
yml files for custom train env
ff05ef3
maggiemhanna and others added 26 commits May 27, 2022 15:32
@maggiemhanna
fixing code url with new dir structure
0b93117
@maggiemhanna
renaming transformed data to prepared data
1a88fe5
@maggiemhanna
simplifying variable names
9ced2fd
@maggiemhanna
batch endpoint using registered azure ml model with mlflow
53d85b8
@maggiemhanna
online endpoint with model registered in azureml using mlflow
b5e43bd
@cindyweng
Merge branch 'dev' of https://github.com/Azure/mlops-project-template …
76053bc 
…into dev
@cindyweng
syntax fixes
3aecf91
@cindyweng
fixed folder paths
3b0236b
@cindyweng
limiting training files
8a8fa02
@cindyweng
deployment debugging
6b88477
@cindyweng
added endpoint creation parameter
499133a
@cindyweng
naming tidying up
3826484
@cindyweng
path fix
4bfd0b6
@murggu
add dev var to naming
3d0a9ba
@murggu
revert to baseName
8c6605a
@murggu
change ado sc for testing
f7e3cbc
@murggu
change postfix for testing
040ff18
@murggu
change postfix for testing
f629ae8
@cindyweng
Merge pull request #17 from Azure/am-infra-0531
2c74d63 
Test ado pipelines
@cindyweng
Delete classical/aml-cli-v2/data-batch directory
2df8818
@maggiemhanna
simplifying training pipeline
a21e512
@maggiemhanna
updating model to use scikit-learn pipeline, feature engineering, ran…
1985a00 
…dom forest regressor
@cindyweng
fixed paths
2c6d86c
@cindyweng
fixed paths
0e80d7e
@maggiemhanna
using log_param for hyperparameters logging
e0b442b
@cindyweng
renamed pipeline
37eeff6
setuc
setuc approved these changes Jun 1, 2022
View reviewed changes
Copy link
Collaborator

@setuc setuc left a comment

Choose a reason for hiding this comment

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

The files and the runs seem ok. They are only failing as we already have a working workspace.

@setuc setuc merged commit ac1f641 into main Jun 1, 2022
@lgtm-com
Copy link

lgtm-com bot commented Jun 1, 2022

This pull request introduces 8 alerts and fixes 9 when merging 37eeff6 into f361d9c - view on LGTM.com

new alerts:

  • 6 for Unused local variable
  • 1 for Module is imported more than once
  • 1 for Unused import

fixed alerts:

  • 8 for Unused import
  • 1 for Module is imported more than once

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@setuc setuc setuc approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@cindyweng @setuc @maggiemhanna @murggu