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.

Sign up for GitHub

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

Jump to bottom

measure.py vs cantherm.py #120

Closed
faribas opened this issue Jun 11, 2013 · 4 comments
Closed

measure.py vs cantherm.py #120

faribas opened this issue Jun 11, 2013 · 4 comments
Labels
Type: Risk of Error

Comments

@faribas
Copy link
Contributor

faribas commented Jun 11, 2013

measure.py doesn't seem to work (various errors in measure/input.py). Has it been entirely superseded by canterm.py? If so, should measure.py (and measure/input.py) be deleted? (or else fixed?)
@connie
Copy link
Member

connie commented Jun 11, 2013

Yes, Measure has been completely deprecated and replaced by CanTherm. We should perform some cleanup soon...

@connie
Copy link
Member

connie commented Aug 19, 2013

Closed by 5b17f78

@connie connie closed this as completed Aug 19, 2013
@rwest
Copy link
Member

rwest commented Aug 27, 2013

Apparently this SHOULD be deprecated, as it has been supplanted by CanTherm for stand-alone jobs, but is still imported and used by RMG-Py.
Someone should change all the imports so RMG-Py uses CanTherm and MEASURE can be deleted.

@rwest rwest reopened this Aug 27, 2013
@connie
Copy link
Member

connie commented Jul 16, 2015

Hopefully completely closed by #410

@connie connie closed this as completed Jul 16, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Risk of Error
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@rwest @connie @faribas