-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Video producer tool using benchmarks in BDV #136
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I apparently started to review a long time ago. I'm just going to submit it now.
src/main/java/bdv/BigDataViewer.java
Outdated
final String fn = "/Users/pietzsch/workspace/data/111010_weber_full.xml"; | ||
// final String fn = "/Users/pietzsch/workspace/data/111010_weber_full.xml"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we keep this for @tpietzsch ?
<dependency> | ||
<groupId>com.google.code.gson</groupId> | ||
<artifactId>gson</artifactId> | ||
<version>2.8.5</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we make the version a property? Any reason not to use 2.8.6 like in parent POM?
### Install: | ||
`$ ./install.sh` | ||
|
||
### Run: | ||
`$ ./bdv INPUT_PATH` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mention Windows install instructions or recommend WSL2.
src/main/java/bdv/BigDataViewer.java
Outdated
import org.scijava.ui.behaviour.util.Actions; | ||
|
||
import javax.swing.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we doing wildcards in the imports now?
import javax.swing.*; | ||
import java.awt.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wildcards?
import javax.swing.*; | ||
import javax.swing.border.Border; | ||
import javax.swing.border.TitledBorder; | ||
import java.awt.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wildcards?
import javax.swing.*; | ||
import java.awt.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wildcards?
import bdv.viewer.animate.SimilarityTransformAnimator; | ||
import net.imglib2.realtransform.AffineTransform3D; | ||
|
||
import javax.swing.*; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wildcards?
@mkitti i created a new repo : https://github.com/JaneliaSciComp/bigdataviewer-core-movie-producer |
this is replaced by #151 now |
I created a tool for video producing using BDV
I used the code made by @axtimwalde in hotknife and I embedded it in a GUI. It was a request from @StephanPreibisch
Here is a small tutorial on how to use it: https://youtu.be/vXu4ZOboEio