knicos / genai-tm Goto Github PK

When user chooses different language, default class titles stay in original language.

Titles will be in chosen language after browser refresh.

Behaviour is confusing, could it be changed so, that language of default class titles were follow chosen language setting?

While you've cleared out a lot of 3rd party trackers, there's one remaining. fonts.gstatic.com serves Google's openly licensed fonts, but also acts as a tracker. You should replace those fonts with locally hosted font files.
This issue applies to genai-tm and somekone.

Social media preview image and description is missing. This is "big" issue because teachers share this tool via social media channels. Now shares show only name of machine, which is not atractive at all.

Add "copy link" feature next to qr code in class menu(s) and feed "add device" dialog.

It's currently available in deploy view but not elsewhere.

Why? I have noticed that students would like to add photos also from their own laptops .

Mobile device: big QR code should be centered (classes, deploy and input qr codes)

Current language menu is not intuitive, because it's "text only", please add country flags to menu! @knicos

Hi!

Users can't pause classification in input and deployment views
Current input and deployment views of the GenAI classifier doesn't have functionality to pause classifier while action cards are being displayed (ie. app is working).

Why it's important to have pause classifier function
The video, images and/or audio should be watchable until the end without being switched to another one. Currently user can stop the camera, which actually will pause classifier, but user doesn't know that it actually is "pause classifier" button.

How to solve this issue?
Add "pause classifier" or just pause button to UX (see photos 1-3). Users can use that functionality to pause classifier (for example, to listen audio to end or watch whole video)

Photo 1: Output view: add new button "pause camera"
Photo 2: Deploy view: Add "pause" graphics inside of camera icon
Photo 3: App view: Add "pause" graphics inside of camera icon

Deploy view: translation(s) misssing. When deploy view is opened e.g. in finnish language version, it's presenting strings in english. Example: "Share to other devices" (actually english version is also missing term "app")

But yep, translation is missing.

LANGUAGE MENU: The languages are not in alphabetical order

Mobile device: deploy view (text next QR code doesnt fit to screen)
My device was android with chrome browser

Share dialog got new function when CLONE code was added to it. In current form it has two functionalitis in menu:

• clone the teachable machine
• use model in external tool

However, share menu doesn't offer intuitive interface which would explain these two different use cases.

I suggest that share menu should have titles as it follows:

TITLE: Share a clone of machine
text text text

TITLE: Use classifier model in external tool
text text text

German language is not available in language menu

Small typo: now it reads "käytä koodia", should be: "käytä koodi"

I understand that implementation of web sites is case spesific and technological choises are different.
However, "only images can be dropped here" error appears also while draging' dropping photo e.g from site like this: https://web.papunet.net/sites/default/files/kuvapankki/20130313/kissi.jpg

URL is pointing to image itself. When that image is opened in browser tab and tried to drag into TM, issue "only images..." will appear.

Is this #bug or feature?

Deploy view: classifier get's stuck and gives error when user moves from rear cam to front cam. Can be same bug than in previous issue.

Copy clone section has term "luokittimestasi" .. it should be "luokittelijastasi"

Some parts of open dialog are not translated into Finnish

@knicos

NOW: "Lataa luokittelija tiedostoksi." <= it's not correct finnish language

SHOULD BE: "Tallenna luokittelija laitteellesi." <= this say: save classifier to device (normal way to express this kind of activity)

Mobile UX: Add space before "training data" text.

Now space is 0px and it's not consistent, because e.g class titles have spacing!

UX Consistency: info box is missing from input / device dialog

It has "start collaboration" button, but no infobox below that (compare with same function in classes)

Deploy view: classifier get's stuck (and gives weird error) if rear camera is not allowed. It should accept missing camera, because images can be also opened from the folder!

Save dialog has confusing information about sharing the file. I would simplify dialog by removing text which tells about sharing.

It would keep only text which describes saving project to computer.

Add description for code in share dialog. @knicos

Current version of text tool in actions is not accessible/usable when photo is added as a part of activity.

Text is quite often impossible to read due to lack of contrast.

I suggest two alternatives:

1. Add dialog for setting background color for text. I suggest two options: black and white.

From contrast point of view it would be enough to fix the issue.

2. Automate that: i don't know how to automate BG color setting, but I still want highlight the possibility 😎

Support for uploading/dragging folders? (student could move/upload folder with images to class)

ISSUE: Privacy notice label and QR code are overlapping in deployment view.
SUGGESTION: align qr code to right

If user doesn't allow webcam to be used, classifier doesn't work. However, it doesn't warn user about this phenomena. This is very confusing for end-users who quite often doesn't allow cam access for webapps.

I suggest that user interface would warn user that he/she should allow webcam access to get classifier running OR maybe better aproach would be to allow classifier to run without webcam access if photo folder or external device is used anyway instead of webcam.

Now disabled webcam access seems to be blocking classifier to run.

Current term in "open" dialog is not intuitive. There is "share code" although it should be "add code"

@knicos

I did test that "make qr code bigger to get also link" feature. However, link what I got was just "https://tm.generation-ai-stn.fi/image/general" ..

So I actually I got just link to main view of the classifier. So, to put in short: link feature doesnt work :)

QR code is very small and located in bottomleft corner, which make it very difficult to share in typical classroom scenarios. Students have had difficulties to use deployment share because of the QR code location.

Suggestion A:
Make button to show BIG QR code in the middle of screen

Suggestion B:
Move whole UI dialog next to main view (classifier action). Remove bottom bar.

In the current form QR UX is not intuitive and usable.

See attached image. Button with text "copy shared" doesn't make sense. When I click it will open clone of machine shared by other user. Right?

So, then, I am not actually person who will copy anything.. What is reason for "copy" function next to copy button? Why I should be able to copy code in that context?

I SUGGEST following change: remove a possibility to "copy" code and reprhase "copy code" text like this: "open clone shared via code" .. or just: "open clone" or just: "use clone code" or just: "use code" :-)