Skip to content
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

Closed
wants to merge 29 commits into from

Conversation

mzouink
Copy link

@mzouink mzouink commented May 9, 2022

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
2

Copy link
Collaborator

@mkitti mkitti left a 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.

Comment on lines 747 to 749
final String fn = "/Users/pietzsch/workspace/data/111010_weber_full.xml";
// final String fn = "/Users/pietzsch/workspace/data/111010_weber_full.xml";
Copy link
Collaborator

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>
Copy link
Collaborator

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?

https://github.com/scijava/pom-scijava/blob/ff35ca810a8717c4f461ef24df4986bf1914c673/pom.xml#L1508-L1509

Comment on lines +7 to +11
### Install:
`$ ./install.sh`

### Run:
`$ ./bdv INPUT_PATH`
Copy link
Collaborator

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.

import org.scijava.ui.behaviour.util.Actions;

import javax.swing.*;
Copy link
Collaborator

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?

Comment on lines +3 to +4
import javax.swing.*;
import java.awt.*;
Copy link
Collaborator

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.*;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wildcards?

Comment on lines +8 to +9
import javax.swing.*;
import java.awt.*;
Copy link
Collaborator

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.*;
Copy link
Collaborator

Choose a reason for hiding this comment

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

Wildcards?

@mzouink
Copy link
Author

mzouink commented Nov 30, 2022

@mkitti i created a new repo : https://github.com/JaneliaSciComp/bigdataviewer-core-movie-producer
in order to have all the changes clear and match to the current version of BDV.

@mzouink mzouink closed this Dec 1, 2022
@tpietzsch tpietzsch mentioned this pull request Dec 1, 2022
@tpietzsch
Copy link
Member

this is replaced by #151 now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants