oliverpetschick commented 3 years ago

Hallo Jörg,

wie gestern gesprochen hier mein Code um mit der model_run methode container auszuführen. Ich hab auch einen Test geschrieben und soweit alles ausprobiert. Für jegliches Feedback bin ich dankbar - habe vorher noch nie an einem großen Projekt gearbeitet.

PS: In älteren Containern muss singularity mit dem Befehl singularity exec --app ogs [..] ausgeführt werden (anstatt singularity exec [..]). Ob dies in einen try/except Block eingebaut werden soll, oder ob der Nutzer am Ende nur die neusten Container (6.4.x aufwärts) benutzen kann - ich war mir nicht sicher.

Beste Grüße Oliver

oliverpetschick commented 3 years ago

I removed the redundand boolean and added a string check for the specific runtime errors.

joergbuchwald commented 3 years ago

Can you remove all trailing white spaces in addition?

codecov[bot] commented 3 years ago

Codecov Report

Merging #19 (1fb423a) into master (36cd401) will decrease coverage by 1.14%. The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
- Coverage   64.59%   63.45%   -1.15%     
==========================================
  Files          18       18              
  Lines        1110     1130      +20     
==========================================
  Hits          717      717              
- Misses        393      413      +20

Impacted Files	Coverage Δ
ogs6py/ogs.py	`72.96% <0.00%> (-4.51%)`	:arrow_down:

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 36cd401...1fb423a. Read the comment docs.

joergbuchwald commented 3 years ago

I will add the tests to the pipeline later on, this will improve the code coverage.

joergbuchwald / ogs6py

run_model um container methode erweitert #19

Codecov Report