added support for sim output with multiple sims

[gidden]

not perfect, but starts to address #5

before this patch:

$ cyan -db ~/work/publications/dre/scenarios/cyclus.sqlite sims
SimId                                Duration Handle      Decay  
fdf043b6-b0c9-4055-8773-6bb9cfaf0ace 600      dre_recycle manual 

after:

$ ./cyan -db ~/work/publications/dre/scenarios/cyclus.sqlite sims
SimId                                Duration Handle      Decay  
fdf043b6-b0c9-4055-8773-6bb9cfaf0ace 600      dre_recycle manual 
SimId                                Duration Handle   Decay  
bbda8c4f-8156-4ceb-9fd5-6fbfad05184c 600      dre_base manual

this uses the basic doCustom() plumbing thus repeats the headers. it could be cleaned up to not repeat headers and get better spacing. or a better sql command could be used to loop over all simids in doSims(). probably the second is a better choice, but my go + sql-foo is nonexistant + rusty.

[rwcarlsen]

I had just flubbed on the sql query - it shouldn't have been filtering
based on the first simid in the db like it does for other commands
[facepalm]. Fixed in 5c6700b. Thanks for
the issue report!

On Thu, Aug 20, 2015 at 2:09 PM, Matthew Gidden [email protected]
wrote:

not perfect, but starts to address #5
#5

before this patch:

$ cyan -db ~/work/publications/dre/scenarios/cyclus.sqlite sims
SimId Duration Handle Decay
fdf043b6-b0c9-4055-8773-6bb9cfaf0ace 600 dre_recycle manual

after:

$ ./cyan -db ~/work/publications/dre/scenarios/cyclus.sqlite sims
SimId Duration Handle Decay
fdf043b6-b0c9-4055-8773-6bb9cfaf0ace 600 dre_recycle manual
SimId Duration Handle Decay
bbda8c4f-8156-4ceb-9fd5-6fbfad05184c 600 dre_base manual

this uses the basic doCustom() plumbing thus repeats the headers. it
could be cleaned up to not repeat headers and get better spacing. or a
better sql command could be used to loop over all simids in doSims().
probably the second is a better choice, but my go + sql-foo is nonexistant

+ rusty.

You can view, comment on, or merge this pull request online at:

#6
Commit Summary

• added support for sim output with multiple sims

File Changes

• M cmd/cyan/main.go
  https://github.com/rwcarlsen/cyan/pull/6/files#diff-0 (13)

Patch Links:

• https://github.com/rwcarlsen/cyan/pull/6.patch
• https://github.com/rwcarlsen/cyan/pull/6.diff

—
Reply to this email directly or view it on GitHub
#6.

[gidden]

Gah, I should have been able to figure that out. sorry =(

On Sat, Aug 22, 2015 at 6:14 PM, Robert Carlsen [email protected]
wrote:

I had just flubbed on the sql query - it shouldn't have been filtering
based on the first simid in the db like it does for other commands
[facepalm]. Fixed in 5c6700b. Thanks for
the issue report!

On Thu, Aug 20, 2015 at 2:09 PM, Matthew Gidden [email protected]
wrote:

not perfect, but starts to address #5
#5

before this patch:

$ cyan -db ~/work/publications/dre/scenarios/cyclus.sqlite sims
SimId Duration Handle Decay
fdf043b6-b0c9-4055-8773-6bb9cfaf0ace 600 dre_recycle manual

after:

$ ./cyan -db ~/work/publications/dre/scenarios/cyclus.sqlite sims
SimId Duration Handle Decay
fdf043b6-b0c9-4055-8773-6bb9cfaf0ace 600 dre_recycle manual
SimId Duration Handle Decay
bbda8c4f-8156-4ceb-9fd5-6fbfad05184c 600 dre_base manual

this uses the basic doCustom() plumbing thus repeats the headers. it
could be cleaned up to not repeat headers and get better spacing. or a
better sql command could be used to loop over all simids in doSims().
probably the second is a better choice, but my go + sql-foo is
nonexistant

+ rusty.

You can view, comment on, or merge this pull request online at:

#6
Commit Summary

• added support for sim output with multiple sims

File Changes

• M cmd/cyan/main.go
  https://github.com/rwcarlsen/cyan/pull/6/files#diff-0 (13)

Patch Links:

• https://github.com/rwcarlsen/cyan/pull/6.patch
• https://github.com/rwcarlsen/cyan/pull/6.diff

—
Reply to this email directly or view it on GitHub
#6.

—
Reply to this email directly or view it on GitHub
#6 (comment).

Matthew Gidden, Ph.D.
Postdoctoral Associate, Nuclear Engineering
The University of Wisconsin -- Madison
Ph. 225.892.3192