Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

Rename to neo-xgboost.json - #1933

Closed
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json
Closed

Rename to neo-xgboost.json#1933
nikhil-sk wants to merge 2 commits into
aws:masterfrom
nikhil-sk:fix_neo_json

Conversation

@nikhil-sk

Copy link
Copy Markdown
Contributor

Issue #, if available:
#1932

Description of changes:
Rename the .json that holds registry information for neo containers.

Testing done:
No - please advise.

Merge Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.

General

  • I have read the CONTRIBUTING doc
  • I used the commit message format described in CONTRIBUTING
  • I have passed the region in to all S3 and STS clients that I've initialized as part of this change.
  • I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

  • I have added tests that prove my fix is effective or that my feature works (if appropriate)
  • I have checked that my tests are not configured for a specific region or account (if appropriate)
  • I have used unique_name_from_base to create resource names in integ tests (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: 0d7cd24
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: 0d7cd24
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-unit-tests
  • Commit ID: b56a238
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-notebook-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-local-mode-tests
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@sagemaker-bot

Copy link
Copy Markdown
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: sagemaker-python-sdk-pr
  • Commit ID: b56a238
  • Result: SUCCEEDED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@icywang86rui

Copy link
Copy Markdown
Contributor

@nikhil-sk
nikhil-sk deleted the fix_neo_json branch October 2, 2020 17:32
rsareddy0329 added a commit that referenced this pull request Dec 3, 2025
* sagemaker-core update, port sft-trainer and ai registry (#1895)
* sagemaker core update with MC service.json, SFT notebook flow succeeded before modelpackage
* remove workaround, modelpackage issue persists
* remove ai registry port
* feat: AIRegistry SDK Implementation (#1893)
* remove sft trainer port, fix ModelPackage issue (#1901)
* Add/port finetuning interfaces to v3 (#1906)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* ported evaluation from v2->v3. pending example notebooks results (#1902)
* v2->v3 porting for eval
* bring back deleted needed files
* unit tests
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Mc deployment (#1909)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Add base_trainer, addition to PR#1906 (#1910)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* AIRegistry Integration Tests (#1908)
* feat: Trainer wait with MLFlow metrics (#1907)
* feat: update eval integs (#1912)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Update get_all method for Evaluator and Dataset (#1913)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* port bedrock model builder and add integ test (#1911)
* port bedrock model builder and add integ test
* Add unit test and notebook, port new version of bedrock model builder
* add docstring
* Mc deployment integ tests and example notebook (#1914)
* Updates for Model Customization
* Model Builder Base change
* Fix
* Support for Model Customization
* New Model Builder changes with Model Customization
* Fix
* Deployment Integ tests
* Tests
* Deployment NOtebook
* fix
* Update model hub name and encoding method (#1918)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add telemetry for model customization (#1920)
* add telemetry to model customization
* add telemetry to model customization 2
* small fix
* INteg tests for nova bedrock (#1926)
* Bedrock Nova
* Unit tests for bedrock nova
* accept eula always (#1928)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Add studio tags to trainer (#1927)
* Update get_all method for Evaluator and Dataset
- Update Dataset.get_all() and added refresh()
- Update Evaluator.get_all() and added refresh()
- Added more tags in condition when importing hub content
* Update dataset, evaluators and examples
* remove hub_content_name
* Update hub name and hash
- Update model hub name to be AIRegistry-{accountId}-{region}
- Update hub name hash
* Add studio tags to trainer
Tested nu checking training job creation inputs and unit tests
---------
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
* Finetuning classes: add accept_eula support & Add integ tests for sft, rlvr,rlaif, dpo trainers (#1915)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* feat: pipeline name fix + mlflow arn fix + jinja dep (#1931)
* changes for pipeline naming
* feat: pipeline name fix + mlflow arn fix + jinja dep
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* AIRegistry: Add Support to user provided session and role (#1932)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* format name for ai reg, stripped name of =, added domain-id tags for … (#1930)
* format name for ai reg, stripped name of =, added domain-id tags for datasets and evaluators, prefix changed t use full path including filename
* pulling metadata path since get_current_domain_id didnt work
* rmvd gamma endpoint
---------
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* fixed train unit tests (#1933)
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
* Updating DataSet versioning pattern for parity with UI (#1934)
* fix dataset arn configuration in eval (#1935)
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Master model customization v3 (#1936)
* Updating DataSet versioning pattern for parity with UI
* Updating default S3 bucket and prefix for datasets
* fix: custom metrics and pipeline name in lineage (#1937)
* remove mlflow config for base model eval only
* fix unit
* fix custom metrics and pipeline name in lineage
---------
Testing:
1. Added/Ran units.
2. Tested LLMAJ eval.
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Integ tests for ModelBuilder (#1939)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* fixing AIRegistry integration and unit tests (#1941)
* Update error messages based on feedback (#1938)
* feat: Inline MLFlow metrics with Serverless MLFlow App (#1942)
* Master model customization v3 (#1945)
* Bedrock Nova
* Unit tests for bedrock nova
* Integ test updates
* Test case updates
* Bedrock tests
* fix: update eval integ tests (#1946)
* update eval integ tests
* fix unit tests
---------
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
* Finetuning classes: Add integ tests (#1947)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
* Workflow change to enable github actions (#1948)
* workflow change and test
* remove test
* Remove beta endpoint info, add region validation and tests (#1950)
* Add/port finetuning interfaces to v3
* Add/port finetuning interfaces to v3
* Add base_trainer, addition to PR#1906
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Add integ tests for sft, rlvr,rlaif, dpo trainers
* Finetuning classes: Add accept_eula support
* Finetuning classes: Add accept_eula support
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* AIRegistry: Support user provided session and role
* Finetuning classes: Add integration tests
* Merge conflicts
* Merge conflicts
* Remove beta endpoint details in sagemaker-core
* add region validation for models
---------
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
---------
Co-authored-by: Molly He <mollyhe@amazon.com>
Co-authored-by: jam-jee <jamjee@amazon.com>
Co-authored-by: rsareddy0329 <rsareddy0329@gmail.com>
Co-authored-by: Roja Reddy Sareddy <rsareddy@amazon.com>
Co-authored-by: Mohamed Zeidan <zeidmo@amazon.com>
Co-authored-by: Gokul Anantha Narayanan <166456257+nargokul@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <89424143+mufaddal-rohawala@users.noreply.github.com>
Co-authored-by: Mufaddal Rohawala <mufi@amazon.com>
Co-authored-by: Zhaoqi <52220743+zhaoqizqwang@users.noreply.github.com>
Sign up for freeto 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.

3 participants

@nikhil-sk@sagemaker-bot@icywang86rui