PublicKey should derive Hash

[inetic]

Re-opening the issue #120 as it hasn't been solved yet.

The original author @dvc94ch seems to open and close issues because it fits his workflow.

The original text:

@dvc94ch wrote

Unless there's a cryptographic reason why it should not derive Hash, it would be a good idea.

@burdges wrote

It should define Hash with .0.as_bytes() not #[derive(Hash)] since the serialized form already exists, and because points in protective coordinates require normalization. Aside from the obvious reasons: Projective Coordinates Leak

There is currently a PR for this #176. Likely motivated by @burdges' comment here. The CI on that PR is currently failing, but it doesn't seem like the failure is caused by the PR changes.

Could someone please re-run the CI?

[dvc94ch]

looks like I may have closed it accidentally

[mimoo]

wondering about this as well

[rozbb]

Resolved in #265