Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: brianjester/learntools
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: master
Choose a base ref
...
head repository: Kaggle/learntools
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: master
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 12 commits
  • 3 files changed
  • 1 contributor

Commits on Mar 27, 2025

  1. Rough implementation using kagglesdk

    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
    jplotts committed Mar 27, 2025
    Configuration menu
    Copy the full SHA
    7c0888b View commit details
    Browse the repository at this point in the history

Commits on Mar 28, 2025

  1. Merge pull request Kaggle#478 from Kaggle/using-kagglesdk

    Rough implementation using kagglesdk
    jplotts authored Mar 28, 2025
    Configuration menu
    Copy the full SHA
    9188caf View commit details
    Browse the repository at this point in the history

Commits on May 27, 2025

  1. Sort the results before comparison

    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
    jplotts committed May 27, 2025
    Configuration menu
    Copy the full SHA
    b00a2aa View commit details
    Browse the repository at this point in the history
  2. Merge pull request Kaggle#480 from Kaggle/fix-ordering-sql-ex

    Sort the results before comparison
    jplotts authored May 27, 2025
    Configuration menu
    Copy the full SHA
    003526b View commit details
    Browse the repository at this point in the history

Commits on Jul 1, 2025

  1. Use the KAGGLE_LEARN_SESSION_ID

    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
    jplotts committed Jul 1, 2025
    Configuration menu
    Copy the full SHA
    a3752b6 View commit details
    Browse the repository at this point in the history

Commits on Jul 2, 2025

  1. Merge pull request Kaggle#481 from Kaggle/use-env-var

    Use the `KAGGLE_LEARN_SESSION_ID`
    jplotts authored Jul 2, 2025
    Configuration menu
    Copy the full SHA
    58d1c06 View commit details
    Browse the repository at this point in the history

Commits on Jul 17, 2025

  1. Post response back to the outer iframe.

    Enables Kaggle Notebooks to continue to show nudges even if using kagglesdk under the hood.
    
    http://b/379083750
    jplotts committed Jul 17, 2025
    Configuration menu
    Copy the full SHA
    a8e9772 View commit details
    Browse the repository at this point in the history

Commits on Jul 18, 2025

  1. Fix comment

    jplotts committed Jul 18, 2025
    Configuration menu
    Copy the full SHA
    3f4775a View commit details
    Browse the repository at this point in the history
  2. More comment formatting

    jplotts committed Jul 18, 2025
    Configuration menu
    Copy the full SHA
    2bd9089 View commit details
    Browse the repository at this point in the history
  3. Merge pull request Kaggle#482 from Kaggle/post-nudges

    Post response back to the outer iframe.
    jplotts authored Jul 18, 2025
    Configuration menu
    Copy the full SHA
    8ee90fc View commit details
    Browse the repository at this point in the history

Commits on Jul 24, 2025

  1. Cast KAGGLE_LEARN_SESSION_ID to int

    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
    jplotts committed Jul 24, 2025
    Configuration menu
    Copy the full SHA
    14475cf View commit details
    Browse the repository at this point in the history

Commits on Jul 25, 2025

  1. Merge pull request Kaggle#483 from Kaggle/fix-session-id

    Cast KAGGLE_LEARN_SESSION_ID to int
    jplotts authored Jul 25, 2025
    Configuration menu
    Copy the full SHA
    a0c08ac View commit details
    Browse the repository at this point in the history
Loading