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

feat: add drop_keys parameter to map operation #8

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

shreyashankar
Copy link
Collaborator

Changes

  • Implemented drop_keys feature in both MapOperation and ParallelMapOperation classes
  • Updated syntax_check() methods to validate drop_keys configuration
  • Modified execute() methods to apply drop_keys to the output

Details

  • drop_keys allows users to specify a list of keys to be removed from the output
  • Added appropriate error handling and type checking for drop_keys configuration

Testing

The new tests cover the drop_keys functionality in both scenarios:

  1. When used in conjunction with LLM-based processing
  2. When used as a standalone key-dropping operation

@shreyashankar shreyashankar merged commit 2db3d26 into main Sep 20, 2024
3 checks passed
@shreyashankar shreyashankar deleted the shreyashankar/dropkeys branch September 20, 2024 18:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant