-
Notifications
You must be signed in to change notification settings - Fork 13
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
Addition of skew, add_prefix, add_suffix, count, std functions #16
Addition of skew, add_prefix, add_suffix, count, std functions #16
Conversation
@cmccarthy1 @rianoc-kx |
Thanks for the changes @marcosvm13, I've approved but @rianoc-kx will need to approve this week additionally to get this included in the next release, we'll look to get this done this week for release with our 2.3 release coming up next week |
I had made my suggested fixes and created a pull request to your repo to review We will be doing a larger refactor of PyKX in a similar fashion to use cleaner q code instead of building F-strings which make the code hard to read and refactor. |
…count Refactor q code. pflake8 fixes.
Feature
What does this change introduce?
An implementation of the
skew
function:https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.skew.html#pandas.DataFrame.skew
An implementation of the
add_prefix
andadd_suffix
function:https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.add_prefix.html
https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.add_suffix.html
Observations:
An implementation of the
count
function:Observations:
An implementation of the
std
function:https://pandas.pydata.org/docs/reference/api/pandas.DataFrame.std.html#pandas.DataFrame.std
General
src/pykx/pykx.q
andsrc/pykx/reimporter.py
src/pykx/util.py
logic which is used for environment variable.zip
been updatedCode
Testing
Documentation
.md
file associated with it been created?mkdocs.yml