Conversation

karthikrangasai

Before submitting

  • Was this discussed/approved via a issue? (no need for typos and docs improvements)
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Fixes # (issue).

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

# First, download the data:

# %%
download_data("https://pl-flash-data.s3.amazonaws.com/kaggle_chaii.zip", DATASET_PATH)
Copy link
Member

@Borda Borda Dec 16, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's move this to meta-file

Suggested change
download_data("https://pl-flash-data.s3.amazonaws.com/kaggle_chaii.zip", DATASET_PATH)

then we can even skip this cell :]

@Borda

@karthikrangasai mind checking the failing tests? :]

@BordaBorda requested review from Borda and removed request for aribornstein March 9, 2022 20:01
@Borda

@karthikrangasai could you pls remind me what I have promised here to do to unblock it? 🐰

@karthikrangasai

Hello @Borda ,

We had spoken about

  1. the Kaggle API key getting outdated and updating it
  2. enrolling the lightinig kaggle testing account into the competition.

Downloading and sharing the dataset again might have licensing issues.

After this, I don't think we had arrived to a conclusive solution.

@BordaBorda force-pushed the main branch 4 times, most recently from acdd659 to 0bb4a7e Compare April 9, 2022 14:25
@codecov

Codecov Report

Merging #125 (5157d01) into main (89b94ba) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #125   +/-   ##
===================================
  Coverage    73%    73%           
===================================
  Files         2      2           
  Lines       382    382           
===================================
  Hits        280    280           
  Misses      102    102           

@Borda
  1. the Kaggle API key getting outdated and updating it

just re-generated Key and the same issue, also check that the user approved dataset conditions... need to check deeper

@BordaBorda requested a review from rohitgr7 as a code owner August 15, 2022 07:21
@BordaBorda marked this pull request as draft October 3, 2023 21:01
@BordaBorda force-pushed the main branch 2 times, most recently from d4acf6f to ac8f7ba Compare July 19, 2024 21:12
@BordaBorda closed this Jul 20, 2024
Sign up for free to join this conversation on . Already have an account? Sign in to comment
ExampleExample / Demo / TutorialPL: Flash
None yet

Successfully merging this pull request may close these issues.