Skip to content
New issue

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

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

Already on GitHub? # to your account

Test patternal 3 #54

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Test patternal 3 #54

wants to merge 3 commits into from

Conversation

aalex
Copy link
Collaborator

@aalex aalex commented Dec 17, 2024

No description provided.

@aalex aalex self-assigned this Dec 17, 2024
auto quants = tk.get_quantification_date(1. / 4);
auto [pos0, q0] = quants[0];
REQUIRE(quants.size() == 1);
REQUIRE(pos0 == -2147483648);
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Pos is always -2147483648

Add test_quantification_date

Add this-> to make the code easier to read

WIP fix errors in test

WIP

Improve the unit test a lot

Improve the unit test

Add some doc
@jcelerier
Copy link
Member

fyi the build on main is fixed so this can be rebased to ensure green CI :)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants