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

Inference Docs Overhaul #385

Merged
merged 3 commits into from
May 14, 2024
Merged

Conversation

LinasKo
Copy link
Contributor

@LinasKo LinasKo commented May 10, 2024

Description

Pages on the left now have a basic section.

The goal is to answer questions:

  • What is inference in the context of Roboflow (+ what other tools are available)
  • What kind of models can I run?
  • How can I get model_ids?
  • I have weights already - what do I do?

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How has this change been tested, please provide a testcase or example of how you tested the change?

mkdocs serve

Any specific deployment considerations

  1. Docs need to be deployed.
  2. It assumes the aliases for keypoint detection are available.

Docs

  • Docs updated.

Copy link
Collaborator

@PawelPeczek-Roboflow PawelPeczek-Roboflow left a comment

Choose a reason for hiding this comment

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

I like the changes, especially clarification of roboflow ecosystem and making index better - especially while looking at top-level entries top-to-bottom.

LGTM, but please make sure with Trevor or Brad that wording about license is as it should be (I do not feel 100% competent judging that)

@LinasKo LinasKo changed the title Revived Aliases page, wrote about roboflow ecosystem, fixed minor issues Inference Docs Overhaul May 14, 2024
@LinasKo
Copy link
Contributor Author

LinasKo commented May 14, 2024

Removed pricing for now. 1 more change fixing dead links, a few style issues is coming in the next 20 min

@PawelPeczek-Roboflow PawelPeczek-Roboflow self-requested a review May 14, 2024 08:08
Copy link
Collaborator

@PawelPeczek-Roboflow PawelPeczek-Roboflow left a comment

Choose a reason for hiding this comment

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

LGTM

@PawelPeczek-Roboflow PawelPeczek-Roboflow merged commit 9b6b56f into main May 14, 2024
34 checks passed
@PawelPeczek-Roboflow PawelPeczek-Roboflow deleted the docs/linas/inference-overhaul branch May 14, 2024 09:43
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.

2 participants