{"comment":{"url":"https://api.github.com/repos/mareuter/sims_catalogs_generation/comments/10926146","html_url":"https://github.com/mareuter/sims_catalogs_generation/commit/2b5ee8dd30e9cc91dfeda13bb38141a9a174d503#commitcomment-10926146","id":10926146,"user":{"login":"mareuter","id":420047,"avatar_url":"https://avatars.githubusercontent.com/u/420047?v=3","gravatar_id":"","url":"https://api.github.com/users/mareuter","html_url":"https://github.com/mareuter","followers_url":"https://api.github.com/users/mareuter/followers","following_url":"https://api.github.com/users/mareuter/following{/other_user}","gists_url":"https://api.github.com/users/mareuter/gists{/gist_id}","starred_url":"https://api.github.com/users/mareuter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mareuter/subscriptions","organizations_url":"https://api.github.com/users/mareuter/orgs","repos_url":"https://api.github.com/users/mareuter/repos","events_url":"https://api.github.com/users/mareuter/events{/privacy}","received_events_url":"https://api.github.com/users/mareuter/received_events","type":"User","site_admin":false},"position":2,"line":null,"path":"python/lsst/sims/catalogs/generation/db/ObservationMetaData.py","commit_id":"2b5ee8dd30e9cc91dfeda13bb38141a9a174d503","created_at":"2015-04-27T20:56:57Z","updated_at":"2015-04-27T20:56:57Z","body":"(Simon Krughoff) If you are just using the haversine formula, I prefer:\nfrom .observationMetadataUtils import haversine\n\nIt makes it clearer where the method is coming from.\n\nI also noticed that the import of observationMetadataUtils was not removed from the __init__.py file in sims_catUtils.  Can you remove that import before you merge?"}}