Summary
The current plugin name nim and executor name nim are quite obscure and don't clearly indicate their purpose. Consider renaming to nvidia-nim for better clarity and discoverability.
Current State
- Plugin ID:
nf-nim (repository name is appropriate)
- Executor name:
nim (used in process { executor 'nim' })
- Service name:
nim (used in TaskPollingMonitor)
Proposed Changes
- Executor name:
nim → nvidia-nim
- Service name:
nim → nvidia-nim
- Plugin name
nf-nim → nvidia-nim
Rationale
- Clarity: "nim" alone doesn't indicate it's related to NVIDIA's inference services
- Discoverability: Users searching for NVIDIA solutions would more easily find
nvidia-nim
- Consistency: Aligns with NVIDIA's branding of their NIM (NVIDIA Inference Microservices) product
- Avoid confusion: "nim" could be confused with the Nim programming language
Files to Update
src/main/groovy/seqeralabs/plugin/NIMExecutor.groovy - ServiceName annotation- Documentation and examples
- README and configuration examples
Alternative naming
Another option could be using nv-nim that's common prefix for nvidia
Summary
The current plugin name
nimand executor namenimare quite obscure and don't clearly indicate their purpose. Consider renaming tonvidia-nimfor better clarity and discoverability.Current State
nf-nim(repository name is appropriate)nim(used inprocess { executor 'nim' })nim(used in TaskPollingMonitor)Proposed Changes
nim→nvidia-nimnim→nvidia-nimnf-nim→nvidia-nimRationale
nvidia-nimFiles to Update
src/main/groovy/seqeralabs/plugin/NIMExecutor.groovy- ServiceName annotationAlternative naming
Another option could be using
nv-nimthat's common prefix for nvidia