Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop evaluating booleans #2614

Closed
wants to merge 3 commits into from

Conversation

drepram
Copy link
Contributor

@drepram drepram commented Jan 13, 2020

This is one of the fix of the problem that has occurred on EvalAI-ngx, particularly Cloud-CV/EvalAI-ngx#249.

The changes I've proposed removed the evaluation though I've not been able to resolve the angular-devkit issue.

Here are the things that I've read and have tried:

akveo/ngx-admin#2138
https://stackoverflow.com/questions/56393158/errors-data-path-buildersapp-shell-should-have-required-property-class
https://stackoverflow.com/questions/56542518/schema-validation-failed-with-the-following-errors-data-path-buildersapp-sh/56624167
https://stackoverflow.com/questions/56506473/ionic-problem-ngschema-validation-failed-with-the-following-errorsngdata-p/56537342#56537342
bpmn-io/bpmn-js-example-angular#15

Perhaps someone else can give me some insight on how I can resolve this issue?

@codecov-io
Copy link

codecov-io commented Jan 13, 2020

Codecov Report

Merging #2614 into master will decrease coverage by 0.06%.
The diff coverage is 8.57%.

@@            Coverage Diff             @@
##           master    #2614      +/-   ##
==========================================
- Coverage   73.21%   73.14%   -0.07%     
==========================================
  Files          82       82              
  Lines        5338     5347       +9     
==========================================
+ Hits         3908     3911       +3     
- Misses       1430     1436       +6
Impacted Files Coverage Δ
scripts/workers/worker_utils.py 0% <ø> (ø)
apps/jobs/admin.py 71.79% <ø> (ø) ⬆️
apps/jobs/serializers.py 91.26% <ø> (ø) ⬆️
apps/jobs/views.py 100% <ø> (ø) ⬆️
scripts/workers/rl_submission_worker.py 0% <0%> (ø) ⬆️
apps/jobs/models.py 94.93% <100%> (+0.06%) ⬆️
frontend/src/js/controllers/challengeCtrl.js 73.78% <100%> (+0.04%) ⬆️
Impacted Files Coverage Δ
scripts/workers/worker_utils.py 0% <ø> (ø)
apps/jobs/admin.py 71.79% <ø> (ø) ⬆️
apps/jobs/serializers.py 91.26% <ø> (ø) ⬆️
apps/jobs/views.py 100% <ø> (ø) ⬆️
scripts/workers/rl_submission_worker.py 0% <0%> (ø) ⬆️
apps/jobs/models.py 94.93% <100%> (+0.06%) ⬆️
frontend/src/js/controllers/challengeCtrl.js 73.78% <100%> (+0.04%) ⬆️

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 275a3a5...5191880. Read the comment docs.

@@ -696,7 +696,7 @@ def main():
q_params["pk"] = challenge_pk

if settings.DEBUG or settings.TEST:
if eval(LIMIT_CONCURRENT_SUBMISSION_PROCESSING):
if LIMIT_CONCURRENT_SUBMISSION_PROCESSING:
Copy link
Member

Choose a reason for hiding this comment

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

@drepram Why do we need to remove this?

@stale
Copy link

stale bot commented Apr 29, 2020

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the inactivity label Apr 29, 2020
@stale
Copy link

stale bot commented May 6, 2020

This pull request has been automatically closed as there is no further activity. Thank you for your contributions.

@stale stale bot closed this May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants