Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refact: Redefintion of InstType, InstTypeNew, InstTypeDel #1074

Closed
5 of 6 tasks
detlefarend opened this issue Oct 29, 2024 · 0 comments · May be fixed by #1043
Closed
5 of 6 tasks

Refact: Redefintion of InstType, InstTypeNew, InstTypeDel #1074

detlefarend opened this issue Oct 29, 2024 · 0 comments · May be fixed by #1043
Assignees
Labels
BF Basic Functions/Infrastructure next release quality Quality assurance refactoring Restructuring, renaming, simplification, understandability, ...

Comments

@detlefarend
Copy link
Member

detlefarend commented Oct 29, 2024

Motivation
The data types mentioned in sub-package mlpro.bf.streams shall be redefined to enable better diagnostics/debugging.

Todo

  • InstType: int -> str
  • InstTypeNew: 0 -> '+'
  • InstTypeDel: 1 -> '-'

Benefit
Better diagnostics/debugging of stream processes.

Are the proposed changes backward compatible?

  • Yes
  • No

Related development objects

  • mlpro.bf.streams: InstType, InstTypeNew, InstTypeDel
@detlefarend detlefarend added BF Basic Functions/Infrastructure next release quality Quality assurance refactoring Restructuring, renaming, simplification, understandability, ... labels Oct 29, 2024
@detlefarend detlefarend self-assigned this Oct 29, 2024
@detlefarend detlefarend linked a pull request Oct 29, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BF Basic Functions/Infrastructure next release quality Quality assurance refactoring Restructuring, renaming, simplification, understandability, ...
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant