Conversation
…ind/validmind-library into john/cleanup-dependencies
…dependency type handling
…ed on dependency type
…tart model documentation
…UwN1-py3.11/bin/activate erge branch 'main' into john/cleanup-dependencies
…blank lines and disabling pip caching
- Bump Poetry version to 2.1.3 in `poetry.lock`. - Introduce optional PII detection capabilities using Microsoft Presidio in `README.md`. - Add `pii_filter.py` for detecting and masking PII in test results. - Modify `check_for_sensitive_data` to utilize the new PII detection functionality. - Update `pyproject.toml` to include `presidio-analyzer` as an optional dependency. - Adjust `TestResult` class to ensure PII checks are performed correctly.
…developer-framework
- Update README.md to reflect new PII filtering options, replacing "Enable PII detection" with "Configure PII filtering" and detailing available modes. - Modify run_e2e_notebooks.py to accept a new command-line option for PII filtering mode, allowing users to specify their desired filtering behavior during notebook execution. - Implement PII filtering in test descriptions by adding a new function to filter PII from summaries before sending to the LLM. - Introduce an Enum for PII filtering modes in pii_filter.py, improving clarity and maintainability of PII filtering logic. - Update existing functions to utilize the new PII filtering capabilities, ensuring that PII is appropriately handled in test results and descriptions.
…nctionality - Update README.md to change "PII filtering" to "PII detection" for clarity and consistency. - Modify run_e2e_notebooks.py to reflect the new environment variable for PII detection. - Refactor test descriptions to check for PII content instead of filtering it, raising exceptions when PII is detected. - Rename PII filtering-related functions and enums in pii_filter.py to align with the new terminology. - Ensure all references to PII handling are updated to use the new detection logic.
… Python versions and add constraint files
…, and all Python versions
…ttp, triton, and virtualenv
…port - Introduce Presidio Structured for improved PII detection in structured data. - Update `check_table_for_pii` and related functions to utilize structured analysis when available. - Implement lazy loading for Presidio Structured components to ensure compatibility. - Modify `generate_description` and `TestResult` classes to include PII checks for tables and descriptions. - Update dependencies in `pyproject.toml` and `poetry.lock` to include `presidio-structured`. - Enhance error handling and logging for PII detection failures.
…developer-framework
sc-11551: Improve Library dependencies
…developer-framework
…ection-in-the-developer-framework SC-2886: PII Detection for Uploaded Test Results
* Drafting complete PII notebook * Editing * Save point * Save point * Save point * Save point * Save point * Editing * Editing * Editing * Editing... * Save point * Save point * Save point * ... * Editing * result_id * Editing * Save point * Save point * Save point * Edit * Edit... * ... * . * override * Ugh * Save point * Finishing up? * hdgkj * Proofreading * Moving file * Edit * UGH * sdjfksd * ToC * Blah
Contributor
PR SummaryThis PR updates the version number in the pyproject.toml file. There are no functional changes to the library code, and the update is strictly focused on version management for release purposes. Test Suggestions
|
eggshell
approved these changes
Sep 4, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Pull Request Description
What and why?
Release
How to test
What needs special review?
Dependencies, breaking changes, and deployment notes
Release notes
Checklist