-
Notifications
You must be signed in to change notification settings - Fork 168
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
Abstract out the matching algorithm architecture to allow other types of comparisons #685
Comments
hello @geekygirlsarah i have gone through the above mentioned detail but im not able to figure out whats the issue is. |
@seabeePraveen sorry, I meant to write back and lost the tab I opened in a sea of other tabs! The algorithm right now is heavily tied to the specific language compare and language reference view code. They are also written very specific to language lookups. Those algorithms need to be refactored out into their own class or something so that other types of lookups could happen (by their own views and thesaurus files) but also so the algorithm is written more generically in how it compares things. I updated the issue's wording a bit too. |
This issue has been inactive for 337 hours (14.04 days) and will be unassigned after 71 more hours (2.96 days). If you have questions, please leave a comment, message @codethesaurus or @geekygirlsarah on Twitter, or email [email protected].If you are still working on this issue, that's fine. Please comment here to tell the bot to give you more time. |
This issue has been inactive for 409 hours (17.04 days) and is past the limit of 408 hours (17.00 days) so is being unassigned.You’ve just been unassigned from this ticket due to inactivity – but feel free to pick it back up (or a new one!) in the future! Thank you for your interest in contributing to this project. |
Oh sorry bot. I'm still working on it! |
This issue has been inactive for 346 hours (14.42 days) and will be unassigned after 62 more hours (2.58 days). If you have questions, please leave a comment, message @codethesaurus or @geekygirlsarah on Twitter, or email [email protected].If you are still working on this issue, that's fine. Please comment here to tell the bot to give you more time. |
This issue has been inactive for 418 hours (17.42 days) and is past the limit of 408 hours (17.00 days) so is being unassigned.You’ve just been unassigned from this ticket due to inactivity – but feel free to pick it back up (or a new one!) in the future! Thank you for your interest in contributing to this project. |
Description
While comparing programming languages was the original purpose of Code Thesaurus, the goal was to enable other types of searches for things. This could include things like:
Right now the main matching algorithm is hard-coded for languages. It is also heavily tied into the reference and compare page views. This work would need to refactor the algorithms out of the views and into their own class, and also refactor the variables to be more generic for any types of comparisons. The end result would be something that's easy to add a new comparison type to the home page and with new views and thesaurus files, we have an easy way to scale this concept to multiple comparison types.
Requirements
Additional Notes
I haven't thought too much on this yet but will probably start looking into it during October 2023 (Hacktoberfest). I welcome anyone that wants to join in on this!
The text was updated successfully, but these errors were encountered: