fix: merging dev changes to main branch #256
Merged
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.
Purpose
Logging and Error Tracking Enhancements:
src/backend/app_kernel.py: Replaced print statements with proper logging or tracking mechanisms, such astrack_event_if_configured, to improve error observability and ensure consistent logging practices. [1] [2] [3]src/backend/handlers/runtime_interrupt_kernel.py: Removed unnecessary print statements in various methods, includingon_message,get_messages, andregister_handlers, to clean up the code and reduce clutter. [1] [2] [3]Azure Monitoring Integration:
src/backend/app_kernel.py: Addedconfigure_azure_monitorfor Application Insights integration, ensuring better monitoring capabilities for Azure services. Updated the logic to configure Application Insights based on environment variables. [1] [2]Code Cleanup:
src/backend/kernel_tools/product_tools.py: Removed redundant print statements across multiple methods, such ascheck_inventory,update_inventory, andgenerate_product_report, to streamline the code and focus on returning meaningful outputs. [1] [2] [3]Configuration Updates:
.github/workflows/deploy.yml: IncreasedGPT_MIN_CAPACITYfrom 50 to 140 to adjust resource allocation for the deployment workflow.Does this introduce a breaking change?
How to Test
What to Check
Verify that the following are valid
Other Information