Skip to content

Merge develop into regions#4623

Merged
sbesson merged 120 commits intoome:regionsfrom
dominikl:regions_develop_merge
May 6, 2016
Merged

Merge develop into regions#4623
sbesson merged 120 commits intoome:regionsfrom
dominikl:regions_develop_merge

Conversation

@dominikl
Copy link
Copy Markdown
Member

@dominikl dominikl commented Apr 29, 2016

This PR merges all the recent develop PRs including Ice 3.6. The last merge commit should fix a trivial conflict related to the rebasing of the getROICount PR.

joshmoore and others added 15 commits April 26, 2016 20:47
Repository: openmicroscopy/openmicroscopy
Already up-to-date.

Repository: ome/scripts
Updating d1bbb59..1de4a24
Previously merged:
  # PR 111 will-moore 'kymograph_script_getDataset'

Generated by OMERO-DEV-latest-submods#202 (https://ci.openmicroscopy.org/job/OMERO-DEV-latest-submods/202/)
Copy ome.api.* documentation to omero.api.* (3)
Conflicts:
	components/insight/SRC/org/openmicroscopy/shoola/env/data/views/ImageDataViewImpl.java
@mtbc
Copy link
Copy Markdown
Member

mtbc commented May 2, 2016

No remaining changes after ImageDataViewImpl conflict. Duplicate removed in 1b6e56f indeed already has same method in regions. Once we get a successful run of http://regions-ci.docker.openmicroscopy.org:8080/ I'd say we are good to merge.

@sbesson
Copy link
Copy Markdown
Member

sbesson commented May 5, 2016

Discussed with @jburel @dominikl @mtbc, it seems that porting the Ice 3.6 PR will require additional test modifications similar to the ones done in #4594 (for the tests added specifically for OMERO 5.3.0). @mtbc will investigate these tests. In the meantime, we should be able to fix the conflicts between develop and regions without including Ice 3.6 into the question by merging cda8446 into regions. Closing this PR to re-open a new one.

@sbesson sbesson closed this May 5, 2016
@dominikl
Copy link
Copy Markdown
Member Author

dominikl commented May 5, 2016

Replaced by #4632

@sbesson sbesson reopened this May 5, 2016
This was referenced May 5, 2016
@sbesson
Copy link
Copy Markdown
Member

sbesson commented May 6, 2016

Tested the ROI count functionality. Worked as expected. All integration tests are passing. Merging.

@sbesson sbesson merged commit 87510c8 into ome:regions May 6, 2016
@sbesson sbesson added this to the 5.3.0 milestone May 10, 2016
@dominikl dominikl deleted the regions_develop_merge branch May 3, 2017 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants