Error in Python test (submission annotations)

Description

Heads up: Was running python tests against a dev stack (more precisely against cwtest stack, to generate activity), based on 316 and got the error below.

Note: I run these tests weekly against dev stack to verify there's no change on the backend that would break the client. This is like a pre-test (haven't updated dev stack yet).

Environment

python 3.6, synapseclient from main branch, backend 316.

Activity

Show:
Xavier Schildwachter
June 22, 2020, 10:43 PM

@bruce, I typically run the test against the dev stack we build every week. This one just happens to run against a separate stack I created (cwtest, build off release-316) to test the CloudwatchLogs. I mentioned it so you guys don't try to repro against the dev stack (still off release-315) before I ugrade it. Sorry I was not was very clear!

Bruce Hoff
June 22, 2020, 10:55 PM

: You describe the issue perfectly. I am pretty certain I introduced the change last week while removing some deprecated code:
https://github.com/Sage-Bionetworks/Synapse-Repository-Services/pull/4089/files#diff-feb41fb221ae44b11b6b4b13a305b876

I'm also fairly sure that clients rarely (if ever) use the annotations placed on an entity when evaluating a challenge submission. I asked Tom Yu, who does most of our work on challenges in this area and he says, "No. We do not instruct people to annotate the original file entity". So I think it's fine to proceed with your pull request and patch the released version of the Python client. I'm not worried about "breaking" any current challenges.

Thanks for diligently checking, !

Jordan Kiang
June 24, 2020, 11:04 PM

Fixed with https://github.com/Sage-Bionetworks/synapsePythonClient/pull/761

% nosetests tests.integration.synapseclient.test_evaluations:test_evaluations
.
----------------------------------------------------------------------
Ran 1 test in 18.666s

OK

Jordan Kiang
July 9, 2020, 6:50 PM

can you validate that the issue is resolved for you (this issue manifested in the python client integration tests which now succeed as in https://travis-ci.org/github/Sage-Bionetworks/synapsePythonClient/builds/706591739)

Xavier Schildwachter
July 10, 2020, 8:23 PM

Verified fixed from develop branch.

Assignee

Jordan Kiang

Reporter

Xavier Schildwachter

Labels

None

Validator

Xavier Schildwachter

Development Area

None

Release Version History

None

Fix versions

Affects versions

Priority

Major
Configure