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.

Sign up for GitHub

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

Jump to bottom

Fix FL loss #257

Merged

AlexandraVolokhova merged 1 commit into new_fl_loss from fl-loss-ahg

Nov 27, 2023

Owner

alexhernandezgarcia commented Nov 27, 2023

Perhaps we may want to squeeze the output of self.state_flow within the state flow class directly.


          Fix bug in FL loss (logflows needed to be squeezed); now it seems to …

391c304

…work like a charm

alexhernandezgarcia requested a review from AlexandraVolokhova

November 27, 2023 23:05

AlexandraVolokhova approved these changes

View reviewed changes

Collaborator

AlexandraVolokhova left a comment

Thank you for finding this subtle bug!

AlexandraVolokhova merged commit e24f966 into new_fl_loss

1 check passed

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

Labels

None yet