fzoepffel commented on PR #1983: URL: https://github.com/apache/systemds/pull/1983#issuecomment-1934494397
> > 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. I just saw. I am a little unsure however, how I can revert this. When I run the formatter (which I configured to use the CodeStyle_eclipse.xml file) it does not necessarily reverse to what it was before. Could it be that some of the files had not been formatted accordingly before? And should these still be reversed now? -- 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
