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

Concatenate Index Files fails for some inputs #170

Closed
atruskie opened this issue Jun 18, 2018 · 3 comments
Closed

Concatenate Index Files fails for some inputs #170

atruskie opened this issue Jun 18, 2018 · 3 comments
Assignees
Labels

Comments

@atruskie
Copy link
Member

atruskie commented Jun 18, 2018

Expected behaviour

ConcatenateIndexFiles should work.

Actual behaviour

A System.ArgumentOutOfRangeException: Index was out of range. Must be non-negative and less than the size of the collection. exception is thrown.

Steps to reproduce the behaviour

  • What command did you use?
    • "F:\FalseColor\Run-ParallelJobs\131738377986629221\0\AP\AnalysisPrograms.exe" concatenateindexfiles "Y:\Results\20180608-103353 - Tshering, Towsey.Indices, #216\Tshering\WBH_Walaytar\WBH_2018\Bermo" -o "Y:\Results\20180608-103353 - Tshering, Towsey.Indices, #216\ConcatResult\TimedGaps\Tshering\WBH_Walaytar\WBH_2018" --directory-filter *.wav -ip "Y:\Results\20180608-103353 - Tshering, Towsey.Indices, #216\IndexPropertiesConfig.yml" -fcs "Y:\Results\20180608-103353 - Tshering, Towsey.Indices, #216\SpectrogramFalseColourConfig.yml" -f Bermo -z 06:00 --gap-rendering TimedGaps --draw-images true -n
  • What config file did you use?
    • The standard config files
  • Where can we find the source data used?
    • Y:\Results\20180608-103353 - Tshering, Towsey.Indices, #216\Tshering\WBH_Walaytar\WBH_2018\Bermo
  • Program version?
    • QUT Ecoacoustics Analysis Programs - version 18.6.0.34 (RELEASE build, 2018-06-10T14:02:04.2309201Z) Git branch-version: master-2cadfe7f0e67684751371a1deaf2224fd0ed6c78, DirtyBuild:False, CI:447

Any other details

It appears to crash on the second day.

Log: Y:\Results\20180608-103353 - Tshering, Towsey.Indices, #216\ConcatResult\TimedGaps\Tshering\WBH_Walaytar\WBH_2018\Bermo\log.txt

@atruskie
Copy link
Member Author

atruskie commented Aug 9, 2018

This bug is repeatable in other datasets:

Y:\Results\20180802-125012 - Hongxiao, Towsey.Indices, ICD=15.0, #225\ConcatResult\TimedGaps\Hongxiao\KiwiRecordings\log_20180807T040638Z.txt

"F:\FalseColor\Run-ParallelJobs\131780883729240605\3\AP\AnalysisPrograms.exe" concatenateindexfiles "Y:\Results\20180802-125012 - Hongxiao, Towsey.Indices, ICD=15.0, #225\Hongxiao\KiwiRecordings" -o "Y:\Results\20180802-125012 - Hongxiao, Towsey.Indices, ICD=15.0, #225\ConcatResult\TimedGaps\Hongxiao" --directory-filter *.wav -ip "Y:\Results\20180802-125012 - Hongxiao, Towsey.Indices, ICD=15.0, #225\IndexPropertiesConfig.yml" -fcs "Y:\Results\20180802-125012 - Hongxiao, Towsey.Indices, ICD=15.0, #225\SpectrogramFalseColourConfig.yml" -f KiwiRecordings -z 10:00 --gap-rendering TimedGaps --draw-images -n
QUT Ecoacoustics Analysis Programs - version 18.7.1.11 (RELEASE build, 2018-07-29T14:02:47.4284203Z)
Git branch-version: master-d4654f21196a1892d3737edb04e7f20351c2bec9, DirtyBuild:False, CI:50

@atruskie
Copy link
Member Author

I've replicated this bug again:
log_20181018T032015Z.txt

towsey added a commit that referenced this issue Jan 31, 2019
Issue #170 turns out to be connected with the file join Issue #210. I believe I have fixed both issues here. However currently file joins are showing both in summary and LDFC images. TODO: remove file join indication from the LDFCs as they are not necessary.
towsey added a commit that referenced this issue Feb 12, 2019
Fix and simplify  code for drawing file joins when concatenating files. Much of the remaining code changes were driven by resharper.
atruskie pushed a commit that referenced this issue Feb 13, 2019
Issue #170 turns out to be connected with the file join Issue #210. I believe I have fixed both issues here. However currently file joins are showing both in summary and LDFC images. TODO: remove file join indication from the LDFCs as they are not necessary.
atruskie pushed a commit that referenced this issue Feb 13, 2019
Fix and simplify  code for drawing file joins when concatenating files. Much of the remaining code changes were driven by resharper.
@atruskie
Copy link
Member Author

atruskie commented Feb 13, 2019

Closed by 981696d and ce80cbc

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

No branches or pull requests

2 participants