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

kontrola toho, co jsem napáchal ( amax() za sum() ) #58

Closed
mjirik opened this issue Jan 29, 2014 · 1 comment
Closed

kontrola toho, co jsem napáchal ( amax() za sum() ) #58

mjirik opened this issue Jan 29, 2014 · 1 comment

Comments

@mjirik
Copy link
Owner

mjirik commented Jan 29, 2014

Zkontrolujte prosím, jestli jsem to nějak nerozbil. Myslím, že ne.

Pozměnil jsem vizualizaci. Nyní se zobrazuje sumární obraz cév, takže je ve výsledku vidět, kde je to tlustší. Vlastně jsem jen narhradil fukci
numpy.amax() funkcí numpy.sum()

Prokrátil jsem funkci updateImage. I teď má přes sto řádek, což není ideální.
Na řádku 418 jsem vytvořil novou funkci, do které jsem přesunul část obsahu z řádky 405 s malými úpravami (amax za sum).

@ghost ghost self-assigned this Jan 29, 2014
@ghost
Copy link

ghost commented Jan 29, 2014

Zkontroloval jsem funkčnost a vše se zdá v pořádku.
Funkci updateImage jsem rozdělil do několika funkcí.

Tato logická proměnná rozhoduje o tom, zda se nastaví experimentální hodnoty binary closing a opening nebo ne. Trošku jsem experimentoval a zavedl jsem nový parametr na vstupu funkce vesselSegmentation() v segmentation.py - a to "smartInitBinaryOperations".
Jak tato proměnná ovlivní počáteční hodnoty pro bin. uzavření a otevření je vidět na následujícím kódu (proměnná seeds jsou vybrané body při volbě seedu neboli prioritních bodů):

if (smartInitBinaryOperations):
if (seeds == None):
closing = 5
opening = 1
else:
closing = 2
opening = 0

ghost pushed a commit that referenced this issue Jan 29, 2014
Checked "sum" visualisation, divided uiThreshold update code to multiple
functions and created new parameter in segmentation:
"smartInitBinaryOperations".
@ghost ghost removed the new info label Feb 20, 2014
@ghost ghost closed this as completed Feb 20, 2014
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant