These are chat archives for fiji/fiji

25th
May 2017
tpietzsch
@tpietzsch
May 25 2017 00:01
@ctrueden :+1: wow, that was fast!
(pom-scijava 15.0.0)
Curtis Rueden
@ctrueden
May 25 2017 00:19
@tpietzsch I decided to leave out a couple of the notebook-related updates. So there will probably be a 15.1.0 end of next week.
John Bogovic
@bogovicj
May 25 2017 11:08
@ctrueden , just merged your PR, thanks!
Curtis Rueden
@ctrueden
May 25 2017 13:34
@bogovicj I'd like to get the opencsv update into the relevant components, and then cut new releases today. Would you be willing to give me push access so I can cut the releases directly? Or have time to take care of it yourself today?
Curtis Rueden
@ctrueden
May 25 2017 13:42
@bogovicj I am doing the PRs for opencsv now, for the three affected components.
Curtis Rueden
@ctrueden
May 25 2017 14:11
@bogovicj It is a little tricky because we need to update jitk-tps first, release it, then update bigwarp_fiji to use that new release.
Curtis Rueden
@ctrueden
May 25 2017 14:31
@bogovicj PRs filed. saalfeldlab/jitk-tps#3, saalfeldlab/bigwarp#34, saalfeldlab/trakem2-tps#5. The first one needs to be merged, and then jitk-tps 2.1.1 needs to be released, before the latter two will work correctly.
John Bogovic
@bogovicj
May 25 2017 14:37
Thanks @ctrueden . I'll take care of these, but I'm happy to give you push access as well, for the future (provided @axtimwalde agrees)
Curtis Rueden
@ctrueden
May 25 2017 14:57
@bogovicj Thanks. I have meetings for the next couple of hours; if you are able to make releases of these three components soon, I'll update pom-scijava accordingly after my meetings, and I'll be able to get the new Fiji out the door! :-)
Stephan Saalfeld
@axtimwalde
May 25 2017 15:04
@bogovicj, sure, your call
@ctrueden thanks!
John Bogovic
@bogovicj
May 25 2017 15:59
@ctrueden, welp, looks like in an effort to save you work, i created a small mess :worried:, should be easy to clean up though. jitk-tps-2.1.1 is now on maven.imagej, but, there were some javadoc issues i didn't catch pre-release (because I hadn't activated travis yet - my bad). After fixing them, the next build seemed not to be able to deploy new artifacts
I assume that latter issue was because artifacts were already there...
Curtis Rueden
@ctrueden
May 25 2017 17:13
@bogovicj You have to bump to the next SNAPSHOT after Travis builds the release.
In the future, I would like to switch to https://imagej.net/Development_Lifecycle#Method_1:_release-version.sh if you don't mind.
It avoids this pitfall and others.
Although @stelfrich and I are pursuing a Grand Unified Release Strategy which might be the best of all worlds, via Travis... still in flux.
I can fix it now.
John Bogovic
@bogovicj
May 25 2017 17:16
Thanks, sorry. I was under the impression that Travis hadn't build the release (due to the error). I'll give Method 1 a try...
Curtis Rueden
@ctrueden
May 25 2017 17:17
To use method 1, you need an ImageJ Nexus account. I can make one for you any time.
Looks like bigwarp won't build with the latest pom-scijava. :-(
E.g.
[ERROR] .../bigwarp/src/main/java/bdv/img/WarpedSource.java:[3,17] cannot find symbol
[ERROR]   symbol:   class CacheHints
[ERROR]   location: package bdv.cache
John Bogovic
@bogovicj
May 25 2017 17:18
on it
Curtis Rueden
@ctrueden
May 25 2017 17:18
You know how to update it? Great!
I'm checking trakem2-tps now.
Curtis Rueden
@ctrueden
May 25 2017 17:24
trakem2-tps builds just fine. So I merged the PR.
@bogovicj Please keep me posted on the CacheHints and SetCacheHints fixes. Maybe @tpietzsch can comment if there is anything tricky there.
John Bogovic
@bogovicj
May 25 2017 17:27
Looks good so far, just testing some more
Curtis Rueden
@ctrueden
May 25 2017 17:27
Glad to hear.
OK if I cut a new release of trakem2-tps?
John Bogovic
@bogovicj
May 25 2017 17:28
I'll push/pr onto your pom-updates branch, so we can just merge your pr. that cool?
Curtis Rueden
@ctrueden
May 25 2017 17:28
Sure.
John Bogovic
@bogovicj
May 25 2017 17:28
that'd be great
Curtis Rueden
@ctrueden
May 25 2017 17:29
Released!
John Bogovic
@bogovicj
May 25 2017 17:31
thanks!
John Bogovic
@bogovicj
May 25 2017 17:40
bigwarp looks good now, that interface is no longer in the newer bigdataviewer. its safe to remove
@ctrueden, here's the PR on your bigwarp branch ctrueden/bigwarp#1
Curtis Rueden
@ctrueden
May 25 2017 17:48
Thanks! I'm going to massage that history a little quickly.
John Bogovic
@bogovicj
May 25 2017 17:48
:+1:
Curtis Rueden
@ctrueden
May 25 2017 17:54
I'm gonna release bigwarp_fiji. How strictly do you want to stick to SemVer? Technically, public API has been removed and we should go to 3.0.0.
I don't feel super strongly either way in this case.
John Bogovic
@bogovicj
May 25 2017 17:56
I don't feel strongly either, so let's just go to 3.0.0
Curtis Rueden
@ctrueden
May 25 2017 17:56
OK!
John Bogovic
@bogovicj
May 25 2017 17:57
Thanks Curtis!
Curtis Rueden
@ctrueden
May 25 2017 18:01
@bogovicj Thank you too for your responsiveness and help today. Hopefully I can stop distracting you now. I checked, and these components all gel now. (Next problem: register_virtual_stack_slices still depends on mpicbg-trakem2, which was replaced by trakem2-transform. The journey continues.)
John Bogovic
@bogovicj
May 25 2017 18:02
My pleasure, and sorry for creating some :poop:, gotta use the release script next time!
Looking forward to updating my Fiji soon :smile:
Curtis Rueden
@ctrueden
May 25 2017 18:04
It's gonna be a doozy.
Curtis Rueden
@ctrueden
May 25 2017 18:11
BTW, looks like you already have an ImageJ Nexus account, so you should be good to go with release-version.sh as long as your local ~/.m2/settings.xml is set up.