The Wayback Machine - https://web.archive.org/web/20201230120800/https://github.com/microsoft/MLOpsPython/pull/302
Skip to content

/ MLOpsPython Template

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

Jump to bottom

Encoding in open function throw error #302

Open
wants to merge 4 commits into
base: master
from
Open

Encoding in open function throw error #302

wants to merge 4 commits into from

Conversation

@azure-walker
Copy link

@azure-walker azure-walker commented Jul 2, 2020

Removed the encoding parameter from the open function as this is not a valid parameter for the function.
azure-walker added 3 commits Jul 2, 2020
@azure-walker
removed encoding from open function as this causes an error
Loading status checks…
8548b04
@azure-walker
Merge pull request #1 from microsoft/master
Verified
This commit was created on GitHub.com and signed with a verified signature using GitHub’s key.
GPG key ID: 4AEE18F83AFDEB23 Learn about signing commits
8e9f144 
Get Latest changes
@azure-walker
Fixed issue with Bootstrapa.py throwing errors with encoding in open …
91b7c22 
…function
@j-so
Copy link
Contributor

@j-so j-so commented Jul 2, 2020

I'm not seeing this error - this shouldn't be an issue as long as you're using Python 3, right?
@azure-walker
shortened the app-ci to make it more piece-meal
Loading status checks…
710291c
@dtzar
Copy link
Member

@dtzar dtzar commented Jul 6, 2020

We should not be changing the name of diabetes_regression to app everywhere
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants
@azure-walker @j-so @dtzar
You can’t perform that action at this time.