8000 Oscar Twomey Resolving bug in Concentration Limit treatment - Resolving bug in Concentration Limit treatment by regnosys-prod-user · Pull Request #3690 · finos/common-domain-model · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Oscar Twomey Resolving bug in Concentration Limit treatment - Resolving bug in Concentration Limit treatment #3690

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

Open
wants to merge 5 commits into
base: 6.x.x
Choose a base branch
from

Conversation

regnosys-prod-user
Copy link
Collaborator

No description provided.

@regnosys-prod-user regnosys-prod-user requested a review from a team as a code owner May 6, 2025 11:21
@regnosys-prod-user regnosys-prod-user added the Rosetta Pull requests which can be viewed in Rosetta label May 6, 2025
@regnosys-prod-user
Copy link
Collaborator Author

Resolving bug in Concentration Limit treatment - Resolving bug in Concentration Limit treatment

Background

A bug was identified when adding concentration limits to a collateral treatment

What is being released?

  • Changed cardinality in CollateralCriteriaBase
  • Added override in EligibleCollateralCriteria to enforce the inclusion of a CollateralCriteria

Review directions

Changes can be reviewed in PR: #3690

Note

This comment was generated via Rosetta.


type EligibleCollateralCriteria extends CollateralCriteriaBase: <"Represents a set of criteria used to specify eligible collateral.">
override collateralCriteria CollateralCriteria (1..1)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add a comment to explain this line

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please also prepare the release note

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@regnosys-prod-user
Copy link
Collaborator Author

What is being released?

Adding comment to overridden field.

Note

This comment was generated via Rosetta.

@JayasriR
Copy link
Contributor

superseded by #3698

Copy link
@llynhiavu llynhiavu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Description looks good to me. Thanks.

Copy link
Contributor
@chrisisla chrisisla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see my comment on the associated Issue #3683

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rosetta Pull requests which can be viewed in Rosetta
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Relax cardinality on CollateralCriteriaBase to resolve Concentration Limit defect
6 participants
0