Skip to content

[FEA]: Deprecate ResponseMemoryProbs & MultiResponseProbsMessage in favor of ResponseMemory & MultiResponseMessage #697

Closed

Description

Is this a new feature, an improvement, or a change to existing functionality?

Change

How would you describe the priority of this feature request

Low (would be nice)

Please provide a clear description of problem this feature solves

MultiResponseMemoryProbs doesn't do much differently than MultiResponseMemory other than enforce that at least one tensor is named "probs" however we want to be a little more flexible and

This would be a breaking change.

Describe your ideal solution

n/a

Describe any alternatives you have considered

No response

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
  • I have searched the open feature requests and have found no duplicates for this feature request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Assignees

Labels

Type

No type

Projects

  • Status

    Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions