Javiermateor commented on PR #2287:
URL: https://github.com/apache/systemds/pull/2287#issuecomment-3129194096
Hi @christinadionysio ,
I ran the failing federated tests locally to rule out any issues with my
changes. Here are the results:
* `mvn test -Dtest=...FederatedFullCumulativeTest` -> **BUILD SUCCESS**
(16 tests passed)
* `mvn test -Dtest=...FederatedFullAggregateTest` -> **BUILD SUCCESS** (22
tests passed)
**Both of the test classes that are failing in the CI pipeline pass
perfectly on my local machine.** The failures may be due to a flaky test in the
CI environment and are not caused by my changes in our `nn` module or in our
implementation for the PR.
For completeness, the issue in the CI appears to be related to `FED°fedinit`
instructions, which points towards other error sources, maybe environmental
factors like networking or process timing.
This pattern strongly suggests that these are flaky tests and the failures
are due to the CI environment rather than a bug introduced by my PR.
Could a maintainer please take a look or advise on how to proceed?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
