Documentation for sage.numerical.knapsack.knapsack incorrectly specifies default values for the arguments

[maxale]

Documentation says:

sage.numerical.knapsack.knapsack(seq, binary, max=True, value_only=1, solver=False, verbose=None, integrality_tolerance=0)

while in the source code we see:

def knapsack(seq, binary=True, max=1, value_only=False, solver=None, verbose=0,
             *, integrality_tolerance=1e-3):

Discrepancies:

• No default value for binary is specified.
• max=1 vs max=True
• value_only=1 vs value_only=False
• solver=False vs solver=None
• verbose=None vs verbose=0
• integrality_tolerance=0 vs integrality_tolerance=1e-3

[DaveWitteMorris]

This is a duplicate of #39627. Default values in the documentation get messed up (shifted to the wrong place) whenever there are keyword-only arguments.

[maxale]

Indeed, thanks!