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

Hypermode docs review changes #62

Merged
merged 7 commits into from
Nov 26, 2024
Merged

Conversation

jacklynlee31
Copy link
Contributor

@jacklynlee31 jacklynlee31 commented Nov 14, 2024

➡️ Writing down thoughts & making notes in this Notion doc.


Some comments/questions that are in the Notion doc but not captured in my changes:

Getting Started > Quickstart

  • Not sure how to get content to appear in Inferences. Do I need to build a model first? If so, we should mention that in this section, since the Observe function execution step makes it seem like all I had to do was run the query to get info on the Inferences page.
    • You do need to invoke the model from the function code first, so I added that into the docs just for extra clarification.

Develop > Work Locally
- The curl command for installing Hyp CLI didn't work for me, but the npm command (on the Hyp CLI page) did work.
The curl command was fixed. See this Slack thread for more details.

Develop > Configure Environment & Tools > Hosted Models

  • On the Configure Environment page, under Model hosting: "refer to the Hosted Models documentation for instructions on setting up dedicated models" - I'm only seeing references to shared models on that page, is there a difference between the two?

⏸️ Manage > Modify Organization

  • This forms the billing envelope for Hypermode. - what does this mean?
  • Comment in Notion doc: Means that Billing will be at the org level. Still thinking on this but I think it's okay to leave this as-is for now.

There are other questions/suggestions in the Notion file but I pulled the ones that I thought were higher priority into this PR.

create-project.mdx Show resolved Hide resolved
introduction.mdx Outdated Show resolved Hide resolved
work-locally.mdx Outdated Show resolved Hide resolved
work-locally.mdx Outdated Show resolved Hide resolved
@jacklynlee31 jacklynlee31 dismissed ryanfoxtyler’s stale review November 25, 2024 17:42

resolved - info is still there, just moved

@johnymontana johnymontana self-requested a review November 26, 2024 01:23
@ryanfoxtyler ryanfoxtyler merged commit dee98fc into main Nov 26, 2024
5 checks passed
@ryanfoxtyler ryanfoxtyler deleted the jacklyn/hypermode-changes branch November 26, 2024 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants