-
Notifications
You must be signed in to change notification settings - Fork 183
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
Remove SD #1094
Remove SD #1094
Conversation
Please, also for this one, wait for the next Espresso meeting
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requires offline discussion
I heard that in the meeting a few weeks ago SD was scheduled for removal. What is blocking the merge of this one? |
The duplicated main loop that comes with SD is very hard to maintain and has already diverged quiet a bit from the other stuff. If @dschwoerer or somebody else is willing to integrate the method with the normal main loop I will remove it. |
I am slightly sad to see SD go. I am not using espresso any more, and if there is nobody else who uses it ... If there would be use for the SD/BD integrators, I could be bothered, but I don't think that this is currently the case ... |
Can this be merged? |
We would also be more happy if we had SD, but we can not fix it at the moment. The code is not lost, and if we find a student in the future we will pick this up again. |
Fixes #233, #624, and #1091