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

docs: image classification tutorial #491

Merged
merged 3 commits into from
Apr 3, 2023

Conversation

axiomofjoy
Copy link
Contributor

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

"metadata": {},
"outputs": [],
"source": [
"train_df = pd.read_parquet(\"https://storage.googleapis.com/arize-assets/fixtures/Embeddings/arize-demo-models-data/CV/human_actions_quality_drift_training.parquet\")\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Call df.head to explain the columns

Comment on lines 132 to 133
" prediction_label_column_name=\"pred_label\",\n",
" actual_label_column_name=\"label\",\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

since these columns are not self-sescriptive, I'd explain them

"cell_type": "markdown",
"metadata": {},
"source": [
"You can now label your exported data and fine-tune your model to improve performance on blurry and noisy images.\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Best to use wording that explains the value of the action - what is it buying the user? How is it making their job easier? Why should they care?

@axiomofjoy axiomofjoy merged commit f83f758 into main Apr 3, 2023
@axiomofjoy axiomofjoy deleted the docs-image-classification-tutorial branch April 3, 2023 00:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants