Skip to content

Refactor codebase for extensibility: centralized algorithm config, re…#29

Open
zappvik wants to merge 5 commits intomainfrom
modCodeForContri
Open

Refactor codebase for extensibility: centralized algorithm config, re…#29
zappvik wants to merge 5 commits intomainfrom
modCodeForContri

Conversation

@zappvik
Copy link
Copy Markdown
Collaborator

@zappvik zappvik commented Feb 18, 2026

...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!

…sponsive landing page layout, and contributing documentation
@zappvik zappvik requested a review from Astrasv February 18, 2026 19:07
@Astrasv
Copy link
Copy Markdown
Member

Astrasv commented Feb 19, 2026

Sure thing
Will review ASAP

@Astrasv
Copy link
Copy Markdown
Member

Astrasv commented Feb 21, 2026

Feature or Bug?

Everything looks cool. And works perfectly fine. I tried implementing the firefly algorithm. Considering everyone uses AI bots
for building stuff, I made the AI work completely according to the CONTRIBUTING.md file (I cross-verified the AI code. OCD issues 😂).

One small thing. 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 - REFER THE IMAGE

We have 2 options

  • Own algos dont have internal steps (Straight-away restriction)
  • Enable users to build internal logics as well (painfully hard to build considering time constraints 😂)
image

@zappvik
Copy link
Copy Markdown
Collaborator Author

zappvik commented Feb 21, 2026

quoted "I made the AI work completely according to the CONTRIBUTING.md file (I cross-verified the AI code. OCD issues 😂)"
first of all, as a fellow OCD, i absolutely get your feelings. its like some imposter has entered your repo and life and changed the whole fking codebase; its alright lol.
also did you take the headache of changing code or is it jus code review?

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"
i actually never wanted to take the burden of doing this so i just skipped it 😭.
also id say we leave it like this for the new algos as it a huge pain in a** to actually do something like that. it is better we focus on the main features instead

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

@Astrasv
Copy link
Copy Markdown
Member

Astrasv commented Feb 21, 2026

@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.

@zappvik
Copy link
Copy Markdown
Collaborator Author

zappvik commented Feb 21, 2026

Cool! @Astrasv
So I'm leaving this repo in the current state ig.
Not gonna change stuff anymore until I see some potential changes.
Will work on other stuffs now

@zappvik
Copy link
Copy Markdown
Collaborator Author

zappvik commented Feb 22, 2026

i made a blunder
gp has 3d like it was back then
thankful that we did not merge this

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.

improvement: modify codebase for easy addition of new algorithms Docs: add CONTRIBUTING.md

2 participants