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

Mallikharjuna/check sum files #2714

Open
wants to merge 32 commits into
base: main
Choose a base branch
from

Conversation

iamrk04 and others added 27 commits April 12, 2024 14:56
* acquire access token before expiry

* temp change to trigger test

* add requested url

* print the current time as well

* update when to get token

* Revert "temp change to trigger test"

This reverts commit 0ef6401.

* prevent logging token

* temp bump version to trigger test

* Revert "prevent logging token"

This reverts commit 847f5ff.

* try switching to resource

* try fetching OID token every 4 mins

* Revert "temp bump version to trigger test"

This reverts commit 26a3a13.
* comment out request_id logic

* fix syntax
* azureml-rag 2.29

* Bump up component version

* 0.2.29.1
* draft

* Create custom environment

* Format source code

* format.

* remove white spaces in blank line.

* Format

* Adding docstring
* Create RAG components

* Update

* the imperative mood doc style
* release Mixtral-8x22B-v0.1

* update model card
* support for oai, oss & text, chat with token count

* add tests

* skip claude component while upgrading components

* add tiktoken in batch_bench_infer_with_compute

* change to text_generation from text_completion

* fix code style issues

* add tiktoken in env

* separate out tests

* undo change in config.py

* reorganize key checking

* update components for tiktoken package

* fix preprocessor test path

* fix failing tests
* fix momo gated tests by refreshing token every 4 mins

* add change

* fix momo ci
Co-authored-by: Richard Li <richli@microsoft.com>
…2015)

* fetch input model asset id in register component vs passing a file

* update spec

* update

* update version
* add trace_id setter for tree node. Expose possible root spans as property

* if multiple root_spans seperate as different trace entries

* use pythonic loop

* add comment

* check special edge case of our algorithm for finding root_span

* WIP add test. Need to fix root_span strings

* uncomment testcases

* fix root strings for new trace test

* syntax

* spruce up print stmt

* add null check on job cleanup

* add more UTs for possible root spans

* add root_span to possible root spans if exists
* Ccozianu/rm bug fix (#2247)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/rm bug fix (#2248)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/rm bug fix (#2259)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2273)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2275)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

* Fix typo in main invocation

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2292)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

* Fix typo in main invocation

* Add SystemLog: prefix for scorer

* Fixed type error on timeout

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2308)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

* Fix typo in main invocation

* Add SystemLog: prefix for scorer

* Fixed type error on timeout

* Go to a different endpoint for connections listkeys call in batch_benchmark_score

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2316)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

* Fix typo in main invocation

* Add SystemLog: prefix for scorer

* Fixed type error on timeout

* Go to a different endpoint for connections listkeys call in batch_benchmark_score

* Synced all batch-benchmark-inference patch version

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2409)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

* Fix typo in main invocation

* Add SystemLog: prefix for scorer

* Fixed type error on timeout

* Go to a different endpoint for connections listkeys call in batch_benchmark_score

* Synced all batch-benchmark-inference patch version

* use default cred

* replaced with managed cred

* add more logs

* add compute target

* remove logs, handle errors better

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* Ccozianu/4office debug (#2428)

* add more logs

* fix stdout logs

* Make sure we recover details

* Make sure we recover details (#2238)

* SystemLog: prefix logging

* Ccozi/temp logging fix (#2246)

* Make sure we recover details

* SystemLog: prefix logging

* Adding more detailed logging

* Fixing typo

* fix NPE in logging

* Cleaned up somehwat dependence on status definitions

* handle folder

* Added finetuned_start_timeout_seconds

* Fix typo in main invocation

* Add SystemLog: prefix for scorer

* Fixed type error on timeout

* Go to a different endpoint for connections listkeys call in batch_benchmark_score

* Synced all batch-benchmark-inference patch version

* use default cred

* replaced with managed cred

* add more logs

* add compute target

* remove logs, handle errors better

* Add notebook to list aoai connections

* cleanup notebook test values

---------

Co-authored-by: svaruag <gasi@microsoft.com>

* add events input and output

* add none check

* add unit tests

* add e2e tests

* fix fleaky

* update usage for retrieval query and document

* add embeddings

* fix format

* fix format

* fix fleaky format

* fix indent

* fix backslash

* fix white space

* address comments

* update test data

* fix fleaky format

* reduce test data size

* return list for embedding

* resolve merge conflicts

* revert resolve conflicts

* Revert "Merge branch 'master' into yuata/events"

This reverts commit 709307c7370e1fcebe49e62ed3af0a60d6e82b44, reversing
changes made to 4f1708c882ffefea29630250244af18817987f28.

* reduce test file size

---------

Co-authored-by: ccozianu <ccozianu@microsoft.com>
Co-authored-by: svaruag <gasi@microsoft.com>
* added component containing all upload, finetune, delete

* added spec for new compoennt

* added log lines

* fixed delete file function

* fixed code quality'

* fixed code quality'

* fixed delete func

* fixed code quality

* missing doc string

* added doc string

* added doc string

* fixed cancel func

* fixed cancel func
* Add chat-completion in supported task types

* update
* add pf azure pkg in ci

* update

---------

Co-authored-by: yalu4 <yalu4@microsoft.com>
…ripts/promptflow-ci/' folder (#2700)

Co-authored-by: yalu4 <yalu4@microsoft.com>
…odified custom scripts too as per modification requested. (#2609)

* 1) Added design modification to read files without jsonl extension and modified custom scripts too as per modification requested.

* 1) Added design modification to read files without jsonl extension and modified custom scripts too as per modification requested.

* 1) Added design modification to read files without jsonl extension and modified custom scripts too as per modification requested.

* 1) Added design modification to read files without jsonl extension and modified custom scripts too as per modification requested.

* Addressed the comments provided by reviewers.
@MallikharjunaThota123 MallikharjunaThota123 added the safe to test Pull request from a fork that's been reviewed and is cleared for additional testing label Apr 18, 2024
Copy link

Test Results for training-model-mgmt-unittests

44 tests   44 ✅  8s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit 1b8df47.

@KonakanchiSwathi
Copy link
Contributor

please resolve the conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Pull request from a fork that's been reviewed and is cleared for additional testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet