Skip to content
This repository has been archived by the owner on Sep 1, 2023. It is now read-only.

Update sm_1d_test to use Monitor Mixin #75

Closed
chetan51 opened this issue Oct 19, 2014 · 4 comments
Closed

Update sm_1d_test to use Monitor Mixin #75

chetan51 opened this issue Oct 19, 2014 · 4 comments
Assignees

Comments

@chetan51
Copy link
Contributor

It's currently using the old TemporalMemoryInspectMixin.

@BoltzmannBrain
Copy link
Contributor

I think sm_1d_test may need an overhaul to incorporate the following:

@chetan51
Copy link
Contributor Author

chetan51 commented Apr 8, 2015

I'm not sure whether those tests are still valuable or not. Probably a question for @subutai, who I believe wrote some of them.

@subutai
Copy link
Member

subutai commented Apr 9, 2015

I'm fine if we delete it. I don't have any long term allegiance to research code. It's actually easier if we have fewer unit tests and only write those that help us debug the functionality.

@BoltzmannBrain
Copy link
Contributor

Sounds good to me. I'll delete it in my next push for #149.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants