search

bids-apps / aa

BIDS App containing an instance of the Automatic Analysis
http://automaticanalysis.org
Other
0 stars 1 forks source link

invalid freesurfer license in circle ci #9

Open Remi-Gau opened 1 year ago

Remi-Gau commented 1 year ago

https://app.circleci.com/pipelines/github/bids-apps/aa/32/workflows/44b8665c-0b59-4bd3-890e-e95b65f3ae0b/jobs/228/parallel-runs/0/steps/0-104

Remi-Gau commented 1 year ago 
ERROR: Invalid FreeSurfer license key found in license file /opt/freesurfer/license.txt
  If you are outside the NMR-Martinos Center,
  go to http://surfer.nmr.mgh.harvard.edu/registration.html to 
  get a valid license file (it's free).
  If you are inside the NMR-Martinos Center,
  make sure to source the standard environment.
tiborauer commented 1 year ago

Fixed by https://github.com/bids-apps/aa/commit/17f9b30e9019857adfa3945a8b1ce877a327a63d

Remi-Gau commented 1 year ago

It is still complaining.

https://app.circleci.com/pipelines/github/bids-apps/aa/34/workflows/bc49cc1e-3eb6-4381-8d02-d5246f0c1978/jobs/249

😭

tiborauer commented 1 year ago

I think the FS version was old. Let's hope it works now.

Remi-Gau commented 1 year ago

Fingers crossed.

By the way it may be worth trying to run this on datasets that have pre computed freesurfer output because it may otherwise always exceed the compute time allowed in circle ci.

Not sure if aa can easily accommodate this possibility.

Also no rush. Fixing the CI of all the bids app is slow moving target.

tiborauer commented 1 year ago

Sure, I can take partially processed datasets as input. However, FreeSurfer processing is not part of the current workflow anyway.

Remi-Gau commented 1 year ago

Ah my bad. I saw freesurfer things and made assumptions.