search

CACDAR-Project / Ohtuprojekti-kesa2020

GNU Lesser General Public License v3.0
0 stars 1 forks source link

Map sort filter detections #189

Closed kordaniel closed 4 years ago

kordaniel commented 4 years ago

Nodejen listat korvattu iteraattoreilla. Aivan viimeisessä vaiheessa luodaan tupleja listojen sijaan. Samalla hieman muokattu node_detector_controll.py :n receive_img()-metodin iffejä. Tämä tehty 1) tehokkuuden parantamiseksi ja 2) sorttausta/filtteröintiä helpottamaan.

codecov[bot] commented 4 years ago

Codecov Report

Merging #189 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #189   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines           83        83           
=========================================
  Hits            83        83

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f7c99d1...c008979. Read the comment docs.

ArktinenKarpalo commented 4 years ago

Asiakas halusi havainnot "erikseen" julkaistaviksi sen takia, että jos modelit vaativat paljon laskentaa, niin havaintoja aletaan julkaista niistä modeleista joiden läpi kuva on jo vedetty, ennen kuin kuva vedetty kaikkien modeleiden läpi. Käsittääkseni tässä ei nyt enää tapahdu näin, vaan kaikki modelit suoritetaan ennen kuin mitään julkaistaan.

kordaniel commented 4 years ago

Asiakas halusi havainnot "erikseen" julkaistaviksi sen takia, että jos modelit vaativat paljon laskentaa, niin havaintoja aletaan julkaista niistä modeleista joiden läpi kuva on jo vedetty, ennen kuin kuva vedetty kaikkien modeleiden läpi. Käsittääkseni tässä ei nyt enää tapahdu näin, vaan kaikki modelit suoritetaan ennen kuin mitään julkaistaan.

Näin tosiaan toimitaan. Ja hyvä huomio. Yritän muuttaa tuota.

For-loopissahan on toki se ongelma, että jos raskas laskenta tapahtuu loopin ensimmäisessä iteraatiossa, ja lopuissa on kevyempiä malleja, niin kaikki muut detectorit odottavat vielä vuoroaan ja kaikki julkaistaan sitten käytännössä samaan aikaan. Ehkä tuohon voisi miettiä jonkunlaista nodejen järjestämistä laskenta-ajan mukaan, jos ei saada ne pyörimään omissa prosesseissa (vaikka ei meillä kyllä aika taida tuohon riittää).

ArktinenKarpalo commented 4 years ago

Asiakas halusi havainnot "erikseen" julkaistaviksi sen takia, että jos modelit vaativat paljon laskentaa, niin havaintoja aletaan julkaista niistä modeleista joiden läpi kuva on jo vedetty, ennen kuin kuva vedetty kaikkien modeleiden läpi. Käsittääkseni tässä ei nyt enää tapahdu näin, vaan kaikki modelit suoritetaan ennen kuin mitään julkaistaan.

Näin tosiaan toimitaan. Ja hyvä huomio. Yritän muuttaa tuota.

For-loopissahan on toki se ongelma, että jos raskas laskenta tapahtuu loopin ensimmäisessä iteraatiossa, ja lopuissa on kevyempiä malleja, niin kaikki muut detectorit odottavat vielä vuoroaan ja kaikki julkaistaan sitten käytännössä samaan aikaan. Ehkä tuohon voisi miettiä jonkunlaista nodejen järjestämistä laskenta-ajan mukaan, jos ei saada ne pyörimään omissa prosesseissa (vaikka ei meillä kyllä aika taida tuohon riittää).

Totta, jonkun ehdottama idea että modeleille voisi antaa prioriteetin, joiden mukaan järjestys jossa modelit suoritetaan muodostettaisiin vaikutti hyvältä.

kordaniel commented 4 years ago

Lisätty for-looppi jos combined on False: https://github.com/Konenako/Ohtuprojekti-kesa2020/pull/189/commits/c008979cfc266e49bf1c3871e3fff798dec8faaa