Skip to content

Commit ae11d47

Browse files
DianeoddmorgankxConor McCarthyConor McCarthyandrewmorrison1
authored
Line length (#93)
* update to xval fitscore to support XGBoost models * xval refactor * xval refactor * update to timeseries; Added save/load functionality * review of xval * review of graph * updated param to camelcase * review of graphing structure * added utils function * made naming verbose. Cleaned up code formatting for if statements * update README * add @Private for util funcs * new fresh format * new fresh format * fixes to hyperparam json file for single values * utils refactor * feature function refactor * final refactoring * formatting * formatting review * timeseries windows tests * updates to fresh tests * python utilities * refactor of util folder * refactor review * refactor or optimize section * Addition of test script for line length exceeding 80 characters, updates in line with this and minor changes to aspects of the optimization code * Added deprecation warning. Updated namespace for xval * review of clust code * updated cutDict comment * upd cutK * Minor changes to deprecation functionality * updated functionMapping for clustering * fixed sigfeat tests * updates resulting from review of clustering refactor/update * fix to scoring tests * Fix bugs * review of clust update * Fixed hierarchial comments * removed redundant min function * added WLS,OLS functionality, updated describe function. Updated failing tests for fresh * updated tests and describe function * addition of stats folder * update order of inputs * updated format of fit/predict inputs * added WLS fit function. Fixed inputs to OLS fit order * fixed travis issue for mac * resolved comments * changed all fit/predict functions to the same format. Updated timeseries tests * fixed indentation * added stats tests to bash script * added time series tests for windows * resolve latest comments * update function mapping and fixed comments * addition of README * updated describe function, fixed errors in timeseries and graphimg * fixed filelength * ml utilities style review * fixed line lengths * fixed crossEntropy * fixed .ml.i.findKey * added changes from comments * review of stats * reviewed fresh and fixed stats test * updating clustering and replying to comments on stats and fresh * try to fix 'branch outside repository error' * new commit on new branch * changes to clust/utils.q after comments * commiting with kx email address * commiting with kx email address * review optimize library * fixed desc from fileoverview * changes for comments * fixed @type comments * review remainder of ml libraries * changes following comments * fileoverview changed in pipeline file * changed init file * fixed .ml.i.ap added .ml.infReplace for all types * added test for keyed table infReplace * change predict -> transform * fixed init file. Clash with AutoML if not in ml namespace * resolved comments * fix roc curve * updated docker image * line length error fix Co-authored-by: Deanna Morgan <dmorgan1@kx.com> Co-authored-by: dmorgankx <44678213+dmorgankx@users.noreply.github.com> Co-authored-by: Conor McCarthy <conormccarthy@192.168.1.119> Co-authored-by: Conor McCarthy <conormccarthy@192.168.1.132> Co-authored-by: unknown <Andrew Morrison> Co-authored-by: Andrew Morrison <amorrison1@firstderivatives.com> Co-authored-by: Andrew Morrison <amorrison1@kx.com>
1 parent 6badd67 commit ae11d47

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

xval/xval.q

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -426,5 +426,7 @@ loadfile`:util/mproc.q
426426
loadfile`:util/pickle.q
427427

428428
// If multiple processes are available, multi-process cross validation library
429-
if[0>system"s";multiProc.init[abs system"s"]enlist".ml.loadfile`:util/pickle.q"];
429+
if[0>system"s";
430+
multiProc.init[abs system"s"]enlist".ml.loadfile`:util/pickle.q"
431+
];
430432
xv.picklewrap:{picklewrap[(0>system"s")&.p.i.isw x]x}

0 commit comments

Comments
 (0)