Skip to content

main - consolidate all feature branches into unified submission#68

Open
DYung26 wants to merge 79 commits into
PlotSenseAI:developfrom
DYung26:main
Open

main - consolidate all feature branches into unified submission#68
DYung26 wants to merge 79 commits into
PlotSenseAI:developfrom
DYung26:main

Conversation

@DYung26
Copy link
Copy Markdown
Contributor

@DYung26 DYung26 commented Oct 31, 2025

🧾 Summary

This pull request consolidates all recent feature developments from the individual branches into the main branch.
It represents a unified update that brings together all implemented enhancements, fixes, and refactors previously developed across multiple feature branches.


🔄 Details

  • Merged all feature branches into the fork’s main branch.
  • Ensured consistency, resolved merge conflicts, and aligned the codebase with the latest stable state.
  • This PR now serves as a single consolidated submission of all accumulated changes to the upstream repository.

✅ Purpose

To streamline review and integration by submitting a comprehensive, up-to-date version of all feature work in one pull request.

…LLM` querying with strategy-based execution, parallel model handling, multimodal message building, and weighted model scoring
…coding functions for `matplotlib` integration
…` models with endpoint, key validation, and model listing
…orting text and multimodal queries with validation
…point with query, list_models, and key validation
…th query, list_models, and key validation
…PI` with query, list_models, and key validation
…providers, manage `API` keys, and handle queries
…order until success, with optional success tracking
…ghest-performance models with fallback to lower tiers
… provider/model pairs with iteration-aware selection
…ace` and `ProviderManager` with strategy-based model selection, remove `Groq`-specific hardcoding, and unify image handling
…add new `AI/LLM` packages; update `setup.py` to read `README` safely and include additional dependencies
…arams (strategy, models, iterations, interactivity, timeout)
DYung26 and others added 30 commits October 31, 2025 20:48
… into structured visualization recommendations
…ting model querying, parsing, and ensemble scoring
…gic to dedicated `recommender` module

- Removed the inlined `VisualizationRecommender` class from `suggestions.py` and imported it from `visual_suggestion.recommender`
- Simplified `suggestions.py` to serve as a lightweight interface for visualization recommendation
- Updated `__init__.py` imports to correctly expose `VisualizationRecommender` from its new package
- Enhanced `recommender()` function to support:
  - `StrategyName` parameter for configurable model selection strategy
  - `selected_models`, `interactive`, and `timeout` arguments for flexible runtime behavior
- Improved module clarity and reduced redundancy by centralizing model, API, and ensemble logic under the recommender system
…tgen extensions, and message orchestration
…ining chat, audio, file, and realtime clients
…tool OpenAI function calling (plotgen, explainer, recommender)
…t` assistant with user instruction injection
…nd `SmartPlotGenerator`, replacing deprecated `PlotGenerator`
… `PlotExplainer` and explainer `API` coverage
…icPlotGenerator`, `SmartPlotGenerator`, and `plotgen` integration tests
…`VisualizationRecommender` initialization, parsing, and `LLM` integration
…est_suggestions` into `test/unit` for clearer test structure organization
…ture

refactor/recommender-modular-structure
…recommender` modules with consistent `API` handling and helper usage

- Added default argument values to `VisualizationRecommender` and `PlotExplainer` constructors for strategy, models, timeout, and interactivity
- Introduced `self.api_keys` and standardized `ProviderManager` initialization across both components
- Updated `test_explanations` to use shared helper functions `encode_image` and `save_plot_to_image` from `core.utils`
- Revised `test_plotgen` to align with new `_default_plots` naming and initialization logic
- Refactored `test_suggestions` to leverage new `ResponseParser`, updated recommendation pipeline, and aligned prompt/query flow
- Commented deprecated internal tests tied to removed private methods (`_get_all_recommendations`, `_parse_recommendations`, etc.)
- Ensured test assertions verify model availability and provider correctness under new recommender structure
refactor(core): unify initialization and testing of `explainer` and `…
… add `plot_chat` to `egg-info`; update live tests for display handling and model selection
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