Baunsgaard commented on PR #1983: URL: https://github.com/apache/systemds/pull/1983#issuecomment-1934355601
> The remaining conflicts were just resolved. Now the PR should be ready to merge. Further validation(e.g. empty inputs, differing dimensions of imaginary and real inputs, etc.) and minor bug fixes like the handling of only zero input matrices will come in a separate PR for the integrated python tests. Currently the workflows are waiting for this PR are awaiting approval. Hopefully the integrationtests that were failing before will succeed now. We are assuming that these integration tests are not related to us. Please let us know if you think differently. Hi @fzoepffel The code passes all tests, great. I think your codeformatter ran in many of the PR files. Can we revert this? We try to avoid formatting existing files (even if the format is wrong) unless we do major changes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@systemds.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
