Viacheslav Kukushkin

Results 51 comments of Viacheslav Kukushkin

> > > > Hi @sarthakpati , thanks for reminder, didn't look on it yet. Let me check what's happening a bit later this week > > > > >...

As `updateVersion` actually is not meant to be used by real users, I removed it from cli tools. Instead, I brought this script back to root directory and renamed. Now,...

## Failed test: test coverage reduced Test coverage dropped significantly: from 95.01% to 90.20%. For better understanding, here is a [report](https://app.codecov.io/gh/mlcommons/GaNDLF/pull/832/tree) from the previous PR to the master branch vs...

## Failed test: Codacy static code analysis The new issues that codacy shows belong to scripts that were moved from `gandlf_*` to `GANDLF/entrypoint/*.py`. As this is actually an old code,...

@sarthakpati As sub-PRs are going to be merged here later, it would be great if you can review the content of this PR before "contamination":)

@sarthakpati @szmazurek Folks, thank you for your notes! As this PR looks like good enough (except of the question with duplicating entrypoints list in testing), may I ask you to...

> I vote to move to `0.1.0` after this PR gets merged rather than `0.0.20`, since it is going to be a pretty big API change. Both are ok for...

@sarthakpati Fixed / commented all previously mentioned issues, the PR (and sub-PRs) are ready for the next review batch. By the way: right now `PR title` check fails because the...

@sarthakpati It seems to me I fixed the code for usual segmentation cases. However, I found that the code is essentially broken for some specific architectures - `deep_*` and `sdnet`,...

@szmazurek Can you plz show the exact config you're using? I'm not familiar with brats challenge:)