tutorials icon indicating copy to clipboard operation
tutorials copied to clipboard

Add Flash Question Answering tutorial

Open karthikrangasai opened this issue 2 years ago • 5 comments

Before submitting

  • [x] Was this discussed/approved via a Github 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 Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

karthikrangasai avatar Dec 15 '21 12:12 karthikrangasai

@karthikrangasai mind checking the failing tests? :]

Borda avatar Jan 07 '22 13:01 Borda

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

Borda avatar Mar 09 '22 20:03 Borda

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.

karthikrangasai avatar Mar 10 '22 09:03 karthikrangasai

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           

codecov[bot] avatar Apr 11 '22 11:04 codecov[bot]

  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

Borda avatar Apr 11 '22 12:04 Borda