Fix to dictionary order bug.

strands-project / mongodb_store

MongoDB tools for storing and analysing runs of ROS systems.

BSD 3-Clause "New" or "Revised" License

49 stars 72 forks source link

Fix to dictionary order bug. #128

Closed sssmanufacturing closed 7 years ago

sssmanufacturing commented 9 years ago

Fix #124 dictionary order bug when querying mongo with message. Also rounds floats to achieve consistent use of float32 and float64 in msg structures.

strands-jenkins commented 9 years ago

Can one of the STRANDS admins verify this patch? Post a comment containing "ok to test" to enable Jenkins builds for this pull request.

hawesie commented 9 years ago

ok to test

hawesie commented 9 years ago

This breaks something, but I'm not sure how yet. If you run the update_id method (e.g. from the example client), the server falls over creating the id-based query.

Service call failed: service [/message_store/update] responded with an error: error processing request: database error: Can't canonicalize query: BadValue unknown operator: $oid

hawesie commented 7 years ago

Closing until @sssmanufacturing addresses potential issue above.