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

Obsolete code scripts and fragments #33

Open
alexlafleur opened this issue Nov 24, 2016 · 2 comments
Open

Obsolete code scripts and fragments #33

alexlafleur opened this issue Nov 24, 2016 · 2 comments
Assignees

Comments

@alexlafleur
Copy link
Collaborator

alexlafleur commented Nov 24, 2016

ldshmm/util/util_evaluation_mm is the update on the "old" script. The difference between these two is that the new version can separate between bayes and naive while the old one always evaluates both.

Can the "old" version be deleted?

Furthermore, we adapted the simulate_and_store method, but still kept the old versions (simulate_and_store_data ... ). Can this old version be removed, so that we adapt our existing codes to use the new version?

@alexlafleur alexlafleur changed the title Can util_evaluation_mm_old be deleted? Obsolete code scripts and fragments Nov 24, 2016
@greenTara
Copy link
Collaborator

In the case that we want to compare the bayes and naive methods, we need to be able to coordinate the data (set the random seed) as well as the scale of the heatmaps or plots. If that can be done using the scripts that work independently, then we can delete the script that does both together.

@alexlafleur
Copy link
Collaborator Author

Okay, then I delete the _old script now.

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

No branches or pull requests

2 participants