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.

Sign up for GitHub

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

Jump to bottom

added wkt representation for chain/polygon #914

Closed
wants to merge 1 commit into from
Closed

added wkt representation for chain/polygon #914

wants to merge 1 commit into from

Conversation

parulsethi
Copy link

Hi,
This PR is to address the issue-812.

It adds __repr__() method for Chain and Polygon following convention similar to this. Please suggest for anymore changes/additions required.

@coveralls
Copy link

coveralls commented Feb 26, 2017 •
edited

Coverage Status

Coverage decreased (-0.04%) to 83.038% when pulling 7a6b0bf on parulsethi:wkt_formats into 7d59a95 on pysal:master.

2 similar comments
@coveralls
Copy link

coveralls commented Feb 26, 2017 •
edited

Coverage Status

Coverage decreased (-0.04%) to 83.038% when pulling 7a6b0bf on parulsethi:wkt_formats into 7d59a95 on pysal:master.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 83.038% when pulling 7a6b0bf on parulsethi:wkt_formats into 7d59a95 on pysal:master.

@sjsrey
Copy link
Member

sjsrey commented Mar 5, 2017

Thanks for this PR.
Please open pull requests against the dev branch as described in our developer docs.

@sjsrey sjsrey closed this Mar 5, 2017
@parulsethi
Copy link
Author

Oh sorry for this. opened another PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@parulsethi @coveralls @sjsrey