Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add HPC pipeline configuration extension #3

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

Conversation

Kuonirad
Copy link

@Kuonirad Kuonirad commented Jan 22, 2025

Add HPC pipeline configuration extension.

  • New Method: Add _configure_hpc_pipeline method to handle HPC pipeline configuration in my_name.py.
  • Configuration Commands: Implement configuration commands with undo support in _configure_hpc_pipeline.
  • Command Execution: Add _execute_command method to execute configuration commands and add them to the undo stack.
  • Undo/Redo: Implement undo and redo methods to handle undoing and redoing configuration commands.
  • Import and Expose: Import _configure_hpc_pipeline method in extension.py and expose HPC pipeline options through CLI.

For more details, open the Copilot Workspace session.

Add HPC pipeline configuration extension.

* **New Method**: Add `_configure_hpc_pipeline` method to handle HPC pipeline configuration in `my_name.py`.
* **Configuration Commands**: Implement configuration commands with undo support in `_configure_hpc_pipeline`.
* **Command Execution**: Add `_execute_command` method to execute configuration commands and add them to the undo stack.
* **Undo/Redo**: Implement `undo` and `redo` methods to handle undoing and redoing configuration commands.
* **Import and Expose**: Import `_configure_hpc_pipeline` method in `extension.py` and expose HPC pipeline options through CLI.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/NVIDIAGameWorks/toolkit-remix?shareId=XXXX-XXXX-XXXX-XXXX).
Implement configuration commands with undo support
Add comprehensive unit and performance tests
Expose HPC pipeline options through CLI
Testing:

Added unit tests for configuration functionality
Added performance tests for CPU/GPU stress testing
Verified all tests pass locally
@nxkb
Copy link
Collaborator

nxkb commented Jan 23, 2025

Hi @Kuonirad Can you give us some more context about what kind of functionality you are trying to add to the toolkit here?

Also if this replaces #2, can you close the outdated one?

@nxkb
Copy link
Collaborator

nxkb commented Feb 13, 2025

Hi @Kuonirad, just following up on the above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants