Genann Integration

[AgoeZero]

This is the integration of the genann library github.com/codeplea/genann into the lambda native framework

[mgorges]

Thanks for submitting your feedfordward artificial neural networks module and library. I didn't evaluate it more comprehensively, as there are two considerations to complete first.
a) Your libgenan code includes compiled objects (there is a .o file) and source code (it includes both a .C and .H file) that is not BSD licensed. Hence it would be better to source it from https://github.com/codeplea/genann/releases/tag/v1.0.0 as we do for most other libraries - e.g. see libraries/libsqlcipher/make.sh#L2, rather than including the source files. In either case the ELF 64 bit file should not be included.
b) Next, the module would benefit from unit tests - e.g. see modules/sqlite/sqlite.scm#L189. There should be deterministic output to test for if you set the initial conditions?

[AgoeZero]

Absolutely, The object file is an oversight. I will source the file using the package_ ("namespace"?) and Include unit tests just as in sqlite.scm

…

-Thanks

On Tue, Oct 21, 2025 at 3:22 AM Matthias Görges ***@***.***> wrote: *mgorges* left a comment (part-cw/lambdanative#473) <#473 (comment)> Thanks for submitting your feedfordward artificial neural networks module and library. I didn't evaluate it more comprehensively, as there are two considerations to complete first. a) Your libgenan code includes compiled objects (there is a .o file) and source code (it includes both a .C and .H file) that is not BSD licensed. Hence it would be better to source it from https://github.com/codeplea/genann/releases/tag/v1.0.0 as we do for most other libraries - e.g. see libraries/libsqlcipher/make.sh#L2 <https://github.com/part-cw/lambdanative/blob/master/libraries/libsqlcipher/make.sh#L2>, rather than including the source files. In either case the ELF 64 bit file should not be included. b) Next, the module would benefit from unit tests - e.g. see modules/sqlite/sqlite.scm#L189 <https://github.com/part-cw/lambdanative/blob/master/modules/sqlite/sqlite.scm#L189>. There should be deterministic output to test for if you set the initial conditions? — Reply to this email directly, view it on GitHub <#473 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BS3AGQFS4PRDFVL3HMAPVVL3YWKGHAVCNFSM6AAAAACJXPHYUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIMRUGQYTMMJWHA> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[bkmgit]

@AgoeZero Thanks for adding this
@mgorges Thanks for reviewing

[AgoeZero]

Its done. libgenann now sources from the genann repository genann-test also included in genann.scm

…

On Wed, Oct 22, 2025 at 4:31 AM Benson Muite ***@***.***> wrote: *bkmgit* left a comment (part-cw/lambdanative#473) <#473 (comment)> @AgoeZero <https://github.com/AgoeZero> Thanks for adding this @mgorges <https://github.com/mgorges> Thanks for reviewing — Reply to this email directly, view it on GitHub <#473 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/BS3AGQHACFHEKEUWIPWNRFT3Y33BLAVCNFSM6AAAAACJXPHYUWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTIMZQGM2TOOJUHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[bkmgit]

@mgorges Will you be able to review this soon?