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

STU-270 | Test Sequencer: export complexe data type produced in test #1118

Merged
merged 16 commits into from
Mar 11, 2024

Conversation

LatentDream
Copy link
Member

@LatentDream LatentDream commented Feb 28, 2024

Demo

Screen.Recording.2024-02-28.at.2.01.37.PM.mov

Copy link

vercel bot commented Feb 28, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 11, 2024 2:20am

@LatentDream LatentDream changed the title Ts cloud data STU-270 Ts cloud data Feb 28, 2024
@LatentDream LatentDream marked this pull request as ready for review February 28, 2024 19:05
@LatentDream LatentDream changed the title STU-270 Ts cloud data STU-270 | Test Sequencer: export complexe data type produced in test Mar 2, 2024
Copy link
Collaborator

@dstrande dstrande left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

May as well add a docs page in this PR so we don't have to refer back to this video for a how-to. Maybe upload this video to youtube (unlisted) and embed it in the docs? Can do that part later.

@dstrande
Copy link
Collaborator

dstrande commented Mar 4, 2024

One question. How does it know the test_id? If you have a test with multiple different test types?

@LatentDream
Copy link
Member Author

May as well add a docs page in this PR so we don't have to refer back to this video for a how-to. Maybe upload this video to youtube (unlisted) and embed it in the docs? Can do that part later.

Yes that's a good idea :)

@LatentDream
Copy link
Member Author

LatentDream commented Mar 4, 2024

One question. How does it know the test_id? If you have a test with multiple different test types?

Oh wait, did I forgot to handle that, let's me test it. Currently, the Test Sequencer retreive the last data exported with:
test_sequencer.export(...). and upload this one. I don't remember what happen when the test expect a bool but we give it a dataframe

@dstrande
Copy link
Collaborator

dstrande commented Mar 4, 2024

One question. How does it know the test_id? If you have a test with multiple different test types?

Oh does it just look at the function name and look for the test with that name?

@LatentDream
Copy link
Member Author

LatentDream commented Mar 4, 2024

One question. How does it know the test_id? If you have a test with multiple different test types?

Oh does it just look at the function name and look for the test with that name?

The test sequencer tells the test where to store the data based on the testName (in the UI).
Example of what going on in the background

  • That way, it works with Pytest, Python and it would works with Matlab in the future.

@LatentDream LatentDream dismissed dstrande’s stale review March 10, 2024 22:15

Let's do the docs in another PR, I'll need this feature for a demo this week

Copy link
Contributor

@izi-on izi-on left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@izi-on izi-on merged commit 091f5fa into flojoy-ai:main Mar 11, 2024
16 checks passed
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.

None yet

3 participants