Skip to content

Fix render py (rebased onto metadata53) #5239

Merged
jburel merged 10 commits intoome:metadata53from
atarkowska:rebased/metadata53/fix-render-py
Apr 11, 2017
Merged

Fix render py (rebased onto metadata53) #5239
jburel merged 10 commits intoome:metadata53from
atarkowska:rebased/metadata53/fix-render-py

Conversation

@atarkowska
Copy link
Copy Markdown
Member

@atarkowska atarkowska commented Apr 10, 2017

This is the same as gh-4872 but rebased onto metadata53 and #4988 #5053 #5163


See jburel#11

This should fix the currently failing metadata52 tests.

@atarkowska
Copy link
Copy Markdown
Member Author

--rebased-from #4872

This was referenced Apr 10, 2017
A new "tracked service" internal API has been added. All calls to
`_create_func` in `ProxyObjectWrapper` now registers the string
reprenstation of the created service as well as the current call
stack to the owning `BlitzGateway` instance. If on deletion, services
are left open, then an ERROR message including the call stack will
be printed.
The metadata plugin now takes a wait option as with
graph operations and converts this into loop and ms
settings to be passed to all write_to_omero methods
(which internally use BaseClient.submit).
During plugin loading, loop over the contents of sys.path
(i.e. PYTHONPATH) and for all directories which contain a
subdirectory `omero/plugin` run the loadpath method as if
it had been passed to `bin/omero` with `--path`.

NB: This currently does not handle plugins packaged as eggs.
@assert_re()
def setActiveChannels(self, channels, windows=None, colors=None,
reverseMaps=None):
reverseMaps=None, noRE=False):
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joshmoore conflict were on noRE as reverseMaps were added in develop. This is something to review

@jburel jburel merged commit b9892fc into ome:metadata53 Apr 11, 2017
@joshmoore
Copy link
Copy Markdown
Member

Note: this minimally leaves one very common situation whereself.c is nulled before self.__del__ is called:

[jamoore@test45-omeroreadonly-1 log]$ grep _assert_unregistered OMEROweb.log | head -n 1
2017-12-14 17:58:45,963 WARNI [                           omero.gateway] (proc.24164) _assert_unregistered():1596 No services could be found: No session available
[jamoore@test45-omeroreadonly-1 log]$ grep _assert_unregistered OMEROweb.log | wc
  98762 1777623 19652522

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants