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

Update capture.md #922

Merged
merged 16 commits into from
Aug 29, 2024
Merged

Update capture.md #922

merged 16 commits into from
Aug 29, 2024

Conversation

libbyhopfauf
Copy link
Member

This update includes reorganizing the capture instructions to include a section for CLI vs. GUI. The CLI instructions were also broken down to include a section explaining how to build a CLI string, examples of frequently used capture commands and a list of the full commands (the last of which still needs to be updated and examples need to be added).

This update includes reorganizing the capture instructions to include a section for CLI vs. GUI. The CLI instructions were also broken down to include a section explaining how to build a CLI string, examples of frequently used capture commands and a list of the full commands (the last of which still needs to be updated and examples need to be added).

7.) Type the files name. Add a space. Example: ```filename.dv```

8.) Type ```-m``` to note you want to merge the multiple capture attempts for problematic frames (dvrescue will automatically rewind and try to capture the frame again when it encouters a problematic frame).
Copy link
Contributor

Choose a reason for hiding this comment

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

this -m also needs an output - in this case, so the total would be -m -


2.) You can either type ```dvrescue``` at the start of the command or include the path to the version of dvrescue you want to use. Example: ```/usr/local/bin/dvrescue```

4.) If you only have one device connected, dvrescue will automatically select it (device 0). If you have more than one device connected, add the device number ```device://1``` (for the corresponding deck you want to use for capture). Example (using the deck listed in step 1): ```dvrescue device://0```
Copy link
Contributor

Choose a reason for hiding this comment

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

hmmm, don't you have the full guid in your result such as

$ dvrescue --list_devices
90:a1838cb4:00000000: UltraStudio 4K Mini [DeckLink]

Copy link
Member Author

Choose a reason for hiding this comment

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

Whoops, looks like that was an old example. Will update. So people need to add full number if they have more than one device plugged in right?

Copy link
Member Author

Choose a reason for hiding this comment

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

@dericed See latest additions to the pull request for changes :)

docs/_sections/capture.md Outdated Show resolved Hide resolved
I translated the information that was in a Google Doc for the GUI capture documentation to markdown. The formatting could use some adjustments to make it look better, but that's not something I know how to do.
I cleaned up the formatting to make it all consistent and make the organizational hierarchy clearer.
@libbyhopfauf
Copy link
Member Author

@dericed I made all of your suggested changes and made sure all of the latest updates are in there. But them me know if you catch anything else I should change before merging this in :)

This is a draft to update the packaging documentation,
I was using the wrong formatting to create hyperlinks in the documentation. This is now corrected.
@libbyhopfauf
Copy link
Member Author

@dericed can this be merged?

updated draft of CLI commands (some not working)
@libbyhopfauf
Copy link
Member Author

libbyhopfauf commented Aug 19, 2024

@dericed So, I'm hitting a little bit of a brick wall trying to make the CLI documentation accurate. Across the board, with only one deck connected, if I do not specify the device, the capture command will not run. Also, it seemed like if I rand the first command that did work that it was rewinding/recapturing/merging even though I hadn't specified a rewind-count.

These commands work:

Capture from a specific device:
dvrescue device://0xdevicename -m /pathtofile/filename.dv
For example:
dvrescue device://0x800460104ce2bad -m /Users/libbyshopfauf/Desktop/12345.dv

Capture from a specific device and show a preview window:
dvrescue device://0xdevicename -m /pathtofile/filename.dv -m - | ffplay -
For example:
dvrescue device://0x800460104ce2bad -m /Users/libbyshopfauf/Desktop/12345.dv -m - | ffplay -

These commands do not:

Capture from a specific device using rewind, recapture and merge (3 attempts):
dvrescue device://0xdevicename --rewind-count 3 -m filename.dv -m -
dvrescue -y device://0xdevicename --rewind-count 3 -m filename.dv -m -
dvrescue device://0xdevicename -y --rewind-count 3 -m filename.dv -m -

Capture from a specific device using rewind, recapture and merge (3 attempts), show a preview window (with only one deck connected):
dvrescue device://0xdevicename --rewind-count 3 -m filename.dv -m - | ffplay -
dvrescue -y device://0xdevicename --rewind-count 3 -m filename.dv -m - | ffplay -
dvrescue device://0xdevicename -y --rewind-count 3 -m filename.dv -m - | ffplay -

Basic capture (with only one deck connected):
dvrescue -m filename.dv

Basic capture (with only one deck connected), using rewind and recapture, plus playback view:
dvrescue -m filename.dv -m - | ffplay -

I also tried this command and the terminal just hangs and doesn't interact with the deck.
usr/local/bin/dvrescue -y device://0x800460104ce2bad -m /Users/libbyshopfauf/Desktop/test0001.dv -m - --verbosity 9 --csv | ffplay -

Here is the Terminal output (the numbers never change from 0, ffplay never opens, and the deck doesn't start playing):

ffplay version 7.0.1 Copyright (c) 2003-2024 the FFmpeg developers
  built with Apple clang version 15.0.0 (clang-1500.3.9.4)
  configuration: --prefix=/opt/homebrew/Cellar/ffmpeg/7.0.1 --enable-shared --enable-pthreads --enable-version3 --cc=clang --host-cflags= --host-ldflags='-Wl,-ld_classic' --enable-ffplay --enable-gnutls --enable-gpl --enable-libaom --enable-libaribb24 --enable-libbluray --enable-libdav1d --enable-libharfbuzz --enable-libjxl --enable-libmp3lame --enable-libopus --enable-librav1e --enable-librist --enable-librubberband --enable-libsnappy --enable-libsrt --enable-libssh --enable-libsvtav1 --enable-libtesseract --enable-libtheora --enable-libvidstab --enable-libvmaf --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxml2 --enable-libxvid --enable-lzma --enable-libfontconfig --enable-libfreetype --enable-frei0r --enable-libass --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenjpeg --enable-libspeex --enable-libsoxr --enable-libzmq --enable-libzimg --disable-libjack --disable-indev=jack --enable-videotoolbox --enable-audiotoolbox --enable-neon
  libavutil      59.  8.100 / 59.  8.100
  libavcodec     61.  3.100 / 61.  3.100
  libavformat    61.  1.100 / 61.  1.100
  libavdevice    61.  1.100 / 61.  1.100
  libavfilter    10.  1.100 / 10.  1.100
  libswscale      8.  1.100 /  8.  1.100
  libswresample   5.  1.100 /  5.  1.100
  libpostproc    58.  1.100 / 58.  1.100
2024-08-19 16:38:17.600 dvrescue[30719:35782559] WARNING: AVCaptureDeviceTypeExternal is deprecated for Continuity Cameras. Please use AVCaptureDeviceTypeContinuityCamera and add NSCameraUseContinuityCameraDeviceType to your Info.plist.
    nan    :  0.000 fd=   0 aq=    0KB vq=    0KB sq=    0B 

@libbyhopfauf
Copy link
Member Author

@dericed I just saw #914 regarding rewind and merge. So that might explain why it seemed like rewind and recapture was working even when I didn't have it as part of the command.

dericed and others added 3 commits August 28, 2024 19:20
@dericed this is ready to be merged! I updated the images to reflect how the GUI looks and what information is present for capture in the latest build.
I retested all of the CLI commands using a DSR-45 and dvrescue CLI version 2024-08-24 and updated this documentation to reflect that. All CLI commands and examples in this documentation have been updated to reflect their current usage. Ready to merge!
@libbyhopfauf
Copy link
Member Author

@dericed this is ready to be merged! I updated the images to reflect how the GUI looks and what information is present for capture in the latest build. I also tested and updated all of the CLI commands and examples. Everything works and is good to go!

@dericed dericed merged commit 74bdf5d into main Aug 29, 2024
26 checks passed
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.

2 participants