Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix for cli loaded metrics #23

Merged
merged 1 commit into from
Jan 29, 2015
Merged

fix for cli loaded metrics #23

merged 1 commit into from
Jan 29, 2015

Conversation

scopatz
Copy link
Member

@scopatz scopatz commented Jan 29, 2015

Small and simple.

@scopatz scopatz added Defect A bug in an existing feature that should work and removed Defect A bug in an existing feature that should work labels Jan 29, 2015
@scopatz
Copy link
Member Author

scopatz commented Jan 29, 2015

For @opotowsky

@opotowsky
Copy link
Member

Thanks so much for your quick attention @scopatz!
Alas, after a clean build, I still get the same error as before:

File "./test_cym.py", line 10, in
f2 = evaler.eval('MaterialsSquared')
File "/home/opotowsky/.local/lib/python2.7/site-packages/cymetric/evaluator.py", line 72, in eval
d = d.add_val(m.schema.byte_names[field], rawd[str(field)][i],
KeyError: 'MassSquared'
Segmentation fault (core dumped)

@opotowsky
Copy link
Member

All looks great, thanks again @scopatz

opotowsky added a commit that referenced this pull request Jan 29, 2015
fix for cli loaded metrics
@opotowsky opotowsky merged commit 48145ac into cyclus:master Jan 29, 2015
@scopatz scopatz deleted the hotfix branch January 29, 2015 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Defect A bug in an existing feature that should work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants