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.

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

User geolocation #1

Closed
wants to merge 7 commits into from
Closed

User geolocation #1

wants to merge 7 commits into from

Conversation

moprescu
Copy link

@moprescu moprescu commented Oct 6, 2015

Users can now specify geolocation either by specifying the path to the Azure ML web service management or by specifying the workspace settings from a configuration file ~/azureml/settings.ini. If the latter option is used, the user can omit the workspace ID and the authorization token when calling the package functions. Documentation was updated to reflect these changes.

In addition, an issue that prevented the user from defining a function with the same name as a function in the imported packages was fixed.

…tions other than South Central US.

User can now specify the path to the AzureML web service management. The path depends on the region
the workspace was created in (e.g. South Central US, West Europe, etc).
Update roxygen documentation to include new package functionality. Update vignettes.
Coerce publishWebService and updateWebService to read user defined functions from
the R global environment.

Previously, publishWebService and updateWebService would throw an error if the user passed in
a function that also exists in the packages sourced by AzureML. For examplem, the "add1" function
from the older tutorial would fail to publish as a web service since another function with the
same name exists in the "stats" package. This fix restricts the function lookup to the global
R environment.
@gaborcsardi
Copy link

Hi, thanks for the PR! This is a read-only mirror of CRAN, so please send your changes to the original package authors. See the DESCRIPTION file.

Thanks!

@gaborcsardi gaborcsardi closed this Dec 5, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants