Refactor codebase for extensibility: centralized algorithm config, re…#29
Refactor codebase for extensibility: centralized algorithm config, re…#29
Conversation
…sponsive landing page layout, and contributing documentation
|
Sure thing |
|
quoted "I made the AI work completely according to the CONTRIBUTING.md file (I cross-verified the AI code. OCD issues 😂)" quoted "I don't know if we need to call it bug or feature. But the internal working of the algorithms (which is shown in the standard 5 algos) is not integrated as part of the newly contributed algorithms" if at all the user is absolutely interested in finding out how the thing actually works, he can go through the other 5 algos and learn stuff so what i say is to leave it like this, if at all it becomes necessary in future (which i dont see it coming), then we'll take the burden of doing it |
|
@zappvik You make total sense quoted "also did you take the headache of changing code or is it jus code review?" - I just reviewed the code. Planning to change a few parts (Not now though) I will keep this PR open. And lets not worry about adding "algo internals logic". Let it be like it is now. |
|
Cool! @Astrasv |
|
i made a blunder |

...sponsive landing page layout, and contributing documentation
fixes #27
fixes #28
refactored the codebase for extensibility. i actually hardcoded a lot before just for 5 algorithms. now centralized the algorithm config and fixed the landing page responsiveness so it doesn't break anymore. Added some docs for contributors too.
i know it’s a lot to review @Astrasv. take your time to review the new codebase; it took 3 hours plus some sleep deprivation for me to do this lol, so you also take your time!