Skip to content

Conversation

Blahpapto1
Copy link
Contributor

@Blahpapto1 Blahpapto1 commented Dec 17, 2024

What does this Pull Request accomplish?

Adding Utility Methods in SampleMeasurement Example to convert Double2DArray To/From NDArray and List(List(Float)). Also, updating SampleMeasurement to demonstrate the methods.

Why should this Pull Request be merged?

Adding Utility methods for Double2DArray.

What testing has been done?

  • Conversion from all the methods produce desired output type.
  • Measurement is running fine after using the conversion method.
  • Tested Sample Measurement on MeasUI Editor by dropping a new "Converted Double 2D Array Out" indicator, which produce desired behavior.

@Blahpapto1 Blahpapto1 changed the title Add Utility Method in SampleMeasurement to convert To/From Double2DArray, NDArray and List(List(Float)) Add Utility Method in SampleMeasurement to convert Double2DArray To/From NDArray and List(List(Float)) Dec 17, 2024
Copy link
Contributor

github-actions bot commented Dec 17, 2024

Test Results

    40 files  ±  0      40 suites  ±0   56m 31s ⏱️ +7s
   702 tests +  4     702 ✅ +  4      0 💤 ±0  0 ❌ ±0 
17 060 runs  +120  15 990 ✅ +120  1 070 💤 ±0  0 ❌ ±0 

Results for commit 25ea96c. ± Comparison against base commit 4002977.

♻️ This comment has been updated with latest results.

@Blahpapto1 Blahpapto1 force-pushed the users/avgoel/add-utilities branch from 06bd431 to 2eba360 Compare December 18, 2024 08:00
@Blahpapto1 Blahpapto1 marked this pull request as ready for review December 18, 2024 09:48
@Blahpapto1 Blahpapto1 merged commit 55e4c76 into main Dec 19, 2024
17 checks passed
@dixonjoel dixonjoel deleted the users/avgoel/add-utilities branch December 19, 2024 20:02
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.

3 participants