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.

Sign up for GitHub

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

Jump to bottom

[NEW-FEATURE] AxoDatamans' method 'Read' should have an output parameter for the result. #171

Closed
1 task
PTKu opened this issue Aug 16, 2023 · 2 comments · Fixed by #212
Closed
1 task

[NEW-FEATURE] AxoDatamans' method 'Read' should have an output parameter for the result. #171

PTKu opened this issue Aug 16, 2023 · 2 comments · Fixed by #212
Assignees
@TomKovac @Brano5
Labels
cognex component
Milestone
0.3

Comments

@PTKu
Copy link
Member

PTKu commented Aug 16, 2023 •
edited
Loading

[Method Read should have an output parameter where the result will be stored upon task completion]
(

AXOpen/src/components.cognex.vision/ctrl/src/AxoDataman/v_6_0_0/AxoDataman.st

Line 387 in f2cc119

METHOD PUBLIC Read : IAxoTaskState
)

The consumer call should look like

IF(MyDataman1.Read(myString).IsDone()) THEN
 //some logic
END_IF

instead of

IF MyDataman1.Read().IsDone() THEN
    //Process the result data
    myString := MyDataman1.Results.Data;
END_IF;
  • Update also the article in the documentation.
@PTKu PTKu added this to the 0.2 milestone Aug 16, 2023
@PTKu PTKu changed the title [NEW-FEATURE] CongexDMC reader; method 'Read' should have an output parameter for the result. [NEW-FEATURE] AxoDatamans' method 'Read' should have an output parameter for the result. Aug 16, 2023
@PTKu PTKu modified the milestones: 0.2, 0.3 Aug 31, 2023
@Brano5
Copy link
Contributor

Brano5 commented Sep 11, 2023

/cib

@IX-BOT
Copy link
Contributor

IX-BOT commented Sep 11, 2023

Branch 171-_NEW-FEATURE_AxoDatamans_method_Read_should_have_an_output_parameter_for_the_result created!

IX-BOT pushed a commit that referenced this issue Sep 11, 2023
@Brano5
Create draft PR for #171
5285a2d
@IX-BOT IX-BOT mentioned this issue Sep 11, 2023
Merged
@PTKu PTKu closed this as completed in #212 Sep 12, 2023
PTKu added a commit that referenced this issue Sep 12, 2023
@IX-BOT @Brano5 @PTKu
[NEW-FEATURE] AxoDatamans' method 'Read' should have an output parame…
88f7db7 
…ter for the result. (#212)

* Create draft PR for #171

* Add output parameter into Read method. Change docs

---------

Co-authored-by: Brano5 <Brano5@users.noreply.github.com>
Co-authored-by: Branko Zachemsky <bzachemsky@gmail.com>
Co-authored-by: Peter Kurhajec <61538034+PTKu@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TomKovac TomKovac

@Brano5 Brano5

Labels
cognex component
Projects
None yet
Milestone
0.3
4 participants
@PTKu @TomKovac @Brano5 @IX-BOT