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

Aspect angle feature #8

Open
wants to merge 4 commits into
base: development
Choose a base branch
from

Conversation

karlgjelland
Copy link

Tried to include aspect (fish orientation). The calculation works and aspect is shown in detections table in the interface, but incorporation into a summary table fails. Hence export fails (except for header), and combining all tracks to a summary table fails. Tracking causes program crash at this point.

beamLookUp.BeamLookUp requires largeLens, not highResolution. Replaced "highResolution" with "0" in cls.firstBeamAngle = beamLookUp.BeamLookUp (Line 346), since there is no largeLens parameter in ddf3 file header.
largeLens condition changed to 0 = breakpoints48 and else telephoto48, to account for default no largeLens in DIDSON ddf3.
Replaced self.largeLens with 0 in BeamLookUp, since there is no largeLens parameter in ddf3 header
…alization with "fish_manager.py", line 955, in setPathVariables

    self.aspect = float(np.mean(valid_dets.aspect))
AttributeError: 'list' object has no attribute 'aspect'
"
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