New ROI objects created when editing existing ROIs? about soma HOT 6 CLOSED

@jayyoung Yes now we also store the history of ROIs as @kunzel requested. Luckily I have also created a service for querying ROIS. You can use the service soma/query_rois for that. But I think there is no option to get the latest roi for each id. I will see how it can be done. Thanks for pointing this out.

from soma.

Yeah that's the service I'm using, I updated everything to use the new methods, but noticed this behaviour.

It'd be really nice to be able to just query the most recent version of all the ROIs with a flag in the service call or something in order to get the current state. Or maybe instead of inserting a new entry into the database entirely when an ROI is updated, just keep the same database entry, but update a vector of the previous poses. So roi.poses[0] gets me the most recent posearray of the ROI, roi.poses[1] gets me the previous posearray, and so on. Not sure what's easiest/best, probably just a flag in the service call.

from soma.

@jayyoung I have updated the roi query service with the flag returnmostrecent which will return you the most recent versions of rois when set to true. I have also opened a pull request for mongodb_store because the latest version of soma needs to have the updated version of mongodb_store. For an immediate solution you can use this version https://github.com/hkaraoguz/mongodb_store/tree/somaroiupdate

from soma.

Woo awesome! Thank you!

from soma.

@jayyoung Did you check the functionality? Should we close this issue?

from soma.

Yep sure sorry I only just got around to testing it. It seems to work just fine. Here is the test code I used to verify it in case you care:

import roslib
import rospy
from soma_msgs.msg import SOMAObject
from soma_manager.srv import *


if __name__ == '__main__':
    rospy.init_node('test_masks', anonymous = False)
    print("getting soma service")
    rospy.wait_for_service('/soma/query_rois')
    print("done")
    print("setting up proxy")
    soma_query = rospy.ServiceProxy('/soma/query_rois',SOMAQueryROIs)
    print("done")
    print("making query")

    query = SOMAQueryROIsRequest()
    query.returnmostrecent = True
    response = soma_query(query)

    print("found: " + str(len(response.rois)) + " rois using return_most_recent")
    for roi in response.rois:
        print(roi.id)

from soma.