Skip to content
This repository was archived by the owner on Jul 4, 2025. It is now read-only.

Commit 4d16714

Browse files
fix: wrong engine list (#890)
1 parent f49820f commit 4d16714

File tree

7 files changed

+51
-10
lines changed

7 files changed

+51
-10
lines changed

cortex-js/src/infrastructure/commanders/engines/engines-get.command.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ export class EnginesGetCommand extends CommandRunner {
2727
} else {
2828
console.table({
2929
...engine,
30-
name: EngineNamesMap[engine.name as Engines],
30+
name: EngineNamesMap[engine.name as Engines] || engine.name,
3131
});
3232
}
3333
});

cortex-js/src/infrastructure/commanders/engines/engines-list.command.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { CommandRunner, SubCommand } from 'nest-commander';
22
import { SetCommandContext } from '../decorators/CommandContext';
33
import { ContextService } from '@/infrastructure/services/context/context.service';
44
import { EnginesUsecases } from '@/usecases/engines/engines.usecase';
5-
import { EngineNamesMap, Engines } from '../types/engine.interface';
5+
import { EngineNamesMap } from '../types/engine.interface';
66

77
@SubCommand({
88
name: 'list',
@@ -21,7 +21,7 @@ export class EnginesListCommand extends CommandRunner {
2121
return this.enginesUsecases.getEngines().then((engines) => {
2222
const enginesTable = engines.map((engine) => ({
2323
...engine,
24-
name: EngineNamesMap[engine.name as Engines],
24+
name: EngineNamesMap[engine.name as string] || engine.name,
2525
}));
2626
console.table(enginesTable);
2727
});

cortex-js/src/infrastructure/providers/cortex/cortex.module.ts

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,9 @@ import { Module } from '@nestjs/common';
22
import CortexProvider from './cortex.provider';
33
import { HttpModule } from '@nestjs/axios';
44
import { FileManagerModule } from '@/infrastructure/services/file-manager/file-manager.module';
5+
import Onnxprovider from './onnx.provider';
6+
import LlamaCPPProvider from './llamacpp.provider';
7+
import TensorrtLLMProvider from './tensorrtllm.provider';
58

69
@Module({
710
imports: [HttpModule, FileManagerModule],
@@ -10,12 +13,18 @@ import { FileManagerModule } from '@/infrastructure/services/file-manager/file-m
1013
provide: 'CORTEX_PROVIDER',
1114
useClass: CortexProvider,
1215
},
16+
Onnxprovider,
17+
LlamaCPPProvider,
18+
TensorrtLLMProvider,
1319
],
1420
exports: [
1521
{
1622
provide: 'CORTEX_PROVIDER',
1723
useClass: CortexProvider,
1824
},
25+
Onnxprovider,
26+
LlamaCPPProvider,
27+
TensorrtLLMProvider,
1928
],
2029
})
2130
export class CortexProviderModule {}
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { Injectable } from '@nestjs/common';
2+
import CortexProvider from './cortex.provider';
3+
import { Engines } from '@/infrastructure/commanders/types/engine.interface';
4+
5+
@Injectable()
6+
export default class LlamaCPPProvider extends CortexProvider {
7+
name = Engines.llamaCPP;
8+
productName = 'LlamaCPP Inference Engine';
9+
description =
10+
'This extension enables chat completion API calls using the LlamaCPP engine';
11+
}
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { Injectable } from '@nestjs/common';
2+
import CortexProvider from './cortex.provider';
3+
import { Engines } from '@/infrastructure/commanders/types/engine.interface';
4+
5+
@Injectable()
6+
export default class Onnxprovider extends CortexProvider {
7+
name = Engines.onnx;
8+
productName = 'Onnx Inference Engine';
9+
description =
10+
'This extension enables chat completion API calls using the Onnx engine';
11+
}
Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
import { Injectable } from '@nestjs/common';
2+
import CortexProvider from './cortex.provider';
3+
import { Engines } from '@/infrastructure/commanders/types/engine.interface';
4+
5+
@Injectable()
6+
export default class TensorrtLLMProvider extends CortexProvider {
7+
name = Engines.tensorrtLLM;
8+
productName = 'TensorrtLLM Inference Engine';
9+
description =
10+
'This extension enables chat completion API calls using the TensorrtLLM engine';
11+
}

cortex-js/src/infrastructure/repositories/extensions/extension.repository.ts

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,10 @@ import { FileManagerService } from '@/infrastructure/services/file-manager/file-
77
import { existsSync } from 'fs';
88
import { Engines } from '@/infrastructure/commanders/types/engine.interface';
99
import { OAIEngineExtension } from '@/domain/abstracts/oai.abstract';
10-
import CortexProvider from '@/infrastructure/providers/cortex/cortex.provider';
1110
import { HttpService } from '@nestjs/axios';
11+
import LlamaCPPProvider from '@/infrastructure/providers/cortex/llamacpp.provider';
12+
import Onnxprovider from '@/infrastructure/providers/cortex/onnx.provider';
13+
import TensorrtLLMProvider from '@/infrastructure/providers/cortex/tensorrtllm.provider';
1214

1315
@Injectable()
1416
export class ExtensionRepositoryImpl implements ExtensionRepository {
@@ -44,35 +46,32 @@ export class ExtensionRepositoryImpl implements ExtensionRepository {
4446
}
4547

4648
private async loadCoreExtensions() {
47-
const llamaCPPEngine = new CortexProvider(
49+
const llamaCPPEngine = new LlamaCPPProvider(
4850
this.httpService,
4951
this.fileManagerService,
5052
);
51-
llamaCPPEngine.name = Engines.llamaCPP;
5253
llamaCPPEngine.initalized = existsSync(
5354
join(
5455
await this.fileManagerService.getCortexCppEnginePath(),
5556
Engines.llamaCPP,
5657
),
5758
);
5859

59-
const onnxEngine = new CortexProvider(
60+
const onnxEngine = new Onnxprovider(
6061
this.httpService,
6162
this.fileManagerService,
6263
);
63-
onnxEngine.name = Engines.onnx;
6464
onnxEngine.initalized = existsSync(
6565
join(
6666
await this.fileManagerService.getCortexCppEnginePath(),
6767
Engines.onnx,
6868
),
6969
);
7070

71-
const tensorrtLLMEngine = new CortexProvider(
71+
const tensorrtLLMEngine = new TensorrtLLMProvider(
7272
this.httpService,
7373
this.fileManagerService,
7474
);
75-
tensorrtLLMEngine.name = Engines.tensorrtLLM;
7675
tensorrtLLMEngine.initalized = existsSync(
7776
join(
7877
await this.fileManagerService.getCortexCppEnginePath(),

0 commit comments

Comments
 (0)