Skip to content
This repository has been archived by the owner on Jun 22, 2024. It is now read-only.

Add algorithm to select players based on credits #4

Open
roysti10 opened this issue Sep 30, 2020 · 0 comments
Open

Add algorithm to select players based on credits #4

roysti10 opened this issue Sep 30, 2020 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers wontfix This will not be worked on

Comments

@roysti10
Copy link
Member

roysti10 commented Sep 30, 2020 •
edited
Loading

Describe the issue
Currently our model predicts only on points but fantasy cricket has limitations on the basis of credits, we would like to implement the algorithm for it

Solution
Create a function that takes input the players list , credits list , maximum credits for the match and points predicted by the model for each player and select the best 11 players, based on the points whilst not crossing the maximum credits.
Note: be careful not to violate the team rule , i.e maximum 7 players from each team

Test
You can test with your own credits as of now, as we are still figuring out an api for it
You do not need to integrate it with current flask model, you can create a function in team.py as of now

Comment if you would like to work on it
@roysti10 roysti10 added enhancement New feature or request good first issue Good for newcomers hacktoberfest Hacktober fest label help wanted Extra attention is needed labels Sep 30, 2020
@robpoll robpoll removed their assignment Oct 15, 2020
@HackerSpace-PESU HackerSpace-PESU deleted a comment from robpoll Oct 15, 2020
@HackerSpace-PESU HackerSpace-PESU deleted a comment from robpoll Oct 15, 2020
@HackerSpace-PESU HackerSpace-PESU deleted a comment from shloka-gupta Oct 18, 2020
@roysti10 roysti10 removed the hacktoberfest Hacktober fest label label Nov 14, 2020
roysti10 added a commit that referenced this issue Nov 14, 2020
@roysti10
# This is a combination of 4 commits.
cf82d7d 
# This is the 1st commit message:

[CODE] Cleaned pylint warnings for check

# The commit message #2 will be skipped:

# [FILES] Deleted all matches

# The commit message #3 will be skipped:

# [FILES] Deleted countries.py

# The commit message #4 will be skipped:

# [CODE] Cleaned code for pylint warnings
roysti10 added a commit that referenced this issue Nov 15, 2020
@roysti10
# This is a combination of 4 commits.
169942b 
# This is the 1st commit message:

[CODE] Cleaned pylint warnings for check

# The commit message #2 will be skipped:

# [FILES] Deleted all matches

# The commit message #3 will be skipped:

# [FILES] Deleted countries.py

# The commit message #4 will be skipped:

# [CODE] Cleaned code for pylint warnings
roysti10 added a commit that referenced this issue Nov 15, 2020
@roysti10
# This is a combination of 4 commits.
ffd8120 
# This is the 1st commit message:

[CODE] Cleaned pylint warnings for check

# The commit message #2 will be skipped:

# [FILES] Deleted all matches

# The commit message #3 will be skipped:

# [FILES] Deleted countries.py

# The commit message #4 will be skipped:

# [CODE] Cleaned code for pylint warnings
roysti10 added a commit that referenced this issue Nov 15, 2020
@roysti10
# This is a combination of 4 commits.
d12d7c7 
# This is the 1st commit message:

[CODE] Cleaned pylint warnings for check

# The commit message #2 will be skipped:

# [FILES] Deleted all matches

# The commit message #3 will be skipped:

# [FILES] Deleted countries.py

# The commit message #4 will be skipped:

# [CODE] Cleaned code for pylint warnings
roysti10 pushed a commit that referenced this issue Jan 5, 2021
@scientes
Merge pull request #4 from scientes/patch-5
78f37d3 
Create DOCKERFILE.development
@roysti10 roysti10 added wontfix This will not be worked on and removed help wanted Extra attention is needed labels Feb 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers wontfix This will not be worked on
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@robpoll @roysti10