Skip to content
This repository has been archived by the owner on Jan 3, 2024. It is now read-only.

[馃敺 Feature request ]: Aliases for feedForward & backpropagate #26

Closed
1 of 10 tasks
matiasvlevi opened this issue Jun 25, 2021 · 2 comments
Closed
1 of 10 tasks
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers Priority: Medium

Comments

@matiasvlevi
Copy link
Owner

matiasvlevi commented Jun 25, 2021

Feature

We could have aliases for the feedForward & backpropagate methods of the Dann class.

Type

  • Dann
  • Matrix
  • Layer
  • Activation functions
  • Loss functions
  • Pool functions
  • Datasets
  • Documentation
  • tests & examples
  • Other

Description

These aliases would be to make method names a little more uniform when we add in the Rann class for RNNs as feedForward & backpropagate would not be the most accurate terms for RNNs. See the issue on RNNs here.

Feedforward

Is located in src/classes/dann/methods/feedForward.js

backpropagate

Is located in src/classes/dann/methods/backpropagate.js

Examples

These would be the aliases
Dann.prototype.backpropagate to Dann.prototype.train
Dann.prototype.feedForward to Dann.prototype.feed

Note

It is important to note that we do not want to remove backpropagate & feedForward names, I think having machine learning terms for methods helps to get a grasp at what the neural network is doing since you can look up the terms.

@matiasvlevi matiasvlevi added enhancement New feature or request good first issue Good for newcomers Priority: Medium labels Jun 25, 2021
@SharkAce
Copy link
Contributor

I'm on it man

@matiasvlevi
Copy link
Owner Author

Just merged your fork! Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request good first issue Good for newcomers Priority: Medium
Projects
None yet
Development

No branches or pull requests

2 participants