Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fully Connected PolyPENN - PolyFCP #11

Open
anthonio9 opened this issue Jun 17, 2024 · 3 comments
Open

Fully Connected PolyPENN - PolyFCP #11

anthonio9 opened this issue Jun 17, 2024 · 3 comments
Assignees

Comments

@anthonio9
Copy link
Owner

anthonio9 commented Jun 17, 2024

PENN is not implemented as a Fully Convolutional Network, however it is rather easy to convert it to one.
The configuration parameter NUM_TRAINING_FRAMES is defining the length of the training sequence, by default set to 1. In the default case the training procedure is provided with unique, independent frames.

@anthonio9 anthonio9 self-assigned this Jun 17, 2024
anthonio9 added a commit that referenced this issue Jun 17, 2024
anthonio9 added a commit that referenced this issue Jun 20, 2024
@anthonio9
Copy link
Owner Author

FCN option still can't be properly evaluated as multi-pitch metrics are not supporting the new shape of data.
Both postprocess_pitch_and_sort and the local expected value need to be fixed to support the FCN output.

@anthonio9
Copy link
Owner Author

anthonio9 commented Jul 5, 2024

Maybe try residual connections recurrent blocks in the last layer? or a hierarchical set of output layers, one for per each string?

Hierarchical:
hierarchicalCNN

@anthonio9
Copy link
Owner Author

anthonio9 commented Jul 7, 2024

Sadly, the hierarchical approach does not seem to yield better results, perhaps my approach could have some more polishing, but still it does not seem to be the one. What about the recurrent blocks? Well, that part has to wait a bit as a FCN RNNs are not that obvious to me at the moment.
Not true, there was a big bug in the hierarchical approach code (regular PolyPENNFCN was launched instead of the hierarchical variant), now setting this up properly is a bit tough and will take some time.

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

When branches are created from issues, their pull requests are automatically linked.

1 participant