Skip to content

Only emit statitics for multiple file operations#59

Merged
Dadstart merged 3 commits into
mainfrom
bugfix/single-stats
Jul 18, 2026
Merged

Only emit statitics for multiple file operations#59
Dadstart merged 3 commits into
mainfrom
bugfix/single-stats

Conversation

@Dadstart

@Dadstart Dadstart commented Jul 17, 2026

Copy link
Copy Markdown
Owner

Convert-VideoFile and Convert-MediaFiles now emit MediaConversionStatistics only when more than one file completed. A single conversion already reports size/duration on MediaConversionResult, so the averages line was redundant.

Also output sizes in megabytes.

Andrew Bishop added 3 commits July 17, 2026 11:39
…istics only when more than one file completed. A single conversion already reports size/duration on MediaConversionResult, so the averages line was redundant.

Invoke-BonusFileProcessing is unchanged and still writes statistics for any completed conversion count.
- Updated MediaConversionResult and MediaConversionStatistics to use InputSizeMegabytes and OutputSizeMegabytes instead of bytes for size reporting.
- Adjusted formatting in MediaForgePS.format.ps1xml to reflect changes in size representation.
- Enhanced MediaConversionHelper with methods for converting bytes to megabytes and formatting megabyte values.
- Updated documentation and tests to ensure consistency with the new size reporting format.
@Dadstart
Dadstart merged commit e1675a5 into main Jul 18, 2026
6 checks passed
@Dadstart
Dadstart deleted the bugfix/single-stats branch July 18, 2026 22:23
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.

1 participant