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

Stack the same values in the tracker #1134

Closed

Conversation

ldorau
Copy link
Contributor

@ldorau ldorau commented Feb 25, 2025

Description

Stack the same values in the tracker.

It is an alternative solution to the #1121 (without splitting and merging tracker entries)

Ref: #1121

Checklist

  • Code compiles without errors locally
  • All tests pass locally
  • CI workflows execute properly

@ldorau ldorau force-pushed the Stack_the_same_values_in_the_tracker branch 2 times, most recently from dec4ce0 to 43ddd6c Compare February 25, 2025 11:03
@ldorau
Copy link
Contributor Author

ldorau commented Feb 25, 2025

@vinser52 @igchor @bratpiorka @lplewa @kswiecicki Please review also this alternative solution without splitting and merging tracker entries.

@ldorau ldorau force-pushed the Stack_the_same_values_in_the_tracker branch from 43ddd6c to 3bb3390 Compare February 25, 2025 13:05
Copy link
Contributor

Choose a reason for hiding this comment

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

Could you please enable the ipcFixture.hpp tests for the Fixed provider on top of L0 pool.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I cannot, because the Fixed provider does not support IPC at all.

@ldorau
Copy link
Contributor Author

ldorau commented Feb 25, 2025

I added these 2 tests to poolFixtures.hpp also - see #1136

@ldorau
Copy link
Contributor Author

ldorau commented Feb 26, 2025

It will be implemented in another way soon.

@ldorau ldorau closed this Feb 26, 2025
@ldorau ldorau deleted the Stack_the_same_values_in_the_tracker branch March 13, 2025 14:08
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.

2 participants