Skip to content

'prob' and 'prob_weights' argument names in Samplers #2334

Description

@lauramurgatroyd

Description

Sampler.random_with_replacement takes 'prob' not 'prob_weights'. Whereas SPDHG takes 'prob_weights' and Sampler.from_function takes prob_weights. Is there a reason for this difference? Or should they all be called the same name? I found it confusing.

See discussion at: TomographicImaging/CIL-Demos#286 (comment)

Environment

import cil, sys
print(cil.version.version, cil.version.commit_hash, sys.version, sys.platform)

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions