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

Add English keyboard #7

Closed
7 of 11 tasks
andrewtavis opened this issue Nov 22, 2021 · 13 comments
Closed
7 of 11 tasks

Add English keyboard #7

andrewtavis opened this issue Nov 22, 2021 · 13 comments
Assignees
Labels
feature New feature or request help wanted Extra attention is needed new keyboard Adding a new language keyboard

Comments

@andrewtavis
Copy link
Member

andrewtavis commented Nov 22, 2021

Terms

What's needed

Adding an English keyboard to Scribe is as of now not a priority, but it will eventually be added, and could be worked on now if there's interest. There is as of now a good amount of data on Wikidata as can be seen in the queries. The English keyboard will be more of a focus for Scribe when the Translate feature is more developed (see project), and further when the Synonym feature is added.

What's needed:

@andrewtavis andrewtavis added feature New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Nov 22, 2021
@andrewtavis andrewtavis added the new keyboard Adding a new language keyboard label Dec 4, 2021
@andrewtavis andrewtavis changed the title Keyboards: Add English Add English keyboard Dec 4, 2021
@andrewtavis andrewtavis mentioned this issue Jan 6, 2022
8 tasks
andrewtavis added a commit that referenced this issue Jan 7, 2022
@muescha
Copy link

muescha commented Jul 18, 2022

i would vote for an english keyboard - i think the most students are learning english and this keyboard would be helpful

@andrewtavis
Copy link
Member Author

Thanks for your feedback, @muescha!

Yes I think that an English keyboard would help for irregular verbs. Issue is that we’re currently only translating from English to other languages. This issue needs to wait until at least #16 is finished as a user would need to be able to select a language to translate from, and then we could add the translation JSONs.

Any further feedback or support you’d like to provide would be very welcome! Again, thanks for reaching out! 😊

@andrewtavis andrewtavis added blocked Another issue is blocking and removed good first issue Good for newcomers labels Jul 18, 2022
@andrewtavis
Copy link
Member Author

andrewtavis commented Jul 18, 2022

@wkyoshida
Copy link
Member

As mentioned in this discussion, I would be curious to hear thoughts on where the English keyboard now fits into the roadmap. I feel that the need is definitely there 🙏

@andrewtavis
Copy link
Member Author

andrewtavis commented Nov 3, 2022

Glad to revisit this, @wkyoshida 😊

Big thing as I mentioned above is that we need to be able to choose which language to translate from. Listening to the interest here, I think it makes sense that we do something along the lines of translations being an option for any language pair that we can find a viable 🤗 Hugging Face model for. This would mean that we can do English to any of the seven languages as we do now, and then any of the seven languages to English. A translation JSON is roughly 2MB, so I don't think that this is preventative for adding any individual language pair. It's not a guarantee that there will be a valid translation model for each language pair, but even if we use a general one it should be ok. We can check the results, and worst comes to worst one translation pair isn't included.

Ultimately we'd need to edit the translation scripts in Scribe-Data to basically generate the translation files where the given language pair is no longer simply the target language from English, but that each of the seven models (we'd have eight total languages then) is ran and we get the resultant JSONs stored. We'd need to change the architecture here to check what the user has selected as a base language, with assigning the system language being a great baseline that they could then choose to switch from. If the default system language isn't offered, we’d choose English as the secondary default for each language, and I'd argue Spanish should be the secondary default base for English.

I think I can update the roadmap based on the above. The English keyboard would definitely come before data downloads :) Question is whether we think that this would qualify for v3.0, which to me it kind of does 😊

@andrewtavis
Copy link
Member Author

Another thing we're hoping to do at some point, @wkyoshida, is to do a launch on Product Hunt. I think that as soon as we have the above translation pairs in place we'd be ready to make a splash 😊

@wkyoshida
Copy link
Member

Gotchu! Thanks - I think that what you shared makes sense, @andrewtavis! 👍
I did notice as well the new issue created for the translation data, scribe-org/Scribe-Data#23.

Another thing we're hoping to do at some point, @wkyoshida, is to do a launch on Product Hunt. I think that as soon as we have the above translation pairs in place we'd be ready to make a splash 😊

Oh - I see! That would be pretty exciting 🙌 I just noticed it also has been in the outreach plans, #181

@andrewtavis
Copy link
Member Author

