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

Make the tagging of vertices and edges undoable. #33

Closed
tinevez opened this issue Jun 16, 2017 · 5 comments
Closed

Make the tagging of vertices and edges undoable. #33

tinevez opened this issue Jun 16, 2017 · 5 comments

Comments

@tinevez
Copy link
Contributor

tinevez commented Jun 16, 2017

As of now, only properties of the Model are undoable because the recorder are created at model construction time.
TagSets are based on PropertyMaps so they could be made undoable, but right now it does not work because they are not registered in the GraphUndoRecorder

@ctrueden
Copy link
Contributor

ctrueden commented Jun 16, 2017

Any chance we can generalize the undo logic so that all of ImageJ2 can benefit? Or is it too big of a project? See imagej/imagej-common#3

@tpietzsch
Copy link
Contributor

@ctrueden Some notes explaining the mastodon undo framework: https://gist.github.com/tpietzsch/b597b2ad8062d52dcad375a810473523
It has kind of a "hybrid" undo history (as mentioned in imagej/imagej-common#3) so the general ideas seem applicable.
That being said, the mastodon undo framework is in its second (and possibly not final) iteration, still too rough that I would recommend it as a starting point for generalization.

@ctrueden
Copy link
Contributor

ctrueden commented Jun 16, 2017

the mastodon undo framework is in its second (and possibly not final) iteration, still too rough that I would recommend it as a starting point for generalization.

Perhaps we could revisit then after it iterates a couple more times? Please just keep it in mind for the future of ImageJ2, that if you can make a general enough undo framework, we could push it to the SciJava layer in the future.

Or maybe to put another way: Mastodon can serve as a big use case for how flexible a general SciJava undo framework would need to be in order to support its requirements for undo. Reading over your notes, we probably want (some day) a general undo framework in SciJava with plugins. And Mastodon simply implements such a plugin using its scheme described in your gist, no?

Thanks guys!

@tinevez
Copy link
Contributor Author

tinevez commented Mar 2, 2018

@tpietzsch We can close this issue right? You addressed it all in the last weeks if I remember correctly.

@tpietzsch
Copy link
Contributor

👍

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

3 participants