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

str and repr for mappymatch objects #111

Closed
wants to merge 6 commits into from
Closed

Conversation

machallboyd
Copy link
Collaborator

Addresses issue [#35], adding string representations to objects

Before opening PR for approval, could use feedback on what's useful string representations of objects. Currently on this branch:

str(matcher)
"LCSSMatcher({'road_map': NXMap('(g, MultiDiGraph with 1239 nodes and 3417 edges), (crs, epsg:3857), (_dist_weight, kilometers), (_time_weight, travel_time), (_geom_key, geometry), (_road_id_key, road_id), (rtree, rtree.index.Index(bounds=[-11684967.812039806, 4815101.583056703, -11678639.588418882, 4829602.061216707], size=3417)), (_nodes, list of 1239 elements), (_roads, list of 3417 elements)'), 'distance_epsilon': 50.0, 'similarity_cutoff': 0.9, 'cutting_threshold': 10.0, 'random_cuts': 0, 'distance_threshold': 10000})"
repr(matcher.road_map)

'NXMap(g: MultiDiGraph with 1239 nodes and 3417 edges, crs: epsg:3857, _dist_weight: kilometers, _time_weight: travel_time, _geom_key: geometry, _road_id_key: road_id, rtree: rtree.index.Index(bounds=[-11684967.812039806, 4815101.583056703, -11678639.588418882, 4829602.061216707], size=3417), _nodes: list of 1239 elements, _roads: list of 3417 elements)'

@machallboyd machallboyd changed the title Feature/35 str and repr str and repr for mappymatch objects May 11, 2022
@machallboyd
Copy link
Collaborator Author

Going back to the drawing board.

@nreinicke nreinicke deleted the feature/35-str-and-repr branch August 10, 2023 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant