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

Benchmark QA support in the SDK #216

Merged
merged 4 commits into from Nov 30, 2022

Conversation

alexey-cord-tech
Copy link
Contributor

@alexey-cord-tech alexey-cord-tech commented Nov 14, 2022

Introduction and Explanation

Introducing support for 'shadow data' in the SDK.

JIRA

https://cord-team.atlassian.net/browse/DEV-2055

Documentation

Not quite.

Tests

SDK tests to follow

Known issues

I'm not entirely sure about the 'ProjectWorkflowType' thing.

@github-actions
Copy link

github-actions bot commented Nov 14, 2022

Unit test report

45 tests  ±0   35 ✔️ ±0   1s ⏱️ ±0s
  1 suites ±0   10 💤 ±0 
  1 files   ±0     0 ±0 

Results for commit 4d99b17. ± Comparison against base commit 530950e.

♻️ This comment has been updated with latest results.

encord/user_client.py Outdated Show resolved Hide resolved
@denis-cord
Copy link
Contributor

denis-cord commented Nov 23, 2022

As discussed, exposing the settings that you can choose on startup would be the right thing to do (can be just an enum and then send over a set/list of the enum entries). (I'm assuming here that all our settings that we want to expose are boolean, that might not be the case)

encord/user_client.py Outdated Show resolved Hide resolved
encord/user_client.py Outdated Show resolved Hide resolved
encord/user_client.py Outdated Show resolved Hide resolved
Copy link
Contributor

@denis-cord denis-cord left a comment

Choose a reason for hiding this comment

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

Some things to consider.

Copy link
Contributor

@denis-cord denis-cord left a comment

Choose a reason for hiding this comment

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

One comment left for somewhere to clarify. Also, there will be a project.rst which walks you through the project creation. Do you think a sentence about project modes and a link maybe to the relevant in code docstrings would be worth there?

@alexey-cord-tech
Copy link
Contributor Author

I would wait with updating the "scenario" documentation for a bit, and when update it once we build the "class weights" thing and expose it here. Right now this is mostly a Tractable-specific thing and we can just point them to the right place in the docs.

Copy link
Contributor

@denis-cord denis-cord left a comment

Choose a reason for hiding this comment

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

Thanks for being so thorough here. It's fine for now if you're in a rush (however you'll need for the BE to roll out anyways).

I'd still recommend you going here to see that it would be nice to add only 2 bullet points, one with the ontology_hash, one with the workflow_settings. They can pretty much link to the new objects you have defined nicely.

You will find the file in code under docs/source/tutorials/project.rst

@alexey-cord-tech alexey-cord-tech merged commit 8942c65 into master Nov 30, 2022
@alexey-cord-tech alexey-cord-tech deleted the ak/shadow-data-filter-and-view branch November 30, 2022 09:53
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

2 participants