Implemented AcquisitionFrameCount on Fake-camera #699

LorenzoBalducci96 · 2022-07-23T16:13:55Z

The first frame is immediately taken, but maybe is correct to wait the framePeriod also on the first frame?

…erted)

EmmanuelP

Hi,

Thanks for your work on this feature. It could go in, but there is several things to fix first.

In addition to the other comments, there is also this 2 points:

FrameCount should only taken into account if AcquisitionMode is MultiFrame.

I'm not sure, but it looks like the counter is not reset at every AcquisitionStart.

EmmanuelP · 2022-07-25T06:24:42Z

.gitignore

+install/
+
+conanfile.txt
+## Ignore Visual Studio temporary files, build results, and
+## files generated by popular Visual Studio add-ons.
+
+# User-specific files
+*.suo
+*.user
+*.userosscache
+*.sln.docstates
+
+# User-specific files (MonoDevelop/Xamarin Studio)
+*.userprefs
+
+# Build results
+[Dd]ebug/
+[Dd]ebugPublic/
+[Rr]elease/
+[Rr]eleases/
+x64/
+x86/
+bld/
+[Bb]in/
+[Oo]bj/
+[Ll]og/
+
+# Visual Studio 2015 cache/options directory
+.vs/
+# Uncomment if you have tasks that create the project's static files in wwwroot
+#wwwroot/
+
+# MSTest test Results
+[Tt]est[Rr]esult*/
+[Bb]uild[Ll]og.*
+
+# NUNIT
+*.VisualState.xml
+TestResult.xml
+
+# Build Results of an ATL Project
+[Dd]ebugPS/
+[Rr]eleasePS/
+dlldata.c
+
+# DNX
+project.lock.json
+artifacts/
+
+*_i.c
+*_p.c
+*_i.h
+*.ilk
+*.meta
+*.obj
+*.pch
+*.pdb
+*.pgc
+*.pgd
+*.rsp
+*.sbr
+*.tlb
+*.tli
+*.tlh
+*.tmp
+*.tmp_proj
+*.log
+*.vspscc
+*.vssscc
+.builds
+*.pidb
+*.svclog
+*.scc
+
+# Chutzpah Test files
+_Chutzpah*
+
+# Visual C++ cache files
+ipch/
+*.aps
+*.ncb
+*.opendb
+*.opensdf
+*.sdf
+*.cachefile
+
+# Visual Studio profiler
+*.psess
+*.vsp
+*.vspx
+*.sap
+
+# TFS 2012 Local Workspace
+$tf/
+
+# Guidance Automation Toolkit
+*.gpState
+
+# ReSharper is a .NET coding add-in
+_ReSharper*/
+*.[Rr]e[Ss]harper
+*.DotSettings.user
+
+# JustCode is a .NET coding add-in
+.JustCode
+
+# TeamCity is a build add-in
+_TeamCity*
+
+# DotCover is a Code Coverage Tool
+*.dotCover
+
+# NCrunch
+_NCrunch_*
+.*crunch*.local.xml
+nCrunchTemp_*
+
+# MightyMoose
+*.mm.*
+AutoTest.Net/
+
+# Web workbench (sass)
+.sass-cache/
+
+# Installshield output folder
+[Ee]xpress/
+
+# DocProject is a documentation generator add-in
+DocProject/buildhelp/
+DocProject/Help/*.HxT
+DocProject/Help/*.HxC
+DocProject/Help/*.hhc
+DocProject/Help/*.hhk
+DocProject/Help/*.hhp
+DocProject/Help/Html2
+DocProject/Help/html
+
+# Click-Once directory
+publish/
+
+# Publish Web Output
+*.[Pp]ublish.xml
+*.azurePubxml
+# TODO: Comment the next line if you want to checkin your web deploy settings 
+# but database connection strings (with potential passwords) will be unencrypted
+*.pubxml
+*.publishproj
+
+# NuGet Packages
+*.nupkg
+# The packages folder can be ignored because of Package Restore
+**/packages/*
+# except build/, which is used as an MSBuild target.
+!**/packages/build/
+# Uncomment if necessary however generally it will be regenerated when needed
+#!**/packages/repositories.config
+# NuGet v3's project.json files produces more ignoreable files
+*.nuget.props
+*.nuget.targets
+
+# Microsoft Azure Build Output
+csx/
+*.build.csdef
+
+# Microsoft Azure Emulator
+ecf/
+rcf/
+
+# Windows Store app package directories and files
+AppPackages/
+BundleArtifacts/
+Package.StoreAssociation.xml
+_pkginfo.txt
+
+# Visual Studio cache files
+# files ending in .cache can be ignored
+*.[Cc]ache
+# but keep track of directories ending in .cache
+!*.[Cc]ache/
+
+# Others
+ClientBin/
+~$*
+*~
+*.dbmdl
+*.dbproj.schemaview
+*.pfx
+*.publishsettings
+node_modules/
+orleans.codegen.cs
+
+# Since there are multiple workflows, uncomment next line to ignore bower_components 
+# (https://github.com/github/gitignore/pull/1529#issuecomment-104372622)
+#bower_components/
+
+# RIA/Silverlight projects
+Generated_Code/
+
+# Backup & report files from converting an old project file
+# to a newer Visual Studio version. Backup files are not needed,
+# because we have git ;-)
+_UpgradeReport_Files/
+Backup*/
+UpgradeLog*.XML
+UpgradeLog*.htm
+
+# SQL Server files
+*.mdf
+*.ldf
+
+# Business Intelligence projects
+*.rdl.data
+*.bim.layout
+*.bim_*.settings
+
+# Microsoft Fakes
+FakesAssemblies/
+
+# GhostDoc plugin setting file
+*.GhostDoc.xml
+
+# Node.js Tools for Visual Studio
+.ntvs_analysis.dat
+
+# Visual Studio 6 build log
+*.plg
+
+# Visual Studio 6 workspace options file
+*.opt
+
+# Visual Studio LightSwitch build output
+**/*.HTMLClient/GeneratedArtifacts
+**/*.DesktopClient/GeneratedArtifacts
+**/*.DesktopClient/ModelManifest.xml
+**/*.Server/GeneratedArtifacts
+**/*.Server/ModelManifest.xml
+_Pvt_Extensions
+
+# Paket dependency manager
+.paket/paket.exe
+paket-files/
+
+# FAKE - F# Make
+.fake/
+
+# JetBrains Rider
+.idea/
+*.sln.iml


Please don't include unrelated changes.

EmmanuelP · 2022-07-25T06:26:14Z

src/arvfakecamera.c


 	fake_camera->priv->trigger_frequency = 25.0;
 	fake_camera->priv->frame_id = 65400; /* Trigger circular counter bugs sooner */
+	fake_camera->priv->frameSeqCount = 65535;//max possible value by default for acquisition count


don't use C++ style comments

EmmanuelP · 2022-07-25T06:27:41Z

src/arvgvfakecamera.c

 	input_vector.size = ARV_GV_FAKE_CAMERA_BUFFER_SIZE;

 	packet_buffer = g_malloc (ARV_GV_FAKE_CAMERA_BUFFER_SIZE);



It looks like there is no real change in this file.

EmmanuelP · 2022-07-25T06:28:50Z

src/arvfakecamera.c

+guint64
+arv_fake_camera_get_frame_period(ArvFakeCamera* camera)
+{
+	return _get_register (camera, ARV_FAKE_CAMERA_REGISTER_ACQUISITION_FRAME_PERIOD_US);
+}
+
+int
+arv_fake_camera_get_acquisition_frame_count (ArvFakeCamera *camera)
+{
+	return _get_register (camera, ARV_FAKE_CAMERA_ACQUISITION_FRAME_COUNT);
+}
+
 void


These functions are never used.

EmmanuelP · 2022-07-25T06:35:16Z

src/arvfakecamera.c


 	guint32 frame_id;
 	double trigger_frequency;
+        int frameSeqCount;


Should be multi_frame_count instead of frameSeqCount.

EmmanuelP · 2022-07-25T06:36:40Z

src/arv-fake-camera.xml

+    <Max>65535</Max>
+  </Integer>
+
+  <IntReg Name="AcquisitionFrameCountRegister" NameSpace="Custom">
+    <Address>0x310</Address>
+    <Length>4</Length>


Use a 64bit integer, 65535 seems to small.

EmmanuelP · 2022-07-25T06:49:49Z

Fake camera should stick to the behavior describe in the Genicam SNFC document: https://www.emva.org/wp-content/uploads/GenICam_SFNC_v2_7.pdf

LorenzoBalducci96 · 2022-07-25T20:12:07Z

Hi thanks for the interest.

From the Genicam SNFC document I read: "AcquisitionFrameRate controls the rate at which the Frames are captured when TriggerMode is Off.".
So it seems that this parameter must be taken into account when not using triggered mode acquisition. (so I guess it defines the framerate when acquisitionMode is continous). So it should be correct to not take into account the framerate when sending multiFrames in triggered mode and send frames immediately as soon as are available in the fake-camera?.
However in basler aca1920-50gm/c you can set the AcquisitionFrameRateAbs parameter when acquiring multiFrame in triggered mode and it will define the framerate of the frames acquired after a trigger.
Maybe basler cameras are slightly out of specification?

"the counter is not reset at every AcquisitionStart"
in arv_fake_camera_check_and_acknowledge_software_trigger when the trigger is acknoledged the frameCount is reset to 1 and is increased every new frame. Since the frameCount is smaller than AcquisitionFrameCount register the arv_fake_camera_check_and_acknowledge_software_trigger still returns true meaning that a new frame must be taken (the trigger was already acknoledged at first frame). Once the acquisitionFrameCount is reached, the frameCount is set to 65535 in order to be bigger than any other acquisitionFrameCount that could be set before the next trigger causes the reset of the frameCount.
Do you propose a better solution?

every other notes are for shure reasonable for the next commit.

EmmanuelP · 2022-08-22T07:41:02Z

From the Genicam SNFC document I read: "AcquisitionFrameRate controls the rate at which the Frames are captured when TriggerMode is Off.". So it seems that this parameter must be taken into account when not using triggered mode acquisition. (so I guess it defines the framerate when acquisitionMode is continous). So it should be correct to not take into account the framerate when sending multiFrames in triggered mode and send frames immediately as soon as are available in the fake-camera?.

Yes.

However in basler aca1920-50gm/c you can set the AcquisitionFrameRateAbs parameter when acquiring multiFrame in triggered mode and it will define the framerate of the frames acquired after a trigger. Maybe basler cameras are slightly out of specification?

Most camera manufacturers don't strictly stick to the SNFC, mostly because they started the work on their firmware and SDK before the SNFC even exists.

"the counter is not reset at every AcquisitionStart" in arv_fake_camera_check_and_acknowledge_software_trigger when the trigger is acknoledged the frameCount is reset to 1 and is increased every new frame. Since the frameCount is smaller than AcquisitionFrameCount register the arv_fake_camera_check_and_acknowledge_software_trigger still returns true meaning that a new frame must be taken (the trigger was already acknoledged at first frame). Once the acquisitionFrameCount is reached, the frameCount is set to 65535 in order to be bigger than any other acquisitionFrameCount that could be set before the next trigger causes the reset of the frameCount. Do you propose a better solution?

There is no reason to couple AcquisitionFrameCount and TriggerSoftware, they are unrelated. AcquisitionFrameCount is here to set the number of acuiqred frames when AcuisitionMode is set to MultiFrame, wether the camera is free running or triggered.

LorenzoBalducci96 added 3 commits July 23, 2022 17:58

Implemented AcquisitionFrameCount in fakeCamera

75e7c62

Implementerd AcquisitionFrameCount on fake-camera (old code style rev…

0acb090

…erted)

Implemented AcquisitionFrameCount on FakeCamera

a5b489c

EmmanuelP requested changes Jul 25, 2022

View reviewed changes

EmmanuelP marked this pull request as draft August 22, 2022 09:06

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Implemented AcquisitionFrameCount on Fake-camera #699

Implemented AcquisitionFrameCount on Fake-camera #699

Uh oh!

LorenzoBalducci96 commented Jul 23, 2022

Uh oh!

EmmanuelP left a comment •

edited

Loading

Uh oh!

EmmanuelP Jul 25, 2022

Uh oh!

EmmanuelP Jul 25, 2022

Uh oh!

EmmanuelP Jul 25, 2022

Uh oh!

EmmanuelP Jul 25, 2022

Uh oh!

EmmanuelP Jul 25, 2022

Uh oh!

EmmanuelP Jul 25, 2022

Uh oh!

EmmanuelP commented Jul 25, 2022

Uh oh!

LorenzoBalducci96 commented Jul 25, 2022

Uh oh!

EmmanuelP commented Aug 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		input_vector.size = ARV_GV_FAKE_CAMERA_BUFFER_SIZE;

		packet_buffer = g_malloc (ARV_GV_FAKE_CAMERA_BUFFER_SIZE);

Uh oh!

Implemented AcquisitionFrameCount on Fake-camera #699

Are you sure you want to change the base?

Implemented AcquisitionFrameCount on Fake-camera #699

Uh oh!

Conversation

LorenzoBalducci96 commented Jul 23, 2022

Uh oh!

EmmanuelP left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

EmmanuelP Jul 25, 2022

Choose a reason for hiding this comment

Uh oh!

EmmanuelP Jul 25, 2022

Choose a reason for hiding this comment

Uh oh!

EmmanuelP Jul 25, 2022

Choose a reason for hiding this comment

Uh oh!

EmmanuelP Jul 25, 2022

Choose a reason for hiding this comment

Uh oh!

EmmanuelP Jul 25, 2022

Choose a reason for hiding this comment

Uh oh!

EmmanuelP Jul 25, 2022

Choose a reason for hiding this comment

Uh oh!

EmmanuelP commented Jul 25, 2022

Uh oh!

LorenzoBalducci96 commented Jul 25, 2022

Uh oh!

EmmanuelP commented Aug 22, 2022

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

EmmanuelP left a comment •

edited

Loading