Adds hash_any to pyon #247

Merged
merged 3 commits into from Feb 8, 2013

Conversation

Projects
None yet
3 participants
Contributor

lukecampbell commented Feb 8, 2013

No description provided.

Contributor

daf commented Feb 8, 2013

Two thoughts:

  • since this doesn't depend on pyon, consider putting it in putil/ which doesn't auto-gevent-monkey-patch
  • any way to get the numpy import isolated into a smaller section so this is mostly functional without numpy?
Contributor

daf commented Feb 8, 2013

Also if the Import fails, this method should still exist/return something.

blazetopher added some commits Feb 8, 2013

@blazetopher blazetopher Moves hash_any to pupil; localizes import try/except c5fcadf
@blazetopher blazetopher Adds option to use putil.hash_any
Adds optional constructor argument to allow utilization of the putil.hash_any function for key generation
8cdb758
Member

blazetopher commented Feb 8, 2013

Dave - how's it looking now?

Contributor

daf commented Feb 8, 2013

:shipit:

daf merged commit 649466c into ooici:master Feb 8, 2013

blazetopher deleted the lukecampbell:hash branch Feb 8, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment