microsoft / solution-accelerator-many-models

MIT License
192 stars 87 forks source link

should be grouped_models.append in both nb's #120

Open eavanvalkenburg opened 3 years ago

eavanvalkenburg commented 3 years ago

https://github.com/microsoft/solution-accelerator-many-models/blob/185d21ec5d870508201a97e55f7e148cc29d109b/Custom_Script/03_CustomScript_Realtime_Forecasting_Deployment.ipynb#L120

This line looks wrong, I think it should be grouped_models.append(group) in both the custom script and AML version