PieterTack / polycap

Polycapillary X-ray raytracing
GNU General Public License v3.0
3 stars 3 forks source link

tests/source.c: added transmission efficiency numeral assertion #36

Closed PieterTack closed 6 years ago

PieterTack commented 6 years ago

Hm, kunnen we ergens zien welke waarden die checks uitkwamen bij hun source checks? Kunnen we mss gebruiken om relevantere waarden te bepalen... Mss is 0.5% nauwkeurigheid beetje te optimistisch... Zeker bij 5000 fotonen simuleren (kan er ook 50k simuleren, maar gaat langer en langer duren natuurlijk, vooral omdat ik het tijdens de tests single threaded laat doen, en had op eerste zicht niet zo'n heel grote invloed op stabiliteit naar ik heel kort testte)

tschoonj commented 6 years ago

Ik zou wat meer fotonen simuleren dan, zolang de tests niet meer duren dan een minuut of zo lijkt mij dat geen probleem.

Ook zou ik het multithreaded doen: dat zou geen enkele invloed mogen hebben op het resultaat, tenzij er een bug inzit natuurlijk...

PieterTack commented 6 years ago

looks like it'll pass the checks now. Plz merge :)

Alsook, hoe ga ik dan terug naar mijn laatste project (die getweights), maar zodanig dat deze test ook al geimplementeerd is? Is dat dan gewoon git checkout getweights en dan git pull?

tschoonj commented 6 years ago

Doe nu:

git fetch
git checkout getweights
git rebase origin/master # normaal zou dit moeten lukken zonder conflicts
git push -f
...werk verder, commit etc
git push