Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Update Instrument with Master Changes - #102

Open
Daniel Ciborowski (dciborow) wants to merge 19 commits into
mabou/instrumentfrom
master
Open

Update Instrument with Master Changes#102
Daniel Ciborowski (dciborow) wants to merge 19 commits into
mabou/instrumentfrom
master

Conversation

@dciborow

Copy link
Copy Markdown
Contributor

No description provided.

* Matrix Strategy
Adding a strategy to run in 3 regions : eastus, westus2, southcentralus
location parameter was never set in the 03 notebook but added azlocation to set that field when running that notebook.
* Re-order parameters
Papermill complained it didn't know abut azurerggroup, but similarly said it didn't know what estimators were....
* Correct Param Names
azurergname wasn't used throughout the matrix strategy, only the first one, which seems to make sense based on the results.
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@dciborow@marabout2015@grecoe