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

fix: Research Assistant Deployment Issue - Prompt Flow version update #287

Merged
merged 4 commits into from
Jan 20, 2025

Conversation

Pradheep-Microsoft
Copy link
Contributor

@Pradheep-Microsoft Pradheep-Microsoft commented Jan 20, 2025

Purpose

It's Prompt flow version update
In AI Studio Deployment Guide, Draft flow Zip file has to be uploaded -> This Draft Flow contain requirements.txt file.
This file need version update to complete the uploading process

Reference link:

Prompt flow version has to be updated from promptflow==1.16.2 to promptflow==1.17.0

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

@Pradheep-Microsoft Pradheep-Microsoft self-assigned this Jan 20, 2025
@Copilot Copilot bot review requested due to automatic review settings January 20, 2025 06:55
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.

Files not reviewed (1)
  • ResearchAssistant/Deployment/scripts/aihub_scripts/requirements.txt: Language not supported

@Pradheep-Microsoft Pradheep-Microsoft changed the title fix: A bug fix - Research Assistant Deployment Issue - Prompt Flow version update chore: A bug fix - Research Assistant Deployment Issue - Prompt Flow version update Jan 20, 2025
@Pradheep-Microsoft Pradheep-Microsoft changed the base branch from main to dev January 20, 2025 08:47
@Roopan-Microsoft Roopan-Microsoft changed the title chore: A bug fix - Research Assistant Deployment Issue - Prompt Flow version update fix: Research Assistant Deployment Issue - Prompt Flow version update Jan 20, 2025
@Roopan-Microsoft Roopan-Microsoft merged commit b785eb6 into dev Jan 20, 2025
7 of 8 checks passed
Roopan-Microsoft added a commit that referenced this pull request Feb 17, 2025
* Downgrade msal[broker] to 1.24.0b1 and azure-mgmt-resource to 23.1.0b2

* Downgrade azure-ai-ml package to version 1.17.1 to resolve package version conflicts (#283)

* Upgrade azure-ai-ml package to compatible version

* test: Research assistant backend unit test (#286)

* Research assistant backend unit test

* Research assistant backend unit test

* Research assistant unit test pylint correction

* pylint correction

* research assistant .yml file

* Update test_research_assistant.yml

---------

Co-authored-by: Roopan-Microsoft <[email protected]>

* fix: Research Assistant Deployment Issue - Prompt Flow version update (#287)

* Prompt flow version update

* Add files via upload

* Add files via upload

* Add files via upload

---------

Co-authored-by: Roopan-Microsoft <[email protected]>

* Implement Consistent Versioning and Tagging for Accelerator Images (#288)

* Dev image version check

* Revert "feat: Dev Image Version Check Test"

* update history method changes (#295)

* fix: Fixed Asset and retirement update query (#332)

* updated Asset and retirement updated code

* minor fix on meeting days

* fixed test case

* optimized query

* fixed the UI issue in Chat history list section (#335)

---------

Co-authored-by: Prasanjeet-Microsoft <[email protected]>
Co-authored-by: Prajwal-Microsoft <[email protected]>
Co-authored-by: Pradheep-Microsoft <[email protected]>
Co-authored-by: Pavan-Microsoft <[email protected]>
Co-authored-by: pradeepjha-microsoft <[email protected]>
Co-authored-by: AjitPadhi-Microsoft <[email protected]>
Co-authored-by: Mohan-Microsoft <[email protected]>
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.

2 participants