search

bgruening / galaxytools

:microscope::books: Galaxy Tool wrappers
MIT License
115 stars 222 forks source link

Fix ML visualisation tool reg confusion matrix #1334

Closed anuprulez closed 10 months ago

anuprulez commented 10 months ago

I think these changes got missed during the upgrade to Galaxy ML 0.10.0 ping @bgruening

bgruening commented 10 months ago

Oo, sorry if I messed that up. Can you fix the linting errors please?

anuprulez commented 10 months ago

no worries, I am also not sure how it was missed

This is the linting error I am getting and coming from keras_batch_models tool XML. I think there is some issue with CI related to parsing the certificates of Keras site. CI is not able to parse this link automatically.

keras: https://keras.io/api/models/model/

When I remove this keras.io link altogether from the tool's XML, there is no linting error. Other links in the same tool get parsed properly.

error1

Error 'HTTPSConnectionPool(host='keras.io', port=443): Max retries exceeded with url: /api/models/model/ (Caused by SSLError(SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1091)')))' accessing https://keras.io/api/models/model/
anuprulez commented 10 months ago

All tests passed ping @bgruening thanks!