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

Make em* pydocs consistent with one another, fix emstd* formulas #4529

Merged
merged 2 commits into from
Sep 21, 2023

Conversation

alexpeters1208
Copy link
Contributor

@alexpeters1208 alexpeters1208 commented Sep 20, 2023

I noticed some inconsistencies with variable stylings in the em* formulas, so I fixed them. I also noticed that the formulae documented for emstd_tick and emstd_time were not consistent with the Java operations and did not yield correct answers when I coded them up by hand, so I fixed those too.

@alexpeters1208 alexpeters1208 added documentation Improvements or additions to documentation python-server-side labels Sep 20, 2023
@alexpeters1208 alexpeters1208 self-assigned this Sep 20, 2023
@alexpeters1208 alexpeters1208 added the NoReleaseNotesNeeded No release notes are needed. label Sep 21, 2023
Copy link
Contributor

@lbooker42 lbooker42 left a comment

Choose a reason for hiding this comment

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

These changes look good to me. Can you confirm that you corrected the indentation problems in the Args: for several operators? I suspect you did but that white-space diffs are hiding the changes.

Copy link
Contributor

@jmao-denver jmao-denver left a comment

Choose a reason for hiding this comment

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

LGTM

@alexpeters1208 alexpeters1208 merged commit 65bc840 into deephaven:main Sep 21, 2023
11 of 12 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Sep 21, 2023
@alexpeters1208 alexpeters1208 deleted the fix-em-pydocs branch September 22, 2023 18:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation NoDocumentationNeeded NoReleaseNotesNeeded No release notes are needed. python-server-side
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants