0506 Demo For Daniel

[sumwmer]

Demo steps:

• make directories in /tmp: /tmp/demo/input
• put any GPM file in /tmp/demo/input: downloaded from here
• run command: docker run --rm -p 5000:5000 -v $(pwd):/ws -v /tmp:/tmp -it mint_dt bash /ws/examples/topoflow4/run_climate.sh
• check /tmp/demo/output: should produce climate_all.rti and climate_all.rts correctly Note: previous config file and adapter variable naming is out of place so I fixed that ;)

[minhptx]

@dgarijo @sirspock please review and let us know if this is what you guys want.

[dgarijo]

Thanks Minh, we will take a look and let you know. One question: why is this the PR blocked? Am I supposed to review and accept it?

[minhptx]

Thanks Minh, we will take a look and let you know. One question: why is this the PR blocked? Am I supposed to review and accept it?

I added you as a reviewer. You can review and approve the the PR.

[dgarijo]

I reviewed it and it looks good. I will give it a try.

[dgarijo]

@summer7xinting @minhptx one question: The output file is specified to be climate_all.rts. However, there is another one generated (climate_all.rti). Without the rti file, the rts won't work. Why is it not then specified as output file?

[minhptx]

@dgarijo I see. I'll ask Xinting to fix this. When Scott asked us to run his transformation, he only need the rts file. That's why we assume only the rts files are needed.

[sumwmer]

I suppose we need to specify output file path and file name (without rts and rti extension)?

[dgarijo]

No, I think both are in tandem. Also, the input to the topoflow model is a zipped file. Is it possible to create it as a zipped file at the end? This is a technicality, but it would make it ready to plug in the model

[sumwmer]

In the case of generating a zipped file, it's reasonable to give a path/climate_all.zip and the adapter will take care of the rest.

[dgarijo]

Is it possible to have it as part of this example?

[sumwmer]

Creating another PR now