Initial support for JSON api /annotations endpoint - #682
will-moore wants to merge 10 commits into
Conversation
knabar
changed the title
674eca6 to
c641f1e
Compare
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
| """Override get() to allow filtering by annotation type.""" | ||
|
|
||
| # set self.OMERO_TYPE, then call super().get() to get the list of Annotations | ||
| # E.g. conn.getObjects("TagAnnotation") - not actually case-sensitive |
There was a problem hiding this comment.
I think that the tests of test_show in https://merge-ci.openmicroscopy.org/jenkins/job/OMERO-test-integration/43/testReport/ might be failing because of this change - the timing of the commit and the timing of the tests starting to fail matches.
|
excluded to verify #682 (comment) |
wrong call, including again, sorry |
|
|
Excluding again, as this PR is coupled with ome/openmicroscopy#6459 and ome/omero-py#489. Will exclude all three to be sure, thanks @sbesson |
|
Removed the exclude label. We need to keep an eye on #682 (comment) tomorrow. |
|
I think the rebase to master shouild be performed here in order to include the |
761fe5e to
2e6b872
Compare
2 participants
Fixes #532 - also discussed at #635
Tests added in ome/openmicroscopy#6459
This wraps ome/omero-py#489 and passes query parameters to getObjects().
Adds support for
/api/v0/m/annotations/endpoint, with various query parameters:/api/v0/m/mapannotations/To test:
Also, owner, offset, limit etc work as for other endpoints: