LPIPS source comments are reversed?

[torridgristle]

At

PerceptualSimilarity/lpips/lpips.py

Lines 52 to 54 in ecfae6f

	pnet_tune
	[False] tune the base/trunk network
	[True] keep base/trunk frozen

it seems like line 53 should be [False] and line 54 should be [True] since enabling it would enable grad, but the comment says that setting it to true would freeze it.

[richzhang]

Thanks, you're right, it was mixed up. I switched them: https://github.com/richzhang/PerceptualSimilarity/blob/master/lpips/lpips.py#L52.

[torridgristle]

It may be pedantic, but the order should probably be True, False in order to match the others. Personally I tend to expect patterns in things and I may assume the first one is true and the second is false even if it's explicitly labelled the other way around, and I reckon other folks could make that same mistake when glossing over it.

[richzhang]

Thanks, the default parameters are listed first