Skip to content

Fix documentation inconsistencies#195

Open
nyooc wants to merge 3 commits into
pfizer-opensource:mainfrom
nyooc:fix-feature-docstrings
Open

Fix documentation inconsistencies#195
nyooc wants to merge 3 commits into
pfizer-opensource:mainfrom
nyooc:fix-feature-docstrings

Conversation

@nyooc

@nyooc nyooc commented Feb 12, 2026

Copy link
Copy Markdown

This PR fixes some inconsistencies in docs:

  • For all features, the axis= keyword for the compute() method was probably copied over from SignalEntropy, and the feature name stuck. Also in the Fortran code.
  • As I see the PfizerRD GitHub org migrated its name to pfizer-opensource. Despite the old address still works, I think it's useful if it doesn't create any ambiguity.

LukasAdamowicz
LukasAdamowicz previously approved these changes Feb 18, 2026
@nyooc nyooc force-pushed the fix-feature-docstrings branch from ef177b9 to 0e2e7d7 Compare February 23, 2026 10:48
@nyooc

nyooc commented Feb 23, 2026

Copy link
Copy Markdown
Author

@LukasAdamowicz thanks for your approve! GitHub told me that you require signed commits so I rebased and force-pushed the same changes with signed commits. Can you please re-approve the GitHub workflow?

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants