Skip to content

Comments

Fail when loss_param is used with EuclideanLoss layer #4920

Open
BlGene wants to merge 1 commit intoBVLC:masterfrom
BlGene:loss_layer_fix
Open

Fail when loss_param is used with EuclideanLoss layer #4920
BlGene wants to merge 1 commit intoBVLC:masterfrom
BlGene:loss_layer_fix

Conversation

@BlGene
Copy link
Contributor

@BlGene BlGene commented Oct 29, 2016

...as this layer does not implemente the functionality specified by the parameter,
although this might plausibly be expected.

@shelhamer shelhamer changed the title Fail when loss_param is used with EuclidanLoss layer Fail when loss_param is used with EuclideanLoss layer Jan 18, 2017
@shelhamer
Copy link
Member

@BlGene care to fix this PR? This is still a helpful check to have.

@BlGene
Copy link
Contributor Author

BlGene commented Feb 17, 2017

@shelhamer: its not clear to me why the tests are failing, in all cases its numerical deviation in AdaDeltaSolverTest. As this is something my code doesn't touch I would hazard a guess that the tests are broken.

does not implemente the functionality specified by the parameter,
although this might plausibly be expected.
@Noiredd
Copy link
Member

Noiredd commented Nov 2, 2017

Try to rebase this, @BlGene. #5973 (among others) introduced some changes to Travis that might eradicate the unrelated failures.

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.

3 participants