vffg#1
Open
Mishatomuch wants to merge 77 commits into
Open
Conversation
[Time Series] Remove pyearth
Change imgur images links to Google hosted
[Game AI] install stable-baselines3 from repo
[Game AI] also need to update tutorial
http://b/287252878
Update timeout to avoid build failures.
Intro to programming Lesson 4 had some missing image files. This change fixes the URLs in the exercise so that they point to the pngs. Bug: 289537275
Fix image links
http://b/297034826
Remove slack notifications
Following pandas upgrade, support for ambiguous timedelta such as `1M` and `1Y` have been dropped.
fix test suite
Jenkins currently defaults to the nvidia docker runtime, however learntools only uses CPU tests and the colab image conflicts with the nvidia docker runtime when in CPU-only mode. http://b/365782129
bug fix q1
restore assert message
restore assert message
Fixes for type errors and NA errors in Intro to SQL and Advanced SQL courses
This shouldn't impact any existing users, because it depends on an environment variable being set. This will enable us to rollout gradually when we're ready. For now, there are still a few things to be done, primarily improving kagglesdk authorization. http://b/379083750
Rough implementation using kagglesdk
The SQL statements don't contain an `ORDER BY` clause, so the ordering can't be an intentional check for this exercise. This exercise happens to work most of the time because BigQuery tends to return results in the same order, but sometimes it doesn't. Learn users shouldn't have to deal with this non-determinism in the check. http://b/415948668
Sort the results before comparison
This environment variable was added in kaggleazure PR 36128. It is set on all Learn sessions. It refers to the version of the exercise the user forked from. Note that this is all still behind a feature flag, so this code is a no-op for now. http://b/379083750
Use the `KAGGLE_LEARN_SESSION_ID`
Enables Kaggle Notebooks to continue to show nudges even if using kagglesdk under the hood. http://b/379083750
Post response back to the outer iframe.
In testing, I was assuming this would already be an int, but it looks like KKB puts all environment variables as strings. http://b/379083750
Cast KAGGLE_LEARN_SESSION_ID to int
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
11 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.