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

Support punctuations #95

Open
KrasnayaPloshchad opened this issue Feb 14, 2022 · 9 comments
Open

Support punctuations #95

KrasnayaPloshchad opened this issue Feb 14, 2022 · 9 comments
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed

Comments

@KrasnayaPloshchad
Copy link

KrasnayaPloshchad commented Feb 14, 2022

If this app could be able to convert a punctuation to morse code or vice versa, this app can present not only a simple phrase like SOS or Hello World but also a complete sentence.

@CrazyMarvin
Copy link
Contributor

This would be a great improvement.
@FahadSaleem is working on refactoring the code and maybe he can directly implement support for punctuations.

@CrazyMarvin CrazyMarvin added the enhancement New feature or request label Feb 18, 2022
@CrazyMarvin
Copy link
Contributor

I suppose the punctuations on https://morsecode.world/international/morse.html would be a good start.

@CrazyMarvin CrazyMarvin added help wanted Extra attention is needed good first issue Good for newcomers hacktoberfest labels Oct 3, 2023
@CrazyMarvin
Copy link
Contributor

Are you interested to help with this too, @fejd? 😅

You have done so much so it's fine if you say no.

@fejd
Copy link
Contributor

fejd commented Oct 17, 2023

Sure, I could take a look. Is the code refactoring still in progress? I was thinking of breaking things out of the main activity and perhaps convert some of that functionality to Kotlin in the process. :)

If I understand the feature request correctly, there should be a mapping of punctuations from the "Punctuation Mark" table in https://morsecode.world/international/morse.html to dots/dahs.

I noticed on https://morsecode.world/international/timing.html that the timing is different depending on the type of space between characters and words. I should probably revisit the sound playback and adapt the pause between sounds based on that. Also, maybe it's useful to share the "playback algorithm" between audio, camera flash and "screen flash"?

@CrazyMarvin
Copy link
Contributor

The refactoring did not start yet unfortunately. So feel welcome to start with it. 😅

Yes, you understood this correctly. 👍

There is another issue about the timing: #36.
This is a bit more complex but it would be awesome if you could tackle it though.

@Noitarud
Copy link

I only have:
morse-sec001a4

@CrazyMarvin
Copy link
Contributor

Do you have time for this or are you busy, @fejd?

@fejd
Copy link
Contributor

fejd commented Jan 29, 2024

@CrazyMarvin Sorry, I'll be a bit busy for the next 3-4 months.

@CrazyMarvin
Copy link
Contributor

@CrazyMarvin Sorry, I'll be a bit busy for the next 3-4 months.

Alright, thanks for the info. 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers hacktoberfest help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

5 participants