-
Notifications
You must be signed in to change notification settings - Fork 0
Assignment3 #3
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
base: master
Are you sure you want to change the base?
Assignment3 #3
Conversation
Assignment3/Assignment3.py
Outdated
| sql1 = "select * from baseball.batter_counts" | ||
| database = "baseball" | ||
| user = "bita" | ||
| password = "salam" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bita don't add your your password onto a file on github, you want to use an environment variable or something so people dont know it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you added the pre-commit hook, it would have caught that and not let it get committed!
| ) | ||
|
|
||
| sql1 = "select * from baseball.batter_counts" | ||
| database = "baseball" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't put your password in plain text on the internet, you're going to want to use an environment variable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Assignment3/Assignment3.py
Outdated
| numTrees=100, | ||
| predictionCol="pred_survived", | ||
| ) | ||
| random_forest_fitted = random_forest.fit(results) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I dont know if your transformer is working right
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok I changed if from random forest regressor to linear regression. for me it runs without error, can you run it too?
TDerig23
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I left an extra comment about your transformer
No description provided.