jetpacapp / DeepBeliefSDK

The SDK for Jetpac's iOS Deep Belief image recognition framework
Other
2.86k stars 437 forks source link

Fix bug in Buffer.transpose #38

Closed CFAndy closed 9 years ago

CFAndy commented 9 years ago

fix the bug in free, you can skip the fix on NEON.

petewarden commented 9 years ago

Thanks! I've merged in all the changes since the NEON powf looks handy too.

CFAndy commented 9 years ago

I think current DBS is very self contained from software license perspective and stay well under your BSD umbrella. the code on math_neon I ported has GNU license. Better to remove my NEON porting code to avoid GNU license contaminant. In this pull request, I just don't know how to provide a Buffer.transpose only request. Not intend you to merge the NEON fix.

petewarden commented 9 years ago

Thanks for the heads-up, I missed the GPL in the NEON code! I have rolled back the entire pull request for now, but I'll try to cherry-pick just the free bug when I get a chance.