Skip to content

Conversation

@affeldt-aist
Copy link
Member

@affeldt-aist affeldt-aist commented Feb 9, 2026

Motivation for this change

One step in addressing issue #1808 .

Maybe the file random_variable.v can be further split, or maybe this is not urgent @Yosuke-Ito-345 @hoheinzollern

Changelog not yet done.

The documentation can certainly be improved.

Checklist
  • added corresponding entries in CHANGELOG_UNRELEASED.md
  • added corresponding documentation in the headers

Reference: How to document

Merge policy

As a rule of thumb:

  • PRs with several commits that make sense individually and that
    all compile are preferentially merged into master.
  • PRs with disorganized commits are very likely to be squash-rebased.
Reminder to reviewers

@affeldt-aist affeldt-aist added this to the 1.16.0 milestone Feb 9, 2026
@affeldt-aist affeldt-aist added the renaming/refactoring 🔧 This is about a renaming or refactoring in the library label Feb 9, 2026
@Yosuke-Ito-345
Copy link
Contributor

At this point, I don't think we should split random_variable.v further, but this is not a strong opinion.
By the way, I am planning to move pmf_measurable before the section tail_expectation_formula, since I will use pmf_measurable to prove le0_expectation_cdf (#1808).

@affeldt-aist
Copy link
Member Author

Then unless somebody objects I will prepare the changelog ASAP (by tomorrow?) and merge so that you work more comfortably.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

renaming/refactoring 🔧 This is about a renaming or refactoring in the library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants