-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Attempt to guess an abbreviation when a predefined one doesn't exist #13
Comments
As a first implementation of this feature, what about something similar the |
Possibly, although I'd prefer to avoid copyright issues from copying the algorithm. What does the license say? |
Hi, I was exploring this issue and tried to implement the removing all vowels strategy. I was thinking whether we should notify user that we are guessing the abbreviation or not? Example : |
I would be cautious about that. I think people will be consuming the output directly and we don't want them to be concerned with filtering out log entries. Could an option be to use a CLI param to select the algorithm used? Maybe we could have something to check if it's a known abbreviation? That might be a separate PR though. |
Can you give an example of how the cli param should look.?.I am understanding it as : |
No, that's not naive. I would probably call the parameter Thinking about it, it makes it easier to add new algorithms later. |
Ok...For lookupOnly we may need a seperate discussion on how it should be implemented...Should I go ahead with implementing removeVowel or there is something more you want to discuss? |
No, I think that's it. I look forward to your PR! |
Hi, I have implemented this but instead of using |
Cool! Are you ready to raise a PR yet? |
#19 Raised a PR |
I have another idea to propose for "guessing" an abbreviation. So the behavior would look like:
In this example:
(Assuming I can find a clever way to find the first syllable of multi-syllable words) Would this new flag be of interested? Let me know and I can work on a PR for hacktoberfest! |
Or Please, feel free to raise a pull request early. Don't worry about having a full solution. Try something, see how the code feels to you. I'm curious to see what you come up with. |
Oh interesting, I was thinking it would only replace words for which an abbreviation is NOT known. But that might be additional option to add later? Something to think about...
Sounds good will do :) |
Is your feature request related to a problem? Please describe.
At the moment, there is a defined list of abbreviations that are looked up. Where an abbreviation isn't present, it just won't be abbreviated.
Describe the solution you'd like
It would be good if the tool could attempt to shorten words taht it doesn't already know about.
A couple of different strategies we could explore:
Additional context
How would this work with other languages?
The text was updated successfully, but these errors were encountered: