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

Concat/merge command #1

Closed
balhoff opened this issue Apr 7, 2016 · 3 comments
Closed

Concat/merge command #1

balhoff opened this issue Apr 7, 2016 · 3 comments

Comments

@balhoff
Copy link
Member

balhoff commented Apr 7, 2016

The merge command needs implementation.

@balhoff
Copy link
Member Author

balhoff commented Apr 7, 2016

@cmungall should the merge command be more sophisticated than just concatenating lists? For example, check if items in merging lists have the same ID, then put into the resulting list only one copy but merge that item's fields?

@balhoff
Copy link
Member Author

balhoff commented Apr 7, 2016

A very dumb version of this works now, but it just combines all the lists.

@balhoff
Copy link
Member Author

balhoff commented Apr 11, 2016

A more comprehensive merge is implemented now (items in lists with same ID are merged). Should get some testing though.

@balhoff balhoff closed this as completed Apr 11, 2016
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

1 participant