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

Fix #FIX-NEEDED #20

Open
1 task done
Agrover112 opened this issue Oct 1, 2021 · 5 comments
Open
1 task done

Fix #FIX-NEEDED #20

Agrover112 opened this issue Oct 1, 2021 · 5 comments
Labels
bug Something isn't working good first issue Good for newcomers Hacktoberfest help wanted Extra attention is needed

Comments

@Agrover112
Copy link
Owner

👟 Reproduction steps

Implemented in: mulit_mutattion in ga_utils.py
Can be tested by replacing mutation with multi_mutation in ga.py.

👍 Expected behavior

Mulit_mutattion in ga_utils.py should change N bits as selected.
[1,2,3,4] for 2 bits could be: [1,3,3,5]

👎 Actual Behavior

Mulit_mutattion in ga_utils.py doesn't work as expected.
Gives an Index Error when used , since the implementation is not correct.

👀 Have you spent some time to check if this issue has been raised before?

  • I checked and didn't find similar issue
@Agrover112 Agrover112 added bug Something isn't working help wanted Extra attention is needed Hacktoberfest labels Oct 1, 2021
@pratik1424
Copy link

I went through the code for multi-mutation. The problem seems to be from the "gene" variable, can I get some sample test cases for running that specific function?

@Agrover112
Copy link
Owner Author

@pratik1424 Sure just look at the expected behaviour.
It's supposed to edit multiple positions. So for ex [a,b,c,d], maybe the pos of the change is random between 0 and len(list)-1 , it could be anything from [a-/+1,b,c+/-1,d] like this!

@pratik1424
Copy link

Okay, I'll try something, I will open a PR after fixing the bug.

@Agrover112
Copy link
Owner Author

@pratik1424 Aare you working on it?

@pratik1424
Copy link

I tried a couple of different implementations, but the bug still persists.
Main issue is the selection of gene2 and gene variables, I tried using numpy.random.uniform, but the bug remained.

@Agrover112 Agrover112 added the good first issue Good for newcomers label Oct 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers Hacktoberfest help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants