Skip to content

Commit

Permalink
Fix conda channels ordering for gpt client
Browse files Browse the repository at this point in the history
Signed-off-by: Paolo Di Tommaso <paolo.ditommaso@gmail.com>
  • Loading branch information
pditommaso committed May 19, 2024
1 parent 08f3f56 commit ed3c0be
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
4 changes: 3 additions & 1 deletion app/src/main/java/io/seqera/wave/cli/App.java
Original file line number Diff line number Diff line change
Expand Up @@ -82,6 +82,8 @@
usageHelpAutoWidth = true)
public class App implements Runnable {

static public final String DEFAULT_CONDA_CHANNELS = "conda-forge,bioconda";

private static final org.slf4j.Logger log = LoggerFactory.getLogger(App.class);

private static final boolean isWindows = System.getProperty("os.name").toLowerCase().contains("windows");
Expand Down Expand Up @@ -158,7 +160,7 @@ public class App implements Runnable {
private List<String> condaRunCommands;

@Option(names = {"--conda-channels"}, paramLabel = "''", description = "Conda channels used to build the container (default: ${DEFAULT-VALUE}).")
private String condaChannels = "conda-forge,bioconda";
private String condaChannels = DEFAULT_CONDA_CHANNELS;

@Option(names = {"--spack-file"}, paramLabel = "''", description = "A Spack file used to build the container e.g. /some/path/spack.yaml.")
private String spackFile;
Expand Down
4 changes: 3 additions & 1 deletion app/src/main/java/io/seqera/wave/cli/util/GptHelper.java
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
package io.seqera.wave.cli.util;

import java.io.IOException;
import java.util.Arrays;
import java.util.List;
import java.util.Map;

Expand All @@ -29,6 +30,7 @@
import dev.langchain4j.model.openai.OpenAiChatModel;
import dev.langchain4j.model.output.Response;
import io.seqera.wave.api.PackagesSpec;
import io.seqera.wave.cli.App;
import io.seqera.wave.cli.exception.BadClientResponseException;
import io.seqera.wave.cli.json.JsonHelper;
import org.apache.commons.lang3.StringUtils;
Expand Down Expand Up @@ -117,7 +119,7 @@ static protected PackagesSpec jsonToPackageSpec(String json) {
throw new IllegalArgumentException("Unable to resolve packages from json response: " + json);
List<String> channels = (List<String>) object.get("channels");
if( Checkers.isEmpty(channels) )
channels = List.of("bioconda","conda-forge");
channels = Arrays.asList(App.DEFAULT_CONDA_CHANNELS.split(","));
return new PackagesSpec()
.withType(PackagesSpec.Type.CONDA)
.withEntries(packages)
Expand Down

0 comments on commit ed3c0be

Please # to comment.