Achilles icon indicating copy to clipboard operation
Achilles copied to clipboard

achilles final merge fails on individual analysis failure

Open fdefalco opened this issue 2 years ago • 6 comments

Issues such as the one found at the following link occur frequently within the community of users.

https://forums.ohdsi.org/t/running-achilles-issue/16579/2

As part of execution the failure of an analysis should be caught and the final merge updated to prevent total failure on the failure of individual analyses.

fdefalco avatar May 16 '22 15:05 fdefalco

I executed the below command on R:

remotes::install_github("OHDSI/Achilles")

then I got message error as below: Error: Failed to install 'unknown package' from GitHub: Timeout was reached: [api.github.com] Resolving timed out after 10000 milliseconds

I think the issues from fire wall and proxy! Is there any command that I can go over the fire wall and proxy?

faisalsafar99 avatar Aug 15 '22 06:08 faisalsafar99

@faisalsafar99 - that error is not related to this issue and instead is an issue contacting the github site, which is an issue local to your environment that I would not be able to diagnose.

fdefalco avatar Aug 16 '22 17:08 fdefalco

Issues such as the one found at the following link occur frequently within the community of users.

https://forums.ohdsi.org/t/running-achilles-issue/16579/2

As part of execution the failure of an analysis should be caught and the final merge updated to prevent total failure on the failure of individual analyses.

Screenshot 2023-03-13 at 9 34 17 PM

Is this what the failure looks like? I'm trying to find the right place to post my issue.

acumenus avatar Mar 14 '23 01:03 acumenus

@acumenus Indeed, it seems like analysis 716 failed, which in turn made the merge fail.

@fdefalco Any progress on this. Related to what I posted under #697

MaximMoinat avatar Mar 14 '23 08:03 MaximMoinat

@alondhe - has this happened to you? Do you know of any fix other than what was posted on the Forums last year?: Screenshot 2023-03-14 at 12 43 52 PM

acumenus avatar Mar 14 '23 16:03 acumenus

@acumenus Indeed, it seems like analysis 716 failed, which in turn made the merge fail.

@fdefalco Any progress on this. Related to what I posted under #697

Yes, we're planning on a revamp of this merge approach where we add capture of data for each analysis execution so we know how long they take, and if they competed properly, then we'll be able to tell which to merge based on which completed successfully.

fdefalco avatar Mar 15 '23 18:03 fdefalco