96f32ab adds the directory for the eventual English keyboard :)

@wkyoshida
Copy link
Member

I can't remember if this is something that we even considered, but could it be possible if we just add the English keyboard, although without a translation option for it?

This would make it the only keyboard without the feature unfortunately, but I'm wondering if the need/desire for English could justify this. The other keyboard features would still be beneficial to users I think. I'm just thinking about the issues that are blocking a complete English keyboard and scribe-org/Scribe-Data#23 seems it could potentially take some time to get completed. My guess, as well, is that the user desire for English is higher than that for Bokmål or Danish.

Something I was wondering recently. Let me know 🤷

@andrewtavis
Copy link
Member Author

andrewtavis commented Apr 12, 2023

I totally agree that there's more need for English than there is for any other language :) I definitely do want to include a translate feature with it as well, but then I think that we can actually start on this pretty soon. My assumption is that #96 will be done soonish so long as the SQLite links are intuitive, and from there it's a data update away — one that will need me to check the process, but it'll be all of a couple hours max to make sure it's back to normal after #96. My hope is by the end of the week 😊

Something we could start on now is the WDQS queries and the Python formatting scripts above? Would you have an interest in looking at those, @wkyoshida?

@wkyoshida
Copy link
Member

Something we could start on now is the WDQS queries and the Python formatting scripts above? Would you have an interest in looking at those, @wkyoshida?

I could give that a go 👍 unless though if we would like to loop it under as one of the items for @SaurabhJamadagni to take on as part of GSoC. We can talk through a plan for this on our call!

@andrewtavis
Copy link
Member Author

I’d say so long as @SaurabhJamadagni does a keyboard it’s all good :) Doesn’t have to be this one, and we have #135 and #133 that would also be fine 😊 Let’s discuss!

@andrewtavis andrewtavis added the -next release- Included in the next release label Aug 14, 2023
@andrewtavis andrewtavis removed the blocked Another issue is blocking label Aug 28, 2023
@andrewtavis andrewtavis self-assigned this Sep 18, 2023
@andrewtavis andrewtavis removed the -next release- Included in the next release label Apr 13, 2024
Jag-Marcel added a commit to Jag-Marcel/Scribe-iOS that referenced this issue Jun 17, 2024
5fd31d2 Merge pull request scribe-org#18 from Jag-Marcel/github-actions-fix
3b71e3d Slight fix to xcstrings conversion workflow
06027db Merge pull request scribe-org#7 from Jag-Marcel/swift-conversions
6b43924 Minor formatting fixes, add pt back in and fix typo in the .yml file
13c9ced Minor spelling fixes
81f9c11 Merge branch 'main' into swift-conversions
f4041c7 Edited scripts to reflect agreed upon coonfiguration
8a99dba Add basic issue templates to bring people to community channels
33cb640 Reorder Matrix logo placement in contributing of readme
9daccff Add navigation to powered by in readme
b1a2932 Minor edits to readme and maintainer checklist
38d4fa7 scribe-org#1 Update readme with Weblate information and directions
ee1eb47 Move test string to base translation file
5bd6b19 Test webhook
cb06f73 Testing adding language file and adding string to repo
3fd3707 Implemented GitHub action for xcstrings conversion script
9b5f645 Adjusted conversion for new location of Xcode file
b3d8b03 Revert "Adjusted conversion to xcstrings"
3de1788 Adjusted conversion to xcstrings
811fe13 Replaced old placeholder
055453f Changed directory and file names
4982d14 Changed file in accordance with PEP 8 style guide
3051cad Revert "Added current localizations to JSON files"
d165540 Implemented conversion from JSON to files for iOS
1a7b45d Implemented script to convert files for iOS to JSON
d452419 Added current localizations to JSON files

git-subtree-dir: Scribe/i18n
git-subtree-split: 5fd31d28c3e2b29fbf4974226862468288ec846e
andrewtavis added a commit to Jag-Marcel/Scribe-iOS that referenced this issue Aug 15, 2024
@andrewtavis
Copy link
Member Author

Closed by #484 :) Sooo looking forward to getting the translation process set and then getting this keyboard into the App Store! Thanks for all the hard work, @Jag-Marcel! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request help wanted Extra attention is needed new keyboard Adding a new language keyboard
Projects
Archived in project
Development

No branches or pull requests

5 participants