search

DIAGNijmegen / rse-gcapi

Python client for the grand-challenge.org API
Apache License 2.0
2 stars 3 forks source link

Remove retina endpoints #154

Closed jmsmkn closed 3 months ago

jmsmkn commented 9 months ago

See https://github.com/DIAGNijmegen/rse-grand-challenge-admin/issues/236

jmsmkn commented 9 months ago

The tests on main are likely broken.

HarmvZ commented 8 months ago

@jmsmkn I just came across this. Do you know what is going on with the failing tests?

jmsmkn commented 8 months ago

Likely something with the GC fixtures changed and the tests broke. They should be re-run on main. If so, it is not ideal to rely on those fixtures kept on GC. Maybe we should create the fixtures that are needed for the GCAPI tests within this repo.

jmsmkn commented 8 months ago

I'm not sure if this PR works, I think it's better to fix the tests on main first then merge/rebase.

HarmvZ commented 3 months ago

Sidenote: Some references to retina stuff are still in the models.py file. This will automatically be removed when we regenerate the file.