Skip to content

Conversation

@errfrom
Copy link
Collaborator

@errfrom errfrom commented Sep 17, 2024

Closes #1581

Pre-review checklist

  • All code has been formatted using our config (make format)
  • Any new API features or modification of existing behavior are covered with tests
  • The template (templates/ctl-scaffold) has been updated
  • The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Changed, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

@errfrom errfrom self-assigned this Sep 17, 2024
@errfrom errfrom force-pushed the dshuiski/cluster-params branch from 4bae6a2 to fa824a2 Compare April 25, 2025 12:02
Copy link
Collaborator

@marcusbfs marcusbfs left a comment

Choose a reason for hiding this comment

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

Looks good. Just a gently reminder to update the dependencies after cardano-package-set is released with the latest balancer changes.

@marcusbfs
Copy link
Collaborator

The test CSL Garbage Collection is failing, and it has happened before and seems to be non-deterministic. I suggest we improve this test: we could introduce a small rolling average (e.g., 3 to 5 samples) to smooth out random spikes and reduce false positives.

@errfrom errfrom force-pushed the dshuiski/cluster-params branch from 60ae676 to 9f7d480 Compare April 30, 2025 14:14
@errfrom errfrom force-pushed the dshuiski/cluster-params branch from 56f2d1c to aa01022 Compare April 30, 2025 14:20
@errfrom errfrom marked this pull request as ready for review April 30, 2025 14:31
@errfrom errfrom merged commit c3c20a8 into develop Apr 30, 2025
3 checks passed
@errfrom errfrom deleted the dshuiski/cluster-params branch April 30, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatic collateral selection does not go well with mustNotSpendUtxos

3 participants