Skip to content
This repository has been archived by the owner on Jul 6, 2020. It is now read-only.

Techniques to pull data from EvalAI from within EvalAI-ngx #246

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jayaike
Copy link
Contributor

@jayaike jayaike commented Dec 23, 2019

As requested in a GCI task:

  • I have added the file with the techniques to pull data from EvalAI from within EvalAI-ngx aside from git clone

@codecov-io
Copy link

codecov-io commented Dec 23, 2019

Codecov Report

Merging #246 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #246   +/-   ##
=======================================
  Coverage   50.87%   50.87%           
=======================================
  Files          66       66           
  Lines        3731     3731           
  Branches      438      438           
=======================================
  Hits         1898     1898           
  Misses       1738     1738           
  Partials       95       95

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1228629...4157798. Read the comment docs.

@pushkalkatara
Copy link

@nsjcorps Nice suggestions. I think COPY helps our usecase. Also, we have EvalAI as a docker image, we can get inspiration from here

@jayaike
Copy link
Contributor Author

jayaike commented Dec 24, 2019

Thanks, @pushkalkatara , I like the sound of that. Please Review @Ram81 , @pushkalkatara

@yashdusing
Copy link

Some really good points have been mentioned already. I was thinking along the lines of maintaining a separate codebase for the django container since that is the only dependency shared between both the repositories.

@jayaike
Copy link
Contributor Author

jayaike commented Dec 24, 2019

That makes a lot of sense. That way you only have to pull the separate codebase that is shared by both repositories

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants