Skip to content

fixed export Dense layer class and add exports in each fold.#4

Open
MarioCarlosChita wants to merge 2 commits into
vxern:mainfrom
MarioCarlosChita:main
Open

fixed export Dense layer class and add exports in each fold.#4
MarioCarlosChita wants to merge 2 commits into
vxern:mainfrom
MarioCarlosChita:main

Conversation

@MarioCarlosChita

Copy link
Copy Markdown

Some classes were not being exported in was not possible to add for example Dense class Layer to sequential NeuralNetworks.

@vxern

vxern commented Feb 24, 2025

Copy link
Copy Markdown
Owner

Thank you for the PR, I'll have a look today when I get a free moment and merge your changes in.

@MarioCarlosChita

MarioCarlosChita commented Feb 24, 2025 via email

Copy link
Copy Markdown
Author

@vxern

vxern commented Feb 24, 2025

Copy link
Copy Markdown
Owner

What do you mean by 'Or port'? @MarioCarlosChita

@MarioCarlosChita

MarioCarlosChita commented Feb 24, 2025 via email

Copy link
Copy Markdown
Author

Comment thread lib/synadart.dart
export 'src/activation.dart';

export 'src/networks/sequential.dart';
export 'src/export.dart';

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead of having a separate export.dart file per each directory, could we just export everything relevant in lib/synadart.dart?

@@ -0,0 +1,3 @@
export 'network.dart';
export 'sequential.dart';
export 'training/backpropagation.dart';

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please format before submitting.

@vxern

vxern commented Feb 25, 2025

Copy link
Copy Markdown
Owner

@MarioCarlosChita I'm still not quite sure what you mean, could you write it in your native language?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants