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

Adds solution for the 2022 April challenge #51

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bubakazouba
Copy link

No description provided.

@isaac-gs
Copy link

We can clean it up and add some extra documentation later, a little messy now.

@MaldoAlberto
Copy link
Collaborator

Hi @bubakazouba, @isaac-gs Thank you very much for your participation 😄 , I can see that you put a lot of effort in your work, but it is difficult for me to understand the code, could you add an explanation of your solution and how it was achieved?, as well as the methods and algorithms that were based, it would be very helpful to understand your work.

@isaac-gs
Copy link

@MaldoAlberto sorry, we already have this doc (still a work in progress) but it should give a high level overview. There's actually 4 of us who worked on it.

https://docs.google.com/document/d/18WvZjXNb6uousWPijxhgJu3tzbyaiOc5TfK5Sedw51Y/edit

We can also add some of this documentation in the notebook if you would prefer.

@MaldoAlberto
Copy link
Collaborator

Hi @isaac-gs It seems to me a good document 👍 , it should be that explanation in the notebook to be accepted, when you have it let me know to review it and to be able to my comments or to accept it.

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

Successfully merging this pull request may close these issues.

3 participants