Přidání checkboxů u data fields pro hromadné přidání - Githubissues

KIZI / EasyMiner-MiningUI

Web UI for mining in new version EasyMiner Project

Apache License 2.0

3 stars 1 forks source link

Přidání checkboxů u data fields pro hromadné přidání #75

Closed PreVaDu closed 8 years ago

PreVaDu commented 8 years ago

Dle domluvy jsem nachystal první verzi grafického řešení spolu s možnou funkčností checkboxů pro hromadné přidání data fields do atributů. Inspiroval jsem se použitým řešením u found rules atd. Momentálně vše končí tak, že se do konzole vypisují IDčka checkboxů. Budete chtít do backendu obdržet čisté názvy, nebo si je od suffixů očistíte? Ono není asi moc ideální operovat s názvem, ale v APFieldu nevidím nic unikátního, dle čeho to rozpoznávat...

vojir commented 8 years ago

Doplňte prosím možnost vybrat vše/invertovat výběr (jako u checkboxů v rámci discovered rules). Budeme pracovat s názvy, odstraňte z nich prosím tu část "-checkbox"

vojir commented 8 years ago

ještě prosím upravte zarovnání checkboxů s ohledem na textové názvy data fields - aktuálně začíná checkbox "pod linkou"

PreVaDu commented 8 years ago

upraveno, opraveno.

vojir commented 8 years ago

Identifikace prostřednictvím ID daného inputu nefunguje korektně!

hodnotou ID by neměl být jen název sloupce, měla by být využita nějaká předpona (jakožto prevence konfliktních ID v rámci aplikace)
identifikační ID je v některých případech odlišné od názvu sloupce - například u sloupce "birth_year" je vracena hodnota "birthyear"

PreVaDu commented 8 years ago

1) Z toho důvodu tam byl původně sufix "-checkbox". Dle připomínek byl odstraněn. Bude v rámci datafields název vždy unikátní? Pak by takový sufix měl stačit. 2) Lze mi přiřadit miner, který daný sloupec používá? Abych vyzkoušel přímo na něm (a nemohlo dojít k jinému chování simulováním s jinými daty). Děkuji.

vojir commented 8 years ago

v data fields jsou jednotlivé názvy vždy unikátní
minery již mezi uživateli vzájemně přehazovat nejdou, birth_year je jeden z výchozích sloupců z loans.csv
problémem není přidání prefixu či suffixu do ID, ale nemůže být následně součástí vracených názvů sloupců...

problém je pravděpodobně způsoben již samotným použitím "id" jakožto identifikátoru... Lepší by patrně bylo buď použití proměnné v JS (jako je tomu například u zobrazení histogramů atp.), nebo použití atributu data-*

vojir commented 8 years ago

Ještě jeden problém - ovládací prvky pro výběr všech či žádného sloupce pro předzpracování se zobrazí až při ručním vybrání některého ze sloupců. Správně by se měly zobrazovat hned při aktivaci multivýběru...

datafields

po kliknutí na hromadné předzpracování by následně již neměly být vybrány dříve vybrané položky a mělo by dojít ke skrytí hromadných ovládacích prvků

PreVaDu commented 8 years ago

Děkuji za feedback. Všechny chyby by měly být opraveny.