Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CCA ingestion causes error message #1179

Open
alexsielicki opened this issue Sep 19, 2024 · 1 comment
Open

CCA ingestion causes error message #1179

alexsielicki opened this issue Sep 19, 2024 · 1 comment
Assignees

Comments

@alexsielicki
Copy link
Collaborator

User writes

I think I was exceeding the maximum number of output variables allowed.

@alexsielicki
Copy link
Collaborator Author

I looked into this and found that we are doing a few checks for number of input and output vars:

  • Number of inputs and outputs must be less than the number of rows
  • Number of inputs and outputs must be at least 1

(see code below)

Are there other limitations?

Best,
Alex

if (
      input_columns.length >= component.row_count() ||
      output_columns.length >= component.row_count()
    ) {
      dialog.dialog({
        message:
          "The number of inputs must be less than " +
          component.row_count() +
          ". The number of outputs must be less than " +
          component.row_count() +
          ". You have selected " +
          input_columns.length +
          " inputs and " +
          output_columns.length +
          " outputs.",
      });
    } else if (input_columns.length == 0) {
      dialog.dialog({
        message: "The number of inputs must be at least one.",
      });
    } else if (output_columns.length == 0) {
      dialog.dialog({
        message: "The number of outputs must be at least one.",
      });
    }

You can see this code in context here:
https://github.com/sandialabs/slycat/blob/alex-cca/web-server/plugins/slycat-cca/js/new-ui.js#L267-L291

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant