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

[SYSTEMDS-3097] Fix simple-aggregation-example #1366

Closed

Conversation

mar-volk
Copy link
Contributor

  • See Jira ticket;

  • Without the change in line 85 the tutorial will not run.

  • The change in line 57 makes the script return the expected result.

@Baunsgaard
Copy link
Contributor

I don't think it is needed to add the header to the file csv file. But the federated line change indeed needs to happen. Thanks for the fix.

@Baunsgaard
Copy link
Contributor

I don't think it is needed to add the header to the file csv file. But the federated line change indeed needs to happen. Thanks for the fix.

Oh i see in the Jira task you found a bug. For reading the matrix when federated. We will have to address this specifically in another commit.

@Baunsgaard
Copy link
Contributor

Thanks for pointing out an error in the CSV parsing when federated and the error in the docs.

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